fix qc urlbar position and show in auto-login

    Bug: 4108540
    Show the url bar when auto-login is in progress
    Move url bar to top in QuickControls to work with instant
    search
    Remove suggestions reversal

Change-Id: I0040fa74dd26b5eecdab278c0104199fe5269aba

4 files changed, 22 insertions, 27 deletions

diff --git a/src/com/android/browser/SuggestionsAdapter.java b/src/com/android/browser/SuggestionsAdapter.java
index 6a9111f24..ecdaa15d9 100644
--- a/src/com/android/browser/SuggestionsAdapter.java
+++ b/src/com/android/browser/SuggestionsAdapter.java
@@ -70,7 +70,6 @@ public class SuggestionsAdapter extends BaseAdapter implements Filterable,
     final int mLinesLandscape;
     final Object mResultsLock = new Object();
     List<String> mVoiceResults;
-    boolean mReverseResults;
     boolean mIncognitoMode;
 
     interface CompletionListener {
@@ -137,9 +136,6 @@ public class SuggestionsAdapter extends BaseAdapter implements Filterable,
 
     @Override
     public SuggestItem getItem(int position) {
-        if (mReverseResults) {
-            position = (getCount() - 1) - position;
-        }
         if (mVoiceResults != null) {
             SuggestItem item = new SuggestItem(mVoiceResults.get(position),
                     null, TYPE_VOICE_SEARCH);
@@ -152,10 +148,6 @@ public class SuggestionsAdapter extends BaseAdapter implements Filterable,
         return mMixedResults.items.get(position);
     }
 
-    public void setReverseResults(boolean reverse) {
-        mReverseResults = reverse;
-    }
-
     @Override
     public long getItemId(int position) {
         return position;
diff --git a/src/com/android/browser/TitleBarXLarge.java b/src/com/android/browser/TitleBarXLarge.java
index be5e90fc1..ccbb38de6 100644
--- a/src/com/android/browser/TitleBarXLarge.java
+++ b/src/com/android/browser/TitleBarXLarge.java
@@ -211,20 +211,33 @@ public class TitleBarXLarge extends TitleBarBase
                 default:
                     throw new IllegalStateException();
             }
-            if (animate) {
-                mAutoLogin.startAnimation(AnimationUtils.loadAnimation(
-                        getContext(), R.anim.autologin_enter));
+            if (mUseQuickControls) {
+                mUi.showTitleBar();
+            } else {
+                if (animate) {
+                    mAutoLogin.startAnimation(AnimationUtils.loadAnimation(
+                            getContext(), R.anim.autologin_enter));
+                }
             }
         } else {
             mAutoLoginHandler = null;
-            if (animate) {
-                hideAutoLogin();
-            } else if (mAutoLogin.getAnimation() == null) {
+            if (mUseQuickControls) {
+                mUi.hideTitleBar();
                 mAutoLogin.setVisibility(View.GONE);
+            } else {
+                if (animate) {
+                    hideAutoLogin();
+                } else if (mAutoLogin.getAnimation() == null) {
+                    mAutoLogin.setVisibility(View.GONE);
+                }
             }
         }
     }
 
+    boolean inAutoLogin() {
+        return mAutoLoginHandler != null;
+    }
+
     private ViewGroup.LayoutParams makeLayoutParams() {
         if (mUseQuickControls) {
             return new FrameLayout.LayoutParams(LayoutParams.MATCH_PARENT,
@@ -247,12 +260,11 @@ public class TitleBarXLarge extends TitleBarBase
 
     void setUseQuickControls(boolean useQuickControls) {
         mUseQuickControls = useQuickControls;
-        mUrlInput.setUseQuickControls(mUseQuickControls);
         setLayoutParams(makeLayoutParams());
     }
 
     void setShowProgressOnly(boolean progress) {
-        if (progress) {
+        if (progress && !inAutoLogin()) {
             mContainer.setVisibility(View.GONE);
         } else {
             mContainer.setVisibility(View.VISIBLE);
diff --git a/src/com/android/browser/UrlInputView.java b/src/com/android/browser/UrlInputView.java
index b7f2bff1f..350d772bd 100644
--- a/src/com/android/browser/UrlInputView.java
+++ b/src/com/android/browser/UrlInputView.java
@@ -127,10 +127,6 @@ public class UrlInputView extends SuggestiveAutoCompleteTextView
         setCustomSelectionActionModeCallback(urlSelectionMode);
     }
 
-    void setUseQuickControls(boolean useQuickControls) {
-        mAdapter.setReverseResults(useQuickControls);
-    }
-
     void setContainer(View container) {
         mContainer = container;
     }
diff --git a/src/com/android/browser/XLargeUi.java b/src/com/android/browser/XLargeUi.java
index 02533b035..13a055703 100644
--- a/src/com/android/browser/XLargeUi.java
+++ b/src/com/android/browser/XLargeUi.java
@@ -107,17 +107,16 @@ public class XLargeUi extends BaseUi implements ScrollListener {
             if ((tab != null) && (tab.getWebView() != null)) {
                 tab.getWebView().setEmbeddedTitleBar(null);
             }
-            setTitleGravity(Gravity.BOTTOM);
         } else {
             mActivity.getActionBar().show();
             if (mPieControl != null) {
                 mPieControl.removeFromContainer(mContentView);
             }
-            setTitleGravity(Gravity.TOP);
             WebView web = mTabControl.getCurrentWebView();
             if (web != null) {
                 web.setEmbeddedTitleBar(mTitleBar);
             }
+            setTitleGravity(Gravity.NO_GRAVITY);
         }
         mTabBar.setUseQuickControls(mUseQuickControls);
     }
@@ -190,11 +189,10 @@ public class XLargeUi extends BaseUi implements ScrollListener {
         if (tab.inForeground()) {
             mTitleBar.setProgress(progress);
             if (progress == 100) {
-                if (!mTitleBar.isEditingUrl()) {
+                if (!mTitleBar.isEditingUrl() && !mTitleBar.inAutoLogin()) {
                     hideTitleBar();
                     if (mUseQuickControls) {
                         mTitleBar.setShowProgressOnly(false);
-                        setTitleGravity(Gravity.BOTTOM);
                     }
                 }
             } else {
@@ -450,9 +448,6 @@ public class XLargeUi extends BaseUi implements ScrollListener {
             // if the page is loading, show it again
             if (mUseQuickControls) {
                 mTitleBar.setShowProgressOnly(true);
-                if (!isTitleBarShowing()) {
-                    setTitleGravity(Gravity.TOP);
-                }
             }
             showTitleBar();
         }