commit 20776cc370cf1f092b73c356c02a0df966cfd185
author Michael Kolb <kolby@google.com> Mon Jan 31 10:19:05 2011 -0800
committer Michael Kolb <kolby@google.com> Mon Jan 31 10:19:10 2011 -0800
tree 0b73b07572c470911f825add2e4d564df6496f2d
parent 5179ce5cad806a4cb3e273b28877b83e8297bb4e

fix IME bug

    http://b/issue?id=3406328
    http://b/issue?id=3406389
    hide title bar before removing tab to avoid IME triggered by
    focus change
    removed unnecessary call to hideComboView

Change-Id: Ia559bae8bc98aed9325e715696f20833ecf5916f

src/com/android/browser/BaseUi.java

diff --git a/src/com/android/browser/BaseUi.java b/src/com/android/browser/BaseUi.java
index 40bc16e..756f8b8 100644
--- a/src/com/android/browser/BaseUi.java
+++ b/src/com/android/browser/BaseUi.java
@@ -292,6 +292,7 @@
     }
 
     private void removeTabFromContentView(Tab tab) {
+        hideFakeTitleBar();
         // Remove the container that contains the main WebView.
         WebView mainView = tab.getWebView();
         View container = tab.getViewContainer();
@@ -301,6 +302,7 @@
         // Remove the container from the content and then remove the
         // WebView from the container. This will trigger a focus change
         // needed by WebView.
+        mainView.setEmbeddedTitleBar(null);
         FrameLayout wrapper =
                 (FrameLayout) container.findViewById(R.id.webview_wrapper);
         wrapper.removeView(mainView);
@@ -311,7 +313,6 @@
         if (errorConsole != null) {
             mErrorConsoleContainer.removeView(errorConsole);
         }
-        mainView.setEmbeddedTitleBar(null);
     }
 
     @Override