service worker: Add NavigationPreload runtime flag and skeleton interface

service worker: Add NavigationPreload runtime flag and skeleton interface

This just adds the runtime flag and a dummy implementation. Actual code and
tests are to come in future patches.

This patch is based on the WIP spec change here:
https://github.com/w3c/ServiceWorker/pull/983/files

BUG=649558
Committed: https://crrev.com/9a2b454a609338388dec903980267f3c7141a97f
Cr-Commit-Position: refs/heads/master@{#424382}

[falken, 2016-10-03 07:57:31]

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

[falken, 2016-10-03 07:57:32]

PTAL. I started writing https://codereview.chromium.org/2388023002 but decided
to split up the patch even more.

[falken, 2016-10-03 07:57:53]

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

[commit-bot: I haz the power, 2016-10-03 07:58:11]

Dry run: 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-03 10:00:04]

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

[commit-bot: I haz the power, 2016-10-03 10:00:06]

Dry run: Try jobs failed on following builders:
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)

[horo, 2016-10-03 14:05:16]

https://codereview.chromium.org/2389693002/diff/1/third_party/WebKit/Source/m...
File
third_party/WebKit/Source/modules/serviceworkers/NavigationPreloadOptions.idl
(right):

https://codereview.chromium.org/2389693002/diff/1/third_party/WebKit/Source/m...
third_party/WebKit/Source/modules/serviceworkers/NavigationPreloadOptions.idl:6:
ByteString value;
Add a link to the spec discussion
(https://github.com/w3c/ServiceWorker/issues/920) and comments about that this
interface is under discussion and not defined in the spec yet.

https://codereview.chromium.org/2389693002/diff/1/third_party/WebKit/Source/m...
File third_party/WebKit/Source/modules/serviceworkers/ServiceWorker.idl (right):

https://codereview.chromium.org/2389693002/diff/1/third_party/WebKit/Source/m...
third_party/WebKit/Source/modules/serviceworkers/ServiceWorker.idl:53:
[CallWith=ScriptState,RuntimeEnabled=ServiceWorkerNavigationPreload]
Promise<Dictionary> getNavigationPreload();
ditto

[falken, 2016-10-07 07:47:26]

Description was changed from

==========
service worker: Add NavigationPreload runtime flag and skeleton interface

This just adds the runtime flag and a dummy implementation. Actual code and
tests are to come in future patches.

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

to

==========
service worker: Add NavigationPreload runtime flag and skeleton interface

This just adds the runtime flag and a dummy implementation. Actual code and
tests are to come in future patches.

This patch is based on the WIP spec change here:
https://github.com/w3c/ServiceWorker/pull/983/files

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

[falken, 2016-10-07 07:47:44]

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

[falken, 2016-10-07 07:47:47]

Updated to match new spec. PTAL.

[commit-bot: I haz the power, 2016-10-07 07:47:54]

Dry run: 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-07 07:51:25]

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

[commit-bot: I haz the power, 2016-10-07 07:51:26]

Dry run: Try jobs failed on following builders:
  android_compile_dbg on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_comp...)
  cast_shell_linux on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)

[falken, 2016-10-07 07:54:24]

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

[commit-bot: I haz the power, 2016-10-07 07:54:42]

Dry run: 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-07 08:25:23]

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

[commit-bot: I haz the power, 2016-10-07 08:25:24]

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

[horo, 2016-10-11 02:41:03]

lgtm

[falken, 2016-10-11 03:31:24]

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

[falken, 2016-10-11 03:31:25]

haraken please review:
- third_party/WebKit/Source/modules/modules_idl_files.gni 
- third_party/WebKit/Source/platform/RuntimeEnabledFeatures.in

[haraken, 2016-10-11 03:38:20]

On 2016/10/11 03:31:25, falken wrote:
> haraken please review:
> - third_party/WebKit/Source/modules/modules_idl_files.gni 
> - third_party/WebKit/Source/platform/RuntimeEnabledFeatures.in

LGTM

[falken, 2016-10-11 03:39:36]

The CQ bit was checked by falken@chromium.org

[commit-bot: I haz the power, 2016-10-11 03:39:51]

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-11 07:03:59]

Description was changed from

==========
service worker: Add NavigationPreload runtime flag and skeleton interface

This just adds the runtime flag and a dummy implementation. Actual code and
tests are to come in future patches.

This patch is based on the WIP spec change here:
https://github.com/w3c/ServiceWorker/pull/983/files

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

to

==========
service worker: Add NavigationPreload runtime flag and skeleton interface

This just adds the runtime flag and a dummy implementation. Actual code and
tests are to come in future patches.

This patch is based on the WIP spec change here:
https://github.com/w3c/ServiceWorker/pull/983/files

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

[commit-bot: I haz the power, 2016-10-11 07:04:00]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-10-11 07:06:13]

Description was changed from

==========
service worker: Add NavigationPreload runtime flag and skeleton interface

This just adds the runtime flag and a dummy implementation. Actual code and
tests are to come in future patches.

This patch is based on the WIP spec change here:
https://github.com/w3c/ServiceWorker/pull/983/files

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

to

==========
service worker: Add NavigationPreload runtime flag and skeleton interface

This just adds the runtime flag and a dummy implementation. Actual code and
tests are to come in future patches.

This patch is based on the WIP spec change here:
https://github.com/w3c/ServiceWorker/pull/983/files

BUG=649558

Committed: https://crrev.com/9a2b454a609338388dec903980267f3c7141a97f
Cr-Commit-Position: refs/heads/master@{#424382}
==========

[commit-bot: I haz the power, 2016-10-11 07:06:15]

Patchset 4 (id:??) landed as
https://crrev.com/9a2b454a609338388dec903980267f3c7141a97f
Cr-Commit-Position: refs/heads/master@{#424382}