Fix bug 5116434 - Bookmark label is not highlighted.

Let the action bar tab strip handle its own height changes.

Change-Id: I9b79702c757c736ef7872a43316965cea1a5cc97

2 files changed, 16 insertions, 9 deletions

diff --git a/core/java/com/android/internal/app/ActionBarImpl.java b/core/java/com/android/internal/app/ActionBarImpl.java
index bc8715379a3b..008f400b3e55 100644
--- a/core/java/com/android/internal/app/ActionBarImpl.java
+++ b/core/java/com/android/internal/app/ActionBarImpl.java
@@ -95,7 +95,6 @@ public class ActionBarImpl extends ActionBar {
 
     private int mContextDisplayMode;
     private boolean mHasEmbeddedTabs;
-    private int mContentHeight;
 
     final Handler mHandler = new Handler();
     Runnable mTabSelector;
@@ -163,8 +162,6 @@ public class ActionBarImpl extends ActionBar {
         mContextDisplayMode = mActionView.isSplitActionBar() ?
                 CONTEXT_DISPLAY_SPLIT : CONTEXT_DISPLAY_NORMAL;
 
-        mContentHeight = mActionView.getContentHeight();
-
         // Older apps get the home button interaction enabled by default.
         // Newer apps need to enable it explicitly.
         setHomeButtonEnabled(mContext.getApplicationInfo().targetSdkVersion <
@@ -188,12 +185,6 @@ public class ActionBarImpl extends ActionBar {
         }
         mActionView.setCollapsable(!mHasEmbeddedTabs &&
                 getNavigationMode() == NAVIGATION_MODE_TABS);
-
-        mContentHeight = mActionView.getContentHeight();
-
-        if (mTabScrollView != null) {
-            mTabScrollView.setContentHeight(mContentHeight);
-        }
     }
 
     private void ensureTabsExist() {
diff --git a/core/java/com/android/internal/widget/ScrollingTabContainerView.java b/core/java/com/android/internal/widget/ScrollingTabContainerView.java
index fefa2232d0a2..718d249dec1b 100644
--- a/core/java/com/android/internal/widget/ScrollingTabContainerView.java
+++ b/core/java/com/android/internal/widget/ScrollingTabContainerView.java
@@ -15,11 +15,15 @@
  */
 package com.android.internal.widget;
 
+import com.android.internal.R;
+
 import android.animation.Animator;
 import android.animation.ObjectAnimator;
 import android.animation.TimeInterpolator;
 import android.app.ActionBar;
 import android.content.Context;
+import android.content.res.Configuration;
+import android.content.res.TypedArray;
 import android.graphics.drawable.Drawable;
 import android.text.TextUtils.TruncateAt;
 import android.view.Gravity;
@@ -92,6 +96,18 @@ public class ScrollingTabContainerView extends HorizontalScrollView {
         requestLayout();
     }
 
+    @Override
+    protected void onConfigurationChanged(Configuration newConfig) {
+        super.onConfigurationChanged(newConfig);
+
+        // Action bar can change size on configuration changes.
+        // Reread the desired height from the theme-specified style.
+        TypedArray a = getContext().obtainStyledAttributes(null, R.styleable.ActionBar,
+                com.android.internal.R.attr.actionBarStyle, 0);
+        setContentHeight(a.getLayoutDimension(R.styleable.ActionBar_height, 0));
+        a.recycle();
+    }
+
     public void animateToVisibility(int visibility) {
         if (mVisibilityAnim != null) {
             mVisibilityAnim.cancel();