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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/util/FeatureUtilities.java

Issue 2946493002: [Chrome Home] Avoid restart if experiment enabled after FRE. (Closed)
Patch Set: Created 3 years, 6 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/android/java/src/org/chromium/chrome/browser/util/FeatureUtilities.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/util/FeatureUtilities.java b/chrome/android/java/src/org/chromium/chrome/browser/util/FeatureUtilities.java
index 3067695a617cdd4329a18327e7734122f8d8a45e..3d201183812ef84c63145a03cedb2123a600af4f 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/util/FeatureUtilities.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/util/FeatureUtilities.java
@@ -267,6 +267,19 @@ public class FeatureUtilities {
} finally {
StrictMode.setThreadPolicy(oldPolicy);
}
+
+ // If the browser has been initialized by this point, check the experiment as well to
+ // avoid the restart logic in cacheChromeHomeEnabled.
+ if (ChromeFeatureList.isInitialized()) {
+ boolean chromeHomeExperimentEnabled =
+ ChromeFeatureList.isEnabled(ChromeFeatureList.CHROME_HOME);
+
+ if (chromeHomeExperimentEnabled != sChromeHomeEnabled) {
+ sChromeHomeEnabled = chromeHomeExperimentEnabled;
+ ChromePreferenceManager.getInstance().setChromeHomeEnabled(
+ chromeHomeExperimentEnabled);
+ }
+ }
}
return sChromeHomeEnabled;
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698