Implement WorkerFetchContext::AddResourceTiming() for off-main-thread-fetch

Implement WorkerFetchContext::AddResourceTiming() for off-main-thread-fetch

After this CL, performance.getEntries() will return the ResourceTiming
information of the requests which are fetched from the worker even if
off-main-thread-fetch is enabled (--enable-features=OffMainThreadFetch).

BUG=443374
Review-Url: https://codereview.chromium.org/2893533002
Cr-Commit-Position: refs/heads/master@{#472652}
Committed: https://chromium.googlesource.com/chromium/src/+/e6bdad2acc6f745d504782ca9e3655d2570393e7

[horo, 2017-05-17 05:50:07]

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

[commit-bot: I haz the power, 2017-05-17 05:51:13]

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-17 05:52:40]

Description was changed from

==========
Implement WorkerFetchContext::AddResourceTiming() for off-main-thread-fetch

BUG=443374
==========

to

==========
Implement WorkerFetchContext::AddResourceTiming() for off-main-thread-fetch

After this CL, performance.getEntries() will returns the ResourceTiming
information of the requests which are fetched from the worker even if
off-main-thread-fetch is enabled (--enable-features=OffMainThreadFetch).

BUG=443374
==========

[horo, 2017-05-17 05:53:00]

Description was changed from

==========
Implement WorkerFetchContext::AddResourceTiming() for off-main-thread-fetch

After this CL, performance.getEntries() will returns the ResourceTiming
information of the requests which are fetched from the worker even if
off-main-thread-fetch is enabled (--enable-features=OffMainThreadFetch).

BUG=443374
==========

to

==========
Implement WorkerFetchContext::AddResourceTiming() for off-main-thread-fetch

After this CL, performance.getEntries() will return the ResourceTiming
information of the requests which are fetched from the worker even if
off-main-thread-fetch is enabled (--enable-features=OffMainThreadFetch).

BUG=443374
==========

[horo, 2017-05-17 05:54:51]

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

[commit-bot: I haz the power, 2017-05-17 05:55:01]

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-17 05:55:48]

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

[horo, 2017-05-17 05:55:48]

falken@

[horo, 2017-05-17 05:55:56]

falken@
Please review this.

[commit-bot: I haz the power, 2017-05-17 08:40:07]

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

[commit-bot: I haz the power, 2017-05-17 08:40:08]

Dry run: Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[falken, 2017-05-18 01:17:25]

lgtm

https://codereview.chromium.org/2893533002/diff/20001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/virtual/off-main-thread-fetch/external/wpt/service-workers/service-worker/performance-timeline.https-expected.txt
(right):

https://codereview.chromium.org/2893533002/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/virtual/off-main-thread-fetch/external/wpt/service-workers/service-worker/performance-timeline.https-expected.txt:6:
Harness: the test ran to completion.
Can the expected.txt be removed, since all tests PASS?

[horo, 2017-05-18 01:49:38]

https://codereview.chromium.org/2893533002/diff/20001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/virtual/off-main-thread-fetch/external/wpt/service-workers/service-worker/performance-timeline.https-expected.txt
(right):

https://codereview.chromium.org/2893533002/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/virtual/off-main-thread-fetch/external/wpt/service-workers/service-worker/performance-timeline.https-expected.txt:6:
Harness: the test ran to completion.
On 2017/05/18 01:17:24, falken wrote:
> Can the expected.txt be removed, since all tests PASS?

The current implementation (without OffMainThreadFetch) doesn't pass the test,
because of thread hop ordering issue.
https://cs.chromium.org/chromium/src/third_party/WebKit/LayoutTests/virtual/o...

So we need this all passing -expected.txt file.

[horo, 2017-05-18 01:50:24]

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

[horo, 2017-05-18 01:50:24]

yhirano@
Could you please review 
third_party/WebKit/Source/core/loader/WorkerFetchContext.*?

[yhirano, 2017-05-18 01:52:44]

lgtm

[horo, 2017-05-18 01:53:35]

The CQ bit was checked by horo@chromium.org

[commit-bot: I haz the power, 2017-05-18 01:55:30]

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-18 03:52:33]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1495072415729810,
"parent_rev": "1cb9864601dad078281ff870a23b308d2e88cb50", "commit_rev":
"e6bdad2acc6f745d504782ca9e3655d2570393e7"}

[commit-bot: I haz the power, 2017-05-18 03:52:46]

Description was changed from

==========
Implement WorkerFetchContext::AddResourceTiming() for off-main-thread-fetch

After this CL, performance.getEntries() will return the ResourceTiming
information of the requests which are fetched from the worker even if
off-main-thread-fetch is enabled (--enable-features=OffMainThreadFetch).

BUG=443374
==========

to

==========
Implement WorkerFetchContext::AddResourceTiming() for off-main-thread-fetch

After this CL, performance.getEntries() will return the ResourceTiming
information of the requests which are fetched from the worker even if
off-main-thread-fetch is enabled (--enable-features=OffMainThreadFetch).

BUG=443374

Review-Url: https://codereview.chromium.org/2893533002
Cr-Commit-Position: refs/heads/master@{#472652}
Committed:
https://chromium.googlesource.com/chromium/src/+/e6bdad2acc6f745d504782ca9e36...
==========

[commit-bot: I haz the power, 2017-05-18 03:52:47]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/e6bdad2acc6f745d504782ca9e36...