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

Unified Diff: content/public/common/content_features.cc

Issue 2637763002: Enable ServiceWorkerNavigationPreload by default for Origin-Trial. (Closed)
Patch Set: incorporated kinuko's comment Created 3 years, 11 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: content/public/common/content_features.cc
diff --git a/content/public/common/content_features.cc b/content/public/common/content_features.cc
index 338a7745240a1b4a6e04a8fd0e4b57109a280e8b..0fd44281b151ebeb12cc22fc5dcc1a23711aa362 100644
--- a/content/public/common/content_features.cc
+++ b/content/public/common/content_features.cc
@@ -189,10 +189,12 @@ const base::Feature kScrollAnchoring{"ScrollAnchoring",
base::FEATURE_ENABLED_BY_DEFAULT};
// Navigation preload feature of service workers.
-// TODO(horo): Revise link when this lands in the spec:
-// https://github.com/w3c/ServiceWorker/pull/983/files
+// Enables this base::Feature by default for Origin-Trial, but enables the
+// corresponding blink::WebRuntimeFeatures only when '--enable-features' command
+// line flag is given. See the comments in runtime_features.cc and
+// service_worker_version.h for the details.
const base::Feature kServiceWorkerNavigationPreload{
- "ServiceWorkerNavigationPreload", base::FEATURE_DISABLED_BY_DEFAULT};
+ "ServiceWorkerNavigationPreload", base::FEATURE_ENABLED_BY_DEFAULT};
// http://tc39.github.io/ecmascript_sharedmem/shmem.html
const base::Feature kSharedArrayBuffer{"SharedArrayBuffer",

Powered by Google App Engine
This is Rietveld 408576698