summaryrefslogtreecommitdiff
path: root/src/com/android/browser/Controller.java
diff options
context:
space:
mode:
authorMichael Kolb <kolby@google.com>2010-11-19 14:49:34 -0800
committerMichael Kolb <kolby@google.com>2010-11-22 09:13:50 -0800
commit77df4568331d527f993f4d69168218ddf5acddda (patch)
tree2cdef545065beb3a3b319978f901e8f7160289e9 /src/com/android/browser/Controller.java
parente8a28338fcd94ef17506b14fed9ebed7dec1124a (diff)
fix current tab has no webview bug
http://b/issue?id=3214823 setCurrentTab is called before setting the active tab in the UI this guarantees the active tab has a webview Change-Id: I5d56c42be5a19389d7bc2cc0c41039bb90cc8478
Diffstat (limited to 'src/com/android/browser/Controller.java')
-rw-r--r--src/com/android/browser/Controller.java5
1 files changed, 2 insertions, 3 deletions
diff --git a/src/com/android/browser/Controller.java b/src/com/android/browser/Controller.java
index 79fff3dc..aabdb266 100644
--- a/src/com/android/browser/Controller.java
+++ b/src/com/android/browser/Controller.java
@@ -1903,10 +1903,9 @@ public class Controller
}
protected void setActiveTab(Tab tab) {
- // Update the UI before setting the current tab in TabControl
- // so the UI can access the old tab to switch over from
- mUi.setActiveTab(tab);
mTabControl.setCurrentTab(tab);
+ // the tab is guaranteed to have a webview after setCurrentTab
+ mUi.setActiveTab(tab);
}
protected void closeEmptyChildTab() {