Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(7)

Unified Diff: chrome/android/java_staging/src/org/chromium/chrome/browser/toolbar/ToolbarPhone.java

Issue 1149213003: Don't update color in tabSwitcher Mode (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/android/java_staging/src/org/chromium/chrome/browser/toolbar/ToolbarPhone.java
diff --git a/chrome/android/java_staging/src/org/chromium/chrome/browser/toolbar/ToolbarPhone.java b/chrome/android/java_staging/src/org/chromium/chrome/browser/toolbar/ToolbarPhone.java
index 8e444b4c1052a05414d6b1f0a45629f785c87f4e..196efc97f486ea852efd1fc86af454f058fade5c 100644
--- a/chrome/android/java_staging/src/org/chromium/chrome/browser/toolbar/ToolbarPhone.java
+++ b/chrome/android/java_staging/src/org/chromium/chrome/browser/toolbar/ToolbarPhone.java
@@ -1613,12 +1613,12 @@ public class ToolbarPhone extends ToolbarLayout
}
@Override
- protected void onPrimaryColorChanged() {
- super.onPrimaryColorChanged();
+ protected void checkPrimaryColorTransition() {
+ super.checkPrimaryColorTransition();
if (mBrandColorTransitionActive) mBrandColorTransitionAnimation.cancel();
final int initialColor = mToolbarBackground.getColor();
final int finalColor = getToolbarDataProvider().getPrimaryColor();
- if (initialColor == finalColor) return;
+ if (mInTabSwitcherMode || initialColor == finalColor) return;
boolean shouldUseOpaque = BrandColorUtils.shouldUseOpaqueTextboxBackground(finalColor);
final int initialAlpha = mUrlBackgroundAlpha;
final int finalAlpha =

Powered by Google App Engine
This is Rietveld 408576698