Use WebURLResponse to pass the navigation preload response from SWContextClient to FetchEvent.

Use WebURLResponse to pass the navigation preload response from SWContextClient to FetchEvent.

Currently we use WebServiceWorkerResponse to pass the navigation preload
response from ServiceWorkerContextClient::NavigationPreloadRequest to FetchEvent.

But to show the network timing information in DevTool's network tab, we need
the timing information in ResourceResponseInfo in blnik side. So this cl change
the code to use WebURLResponse which contains the timing information.


BUG=649558
Review-Url: https://codereview.chromium.org/2618313003
Cr-Commit-Position: refs/heads/master@{#442813}
Committed: https://chromium.googlesource.com/chromium/src/+/b9937a428eaf8d98b0ad9e05707ab2d0559a62b3

[horo, 2017-01-10 10:39:19]

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

[commit-bot: I haz the power, 2017-01-10 10:39:33]

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

[horo, 2017-01-10 10:43:04]

Description was changed from

==========
Use WebURLResponse to pass the navigation preload response from SWContextClient
to FetchEvent.

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

to

==========
Use WebURLResponse to pass the navigation preload response from SWContextClient
to FetchEvent.

Currently we use WebServiceWorkerResponse to pass the navigation preload
response from ServiceWorkerContextClient::NavigationPreloadRequest to
FetchEvent.

But to show the network timing information in DevTool's network tab, we need
the timing information in ResourceResponseInfo. So this cl change the code to
use WebURLResponse which contains the timing information.

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

[horo, 2017-01-10 10:43:44]

Description was changed from

==========
Use WebURLResponse to pass the navigation preload response from SWContextClient
to FetchEvent.

Currently we use WebServiceWorkerResponse to pass the navigation preload
response from ServiceWorkerContextClient::NavigationPreloadRequest to
FetchEvent.

But to show the network timing information in DevTool's network tab, we need
the timing information in ResourceResponseInfo. So this cl change the code to
use WebURLResponse which contains the timing information.

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

to

==========
Use WebURLResponse to pass the navigation preload response from SWContextClient
to FetchEvent.

Currently we use WebServiceWorkerResponse to pass the navigation preload
response from ServiceWorkerContextClient::NavigationPreloadRequest to
FetchEvent.

But to show the network timing information in DevTool's network tab, we need
the timing information in ResourceResponseInfo in blnik side. So this cl change
the code to use WebURLResponse which contains the timing information.


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

[horo, 2017-01-10 11:27:56]

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

[horo, 2017-01-10 11:27:57]

falken@
Could you please review this?

[commit-bot: I haz the power, 2017-01-10 12:14:04]

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

[commit-bot: I haz the power, 2017-01-10 12:14:05]

Dry run: This issue passed the CQ dry run.

[falken, 2017-01-11 01:15:18]

lgtm

https://codereview.chromium.org/2618313003/diff/1/content/renderer/service_wo...
File content/renderer/service_worker/service_worker_context_client.cc (right):

https://codereview.chromium.org/2618313003/diff/1/content/renderer/service_wo...
content/renderer/service_worker/service_worker_context_client.cc:264: //
TODO(horo): Set report_security_info true when DevTool is atached.
nit: "to true", "DevTools", "attached"

[horo, 2017-01-11 01:40:54]

Thank you!

https://codereview.chromium.org/2618313003/diff/1/content/renderer/service_wo...
File content/renderer/service_worker/service_worker_context_client.cc (right):

https://codereview.chromium.org/2618313003/diff/1/content/renderer/service_wo...
content/renderer/service_worker/service_worker_context_client.cc:264: //
TODO(horo): Set report_security_info true when DevTool is atached.
On 2017/01/11 01:15:18, falken wrote:
> nit: "to true", "DevTools", "attached"

Done.

[horo, 2017-01-11 01:42:09]

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

[horo, 2017-01-11 01:42:09]

tkent@

Could you please review
third_party/WebKit/Source/web/ServiceWorkerGlobalScopeProxy.*?

[tkent, 2017-01-11 01:44:08]

lgtm

[horo, 2017-01-11 01:53:03]

The CQ bit was checked by horo@chromium.org

[horo, 2017-01-11 01:53:03]

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/2618313003/#ps20001
(title: "incorporated falken's comment")

[commit-bot: I haz the power, 2017-01-11 01:53:26]

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-01-11 04:01:13]

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

[commit-bot: I haz the power, 2017-01-11 04:01:14]

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_...)

[horo, 2017-01-11 04:17:39]

The CQ bit was checked by horo@chromium.org

[commit-bot: I haz the power, 2017-01-11 04:18:06]

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-01-11 05:33:43]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1484108259118380,
"parent_rev": "07f3c9889f504d58669269f4cd0983cc1e2fc1de", "commit_rev":
"b9937a428eaf8d98b0ad9e05707ab2d0559a62b3"}

[commit-bot: I haz the power, 2017-01-11 05:34:11]

Description was changed from

==========
Use WebURLResponse to pass the navigation preload response from SWContextClient
to FetchEvent.

Currently we use WebServiceWorkerResponse to pass the navigation preload
response from ServiceWorkerContextClient::NavigationPreloadRequest to
FetchEvent.

But to show the network timing information in DevTool's network tab, we need
the timing information in ResourceResponseInfo in blnik side. So this cl change
the code to use WebURLResponse which contains the timing information.


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

to

==========
Use WebURLResponse to pass the navigation preload response from SWContextClient
to FetchEvent.

Currently we use WebServiceWorkerResponse to pass the navigation preload
response from ServiceWorkerContextClient::NavigationPreloadRequest to
FetchEvent.

But to show the network timing information in DevTool's network tab, we need
the timing information in ResourceResponseInfo in blnik side. So this cl change
the code to use WebURLResponse which contains the timing information.


BUG=649558

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

[commit-bot: I haz the power, 2017-01-11 05:34:13]

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