diff options
| author | Michael Kolb <kolby@google.com> | 2010-11-19 14:49:34 -0800 |
|---|---|---|
| committer | Michael Kolb <kolby@google.com> | 2010-11-22 09:13:50 -0800 |
| commit | 77df4568331d527f993f4d69168218ddf5acddda (patch) | |
| tree | 2cdef545065beb3a3b319978f901e8f7160289e9 /src/com/android/browser/Controller.java | |
| parent | e8a28338fcd94ef17506b14fed9ebed7dec1124a (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.java | 5 |
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() { |
