Introduce ServiceWorkerNavigationPreload feature name

Introduce ServiceWorkerNavigationPreload feature name

https://crrev.com/2389693002 added NavigationPreload runtime flag as
experimental status. This feature is enabled when
"--enable-experimental-web-platform-features" flag is set. But according to the
comment https://crrev.com/2410333006/#msg42 we should introduce a new flag
instead. So this CL introduces ServiceWorkerNavigationPreload feature which is
enabled when "--enable-features=ServiceWorkerNavigationPreload" flag is set.

BUG=649558
Committed: https://crrev.com/752388328b9fa105ce1eb822e0a58e33f1a892ef
Cr-Commit-Position: refs/heads/master@{#426395}

[horo, 2016-10-19 04:11:47]

Description was changed from

==========
Introduce ServiceWorkerNavigationPreload feature name

https://codereview.chromium.org/2389693002 added NavigationPreload runtime flag
as experimental status.
This feature is enabled when "--enable-experimental-web-platform-features" flag
is set.
But according to the comment https://codereview.chromium.org/2410333006/#msg42
we should introduce a new flag instead.
So this CL introduces ServiceWorkerNavigationPreload feature which is enabled
when "--enable-features=ServiceWorkerNavigationPreload" flag is set.

BUG=649558
==========

to

==========
Introduce ServiceWorkerNavigationPreload feature name

https://crrev.com/2389693002 added NavigationPreload runtime flag as
experimental status. This feature is enabled when
"--enable-experimental-web-platform-features" flag is set. But according to the
comment https://crrev.com/2410333006/#msg42 we should introduce a new flag
instead. So this CL introduces ServiceWorkerNavigationPreload feature which is
enabled when "--enable-features=ServiceWorkerNavigationPreload" flag is set.

BUG=649558
==========

[horo, 2016-10-19 04:13:57]

horo@chromium.org changed reviewers:
+ falken@chromium.org

[horo, 2016-10-19 04:13:58]

falken@
Could you please review this?

[falken, 2016-10-19 04:43:09]

lgtm

https://codereview.chromium.org/2431003003/diff/1/content/public/common/conte...
File content/public/common/content_features.h (right):

https://codereview.chromium.org/2431003003/diff/1/content/public/common/conte...
content/public/common/content_features.h:48: CONTENT_EXPORT extern const
base::Feature kServiceWorkerNavigationPreload;
nit: alphabetical order

[horo, 2016-10-19 04:55:47]

horo@chromium.org changed reviewers:
+ tkent@chromium.org

[horo, 2016-10-19 04:55:48]

tkent@
Please review RuntimeEnabledFeatures.in, WebRuntimeFeatures.cpp and
WebRuntimeFeatures.h.

[horo, 2016-10-19 04:56:33]

The CQ bit was checked by horo@chromium.org to run a CQ dry run

[horo, 2016-10-19 04:57:34]

horo@chromium.org changed reviewers:
+ nick@chromium.org

[horo, 2016-10-19 04:57:35]

nick@
Could you please review runtime_features.*?

[commit-bot: I haz the power, 2016-10-19 04:57:58]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[tkent, 2016-10-19 05:29:27]

On 2016/10/19 at 04:55:48, horo wrote:
> tkent@
> Please review RuntimeEnabledFeatures.in, WebRuntimeFeatures.cpp and
WebRuntimeFeatures.h.

lgtm

[commit-bot: I haz the power, 2016-10-19 10:33:59]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-19 10:34:00]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[ncarter (slow), 2016-10-19 17:37:19]

content lgtm

[horo, 2016-10-20 00:16:59]

The CQ bit was checked by horo@chromium.org

[horo, 2016-10-20 00:16:59]

The patchset sent to the CQ was uploaded after l-g-t-m from falken@chromium.org
Link to the patchset: https://codereview.chromium.org/2431003003/#ps20001
(title: "alphabetical order")

[commit-bot: I haz the power, 2016-10-20 00:17:42]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-20 03:56:51]

Description was changed from

==========
Introduce ServiceWorkerNavigationPreload feature name

https://crrev.com/2389693002 added NavigationPreload runtime flag as
experimental status. This feature is enabled when
"--enable-experimental-web-platform-features" flag is set. But according to the
comment https://crrev.com/2410333006/#msg42 we should introduce a new flag
instead. So this CL introduces ServiceWorkerNavigationPreload feature which is
enabled when "--enable-features=ServiceWorkerNavigationPreload" flag is set.

BUG=649558
==========

to

==========
Introduce ServiceWorkerNavigationPreload feature name

https://crrev.com/2389693002 added NavigationPreload runtime flag as
experimental status. This feature is enabled when
"--enable-experimental-web-platform-features" flag is set. But according to the
comment https://crrev.com/2410333006/#msg42 we should introduce a new flag
instead. So this CL introduces ServiceWorkerNavigationPreload feature which is
enabled when "--enable-features=ServiceWorkerNavigationPreload" flag is set.

BUG=649558
==========

[commit-bot: I haz the power, 2016-10-20 03:56:52]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-21 13:15:14]

Description was changed from

==========
Introduce ServiceWorkerNavigationPreload feature name

https://crrev.com/2389693002 added NavigationPreload runtime flag as
experimental status. This feature is enabled when
"--enable-experimental-web-platform-features" flag is set. But according to the
comment https://crrev.com/2410333006/#msg42 we should introduce a new flag
instead. So this CL introduces ServiceWorkerNavigationPreload feature which is
enabled when "--enable-features=ServiceWorkerNavigationPreload" flag is set.

BUG=649558
==========

to

==========
Introduce ServiceWorkerNavigationPreload feature name

https://crrev.com/2389693002 added NavigationPreload runtime flag as
experimental status. This feature is enabled when
"--enable-experimental-web-platform-features" flag is set. But according to the
comment https://crrev.com/2410333006/#msg42 we should introduce a new flag
instead. So this CL introduces ServiceWorkerNavigationPreload feature which is
enabled when "--enable-features=ServiceWorkerNavigationPreload" flag is set.

BUG=649558

Committed: https://crrev.com/752388328b9fa105ce1eb822e0a58e33f1a892ef
Cr-Commit-Position: refs/heads/master@{#426395}
==========

[commit-bot: I haz the power, 2016-10-21 13:15:16]

Patchset 2 (id:??) landed as
https://crrev.com/752388328b9fa105ce1eb822e0a58e33f1a892ef
Cr-Commit-Position: refs/heads/master@{#426395}