DataSaver support for off-main-thread-fetch

DataSaver support for off-main-thread-fetch

This CL gets the DataSaver setting on the main thread and pass it to
WebWorkerFetchContext (WorkerFetchContextImpl, ServiceWorkerFetchContextImpl).
WorkerFetchContext::AddAdditionalRequestHeaders() will check the flag on the
worker thread and set "Save-Data" header if enabled.

BUG=443374, 722774
Review-Url: https://codereview.chromium.org/2886663002
Cr-Commit-Position: refs/heads/master@{#472288}
Committed: https://chromium.googlesource.com/chromium/src/+/137e05d0105abedb195dae64225b1f7b954da5e5

[horo, 2017-05-16 04:10:57]

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

[commit-bot: I haz the power, 2017-05-16 04:11:16]

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, 2017-05-16 05:47:58]

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

[commit-bot: I haz the power, 2017-05-16 05:47:59]

Dry run: This issue passed the CQ dry run.

[horo, 2017-05-16 14:03:04]

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

[commit-bot: I haz the power, 2017-05-16 14:03:36]

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, 2017-05-16 14:07:25]

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

[commit-bot: I haz the power, 2017-05-16 14:07:26]

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

[horo, 2017-05-16 14:07:28]

Description was changed from

==========
DataSaver for worker

BUG=
==========

to

==========
DataSaver support for off-main-thread-fetch

This CL gets the DataSaver setting on the main thread and pass it to
WebWorkerFetchContext (WorkerFetchContextImpl, ServiceWorkerFetchContextImpl).
WorkerFetchContext::AddAdditionalRequestHeaders() will check the flag on the
worker thread and set "Save-Data" header if enabled.

BUG=443374,722774
==========

[horo, 2017-05-16 14:11:24]

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

[commit-bot: I haz the power, 2017-05-16 14:12:02]

Dry run: CQ is trying da patch.

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

[horo, 2017-05-16 14:13:53]

Patchset #4 (id:60001) has been deleted

[commit-bot: I haz the power, 2017-05-16 14:14:35]

Dry run: Try jobs failed on following builders:
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[commit-bot: I haz the power, 2017-05-16 14:20:12]

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

[commit-bot: I haz the power, 2017-05-16 14:20:13]

Dry run: Try jobs failed on following builders:
  android_arm64_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_arm6...)
  android_compile_dbg on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_comp...)
  android_cronet on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_cron...)
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)
  cast_shell_android on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/cast_shell_a...)
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)
  chromeos_daisy_chromium_compile_only_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[horo, 2017-05-16 14:41:27]

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

[commit-bot: I haz the power, 2017-05-16 14:42:03]

Dry run: CQ is trying da patch.

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

[horo, 2017-05-16 14:43:52]

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

[horo, 2017-05-16 14:43:52]

kinuko@
Could you please review this?

[commit-bot: I haz the power, 2017-05-16 16:10:33]

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

[commit-bot: I haz the power, 2017-05-16 16:10:34]

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

[kinuko, 2017-05-17 02:12:45]

lgtm

[horo, 2017-05-17 02:17:14]

The CQ bit was checked by horo@chromium.org

[commit-bot: I haz the power, 2017-05-17 02:19:04]

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, 2017-05-17 02:28:09]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1494987434487810,
"parent_rev": "09f616277fc148aaac61828f5e76e392c33327d1", "commit_rev":
"137e05d0105abedb195dae64225b1f7b954da5e5"}

[commit-bot: I haz the power, 2017-05-17 02:28:23]

Description was changed from

==========
DataSaver support for off-main-thread-fetch

This CL gets the DataSaver setting on the main thread and pass it to
WebWorkerFetchContext (WorkerFetchContextImpl, ServiceWorkerFetchContextImpl).
WorkerFetchContext::AddAdditionalRequestHeaders() will check the flag on the
worker thread and set "Save-Data" header if enabled.

BUG=443374,722774
==========

to

==========
DataSaver support for off-main-thread-fetch

This CL gets the DataSaver setting on the main thread and pass it to
WebWorkerFetchContext (WorkerFetchContextImpl, ServiceWorkerFetchContextImpl).
WorkerFetchContext::AddAdditionalRequestHeaders() will check the flag on the
worker thread and set "Save-Data" header if enabled.

BUG=443374,722774

Review-Url: https://codereview.chromium.org/2886663002
Cr-Commit-Position: refs/heads/master@{#472288}
Committed:
https://chromium.googlesource.com/chromium/src/+/137e05d0105abedb195dae64225b...
==========

[commit-bot: I haz the power, 2017-05-17 02:28:25]

Committed patchset #4 (id:80001) as
https://chromium.googlesource.com/chromium/src/+/137e05d0105abedb195dae64225b...