Index: chrome/android/java/src/org/chromium/chrome/browser/document/DocumentActivity.java |
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/document/DocumentActivity.java b/chrome/android/java/src/org/chromium/chrome/browser/document/DocumentActivity.java |
index 1ede45b25d91c86f03fb4ceb8fdd613deaf53d63..b22e66a8afb42569560719b9fe2332c1f37674e8 100644 |
--- a/chrome/android/java/src/org/chromium/chrome/browser/document/DocumentActivity.java |
+++ b/chrome/android/java/src/org/chromium/chrome/browser/document/DocumentActivity.java |
@@ -58,7 +58,6 @@ import org.chromium.chrome.browser.tabmodel.document.TabDelegate; |
import org.chromium.chrome.browser.toolbar.ToolbarControlContainer; |
import org.chromium.chrome.browser.util.FeatureUtilities; |
import org.chromium.chrome.browser.util.IntentUtils; |
-import org.chromium.chrome.browser.widget.ControlContainer; |
import org.chromium.chrome.browser.widget.RoundedIconGenerator; |
import org.chromium.chrome.browser.widget.findinpage.FindToolbarManager; |
import org.chromium.components.service_tab_launcher.ServiceTabLauncher; |
@@ -104,7 +103,6 @@ public class DocumentActivity extends ChromeActivity { |
// Indicates whether mIcon was generated by RoundedIconGenerator. |
private boolean mIsUsingGeneratedIcon; |
- private Integer mThemeColor; |
private int mDefaultThemeColor; |
private DocumentTab mDocumentTab; |
@@ -534,9 +532,7 @@ public class DocumentActivity extends ChromeActivity { |
@Override |
public void onWebContentsSwapped(Tab tab, boolean didStartLoad, boolean didFinishLoad) { |
if (!didStartLoad) return; |
- mThemeColor = tab.getWebContents().getThemeColor(mDefaultThemeColor); |
- mIcon = null; |
- updateTaskDescription(); |
+ onFaviconReceived(tab.getFavicon()); |
} |
@Override |
@@ -572,7 +568,7 @@ public class DocumentActivity extends ChromeActivity { |
@Override |
public void onSSLStateUpdated(Tab tab) { |
- if (hasSecurityWarningOrError(tab)) resetThemeColorAndIcon(); |
+ if (hasSecurityWarningOrError(tab)) resetIcon(); |
} |
@Override |
@@ -592,30 +588,6 @@ public class DocumentActivity extends ChromeActivity { |
} |
@Override |
- public void onDidChangeThemeColor(int color) { |
- if (hasSecurityWarningOrError(mDocumentTab)) return; |
- if (color == Color.TRANSPARENT) color = mDefaultThemeColor; |
- |
- // Ignore any transparency value. |
- color |= 0xFF000000; |
- |
- mThemeColor = Integer.valueOf(color); |
- updateTaskDescription(); |
- } |
- |
- @Override |
- public void onDidAttachInterstitialPage(Tab tab) { |
- resetThemeColorAndIcon(); |
- } |
- |
- @Override |
- public void onDidDetachInterstitialPage(Tab tab) { |
- mThemeColor = tab.getWebContents().getThemeColor(mDefaultThemeColor); |
- mIcon = null; |
- updateTaskDescription(); |
- } |
- |
- @Override |
public void onCrash(Tab tab, boolean sadTabShown) { |
int currentState = ApplicationStatus.getStateForActivity(DocumentActivity.this); |
if (currentState != ActivityState.STOPPED) return; |
@@ -655,8 +627,7 @@ public class DocumentActivity extends ChromeActivity { |
} |
} |
- private void resetThemeColorAndIcon() { |
- mThemeColor = null; |
+ private void resetIcon() { |
mIcon = null; |
updateTaskDescription(); |
} |
@@ -828,27 +799,19 @@ public class DocumentActivity extends ChromeActivity { |
updateTaskDescription(label, mIcon); |
} |
- protected int getThemeColor() { |
- if (isIncognito()) { |
- return mDefaultThemeColor; |
- } else { |
- return mThemeColor != null ? mThemeColor.intValue() : mDefaultThemeColor; |
- } |
- } |
- |
private boolean shouldUseDefaultStatusBarColor() { |
- return isIncognito() || mThemeColor == null || mThemeColor == mDefaultThemeColor; |
+ return isIncognito() || mThemeColor == mDefaultThemeColor; |
} |
protected void updateTaskDescription(String label, Bitmap icon) { |
- int color = getThemeColor(); |
- DocumentUtils.updateTaskDescription(this, label, icon, color, |
+ DocumentUtils.updateTaskDescription(this, label, icon, mThemeColor, |
shouldUseDefaultStatusBarColor()); |
- getToolbarManager().updatePrimaryColor(color); |
+ } |
- ControlContainer controlContainer = |
- (ControlContainer) findViewById(R.id.control_container); |
- controlContainer.getToolbarResourceAdapter().invalidate(null); |
+ @Override |
+ protected void onThemeColorUpdate() { |
+ super.onThemeColorUpdate(); |
+ updateTaskDescription(); |
} |
/** |