Revert of Show service worker navigation preload requests in DevTools Network tab

Revert of Show service worker navigation preload requests in DevTools Network tab (patchset #10 id:480001 of https://codereview.chromium.org/2620463002/ )

Reason for revert:
Introduced DCHECK crash.

BUG=685493
Original issue's description:
> Show service worker navigation preload requests in DevTools Network tab.
> 
> Demo: https://youtu.be/I-Qe_Y-xYxE
> 
> Navigation Preload requests are initiated from the browser process.
> This is different from the normal network requests which are initiated from the
> renderer process.
> 
> When the DevTools show the normal requests in the Network tab, DevTool's
> Network events (requestWillBeSent, responseReceived, loadingFinished etc) are
> dispatched via InspectorInstrumentation and InspectorNetworkAgent.
> 
> This CL introduces new DevTool's Network events (navigationPreloadSent,
> navigationPreloadResponseReceived, navigationPreloadFailed,
> navigationPreloadFinished) which are dispatched via InspectorInstrumentation and
> InspectorNetworkAgent from ServiceWorkerContextClient.
> 
> In the normal requests case, we record the request sending timestamp when the
> renderer process will send the request in InspectorNetworkAgent::
> willSendRequestInternal(). But in the navigation preload case, we record the
> timestamp in the browser process, and send it to the service worker's renderer
> process using FetchEventPreloadHandle.
> 
> 
> BUG=649558
> 
> Review-Url: https://codereview.chromium.org/2620463002
> Cr-Original-Commit-Position: refs/heads/master@{#445630}
> Committed: https://chromium.googlesource.com/chromium/src/+/4c6b3b0f11b62e069d9d84ba99dbe94a45a5c622
> Review-Url: https://codereview.chromium.org/2620463002
> Cr-Commit-Position: refs/heads/master@{#445699}
> Committed: https://chromium.googlesource.com/chromium/src/+/23276c71698583a0142e8213bada22e3ae8d0a0e

TBR=falken@chromium.org,pfeldman@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=649558
Review-Url: https://codereview.chromium.org/2654433012
Cr-Commit-Position: refs/heads/master@{#446270}
Committed: https://chromium.googlesource.com/chromium/src/+/16de2c25e724f07d46b68f5758e49fae6f136073

[horo, 2017-01-26 03:45:58]

The CQ bit was checked by horo@chromium.org

[horo, 2017-01-26 03:45:58]

Created Revert of Show service worker navigation preload requests in DevTools
Network tab

[commit-bot: I haz the power, 2017-01-26 03:46:45]

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-26 03:50:00]

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

[commit-bot: I haz the power, 2017-01-26 03:50:01]

Try jobs failed on following builders:
  ios-device-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-xcode-...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[horo, 2017-01-26 04:41:47]

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

[commit-bot: I haz the power, 2017-01-26 04:42:05]

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-26 04:43:58]

Patchset #2 (id:290001) has been deleted

[commit-bot: I haz the power, 2017-01-26 04:44:39]

Dry run: Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-device-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-xcode-...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)
  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-01-26 04:47:30]

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

[commit-bot: I haz the power, 2017-01-26 04:47:31]

Dry run: Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-device-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-xcode-...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)

[horo, 2017-01-26 04:59:06]

I understood the crash reason.
I want to land the fix patch instead of reverting.
https://codereview.chromium.org/2650973006/

[horo, 2017-01-26 04:59:14]

Description was changed from

==========
Revert of Show service worker navigation preload requests in DevTools Network
tab (patchset #10 id:480001 of https://codereview.chromium.org/2620463002/ )

Reason for revert:
Introduced DCHECK crash.

BUG=685493


Original issue's description:
> Show service worker navigation preload requests in DevTools Network tab.
> 
> Demo: https://youtu.be/I-Qe_Y-xYxE
> 
> Navigation Preload requests are initiated from the browser process.
> This is different from the normal network requests which are initiated from
the
> renderer process.
> 
> When the DevTools show the normal requests in the Network tab, DevTool's
> Network events (requestWillBeSent, responseReceived, loadingFinished etc) are
> dispatched via InspectorInstrumentation and InspectorNetworkAgent.
> 
> This CL introduces new DevTool's Network events (navigationPreloadSent,
> navigationPreloadResponseReceived, navigationPreloadFailed,
> navigationPreloadFinished) which are dispatched via InspectorInstrumentation
and
> InspectorNetworkAgent from ServiceWorkerContextClient.
> 
> In the normal requests case, we record the request sending timestamp when the
> renderer process will send the request in InspectorNetworkAgent::
> willSendRequestInternal(). But in the navigation preload case, we record the
> timestamp in the browser process, and send it to the service worker's renderer
> process using FetchEventPreloadHandle.
> 
> 
> BUG=649558
> 
> Review-Url: https://codereview.chromium.org/2620463002
> Cr-Original-Commit-Position: refs/heads/master@{#445630}
> Committed:
https://chromium.googlesource.com/chromium/src/+/4c6b3b0f11b62e069d9d84ba99db...
> Review-Url: https://codereview.chromium.org/2620463002
> Cr-Commit-Position: refs/heads/master@{#445699}
> Committed:
https://chromium.googlesource.com/chromium/src/+/23276c71698583a0142e8213bada...

TBR=falken@chromium.org,pfeldman@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=649558
==========

to

==========
Revert of Show service worker navigation preload requests in DevTools Network
tab (patchset #10 id:480001 of https://codereview.chromium.org/2620463002/ )

Reason for revert:
Introduced DCHECK crash.

BUG=685493


Original issue's description:
> Show service worker navigation preload requests in DevTools Network tab.
> 
> Demo: https://youtu.be/I-Qe_Y-xYxE
> 
> Navigation Preload requests are initiated from the browser process.
> This is different from the normal network requests which are initiated from
the
> renderer process.
> 
> When the DevTools show the normal requests in the Network tab, DevTool's
> Network events (requestWillBeSent, responseReceived, loadingFinished etc) are
> dispatched via InspectorInstrumentation and InspectorNetworkAgent.
> 
> This CL introduces new DevTool's Network events (navigationPreloadSent,
> navigationPreloadResponseReceived, navigationPreloadFailed,
> navigationPreloadFinished) which are dispatched via InspectorInstrumentation
and
> InspectorNetworkAgent from ServiceWorkerContextClient.
> 
> In the normal requests case, we record the request sending timestamp when the
> renderer process will send the request in InspectorNetworkAgent::
> willSendRequestInternal(). But in the navigation preload case, we record the
> timestamp in the browser process, and send it to the service worker's renderer
> process using FetchEventPreloadHandle.
> 
> 
> BUG=649558
> 
> Review-Url: https://codereview.chromium.org/2620463002
> Cr-Original-Commit-Position: refs/heads/master@{#445630}
> Committed:
https://chromium.googlesource.com/chromium/src/+/4c6b3b0f11b62e069d9d84ba99db...
> Review-Url: https://codereview.chromium.org/2620463002
> Cr-Commit-Position: refs/heads/master@{#445699}
> Committed:
https://chromium.googlesource.com/chromium/src/+/23276c71698583a0142e8213bada...

TBR=falken@chromium.org,pfeldman@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=649558
==========

[horo, 2017-01-26 05:50:54]

On 2017/01/26 04:59:06, horo wrote:
> I understood the crash reason.
> I want to land the fix patch instead of reverting.
> https://codereview.chromium.org/2650973006/

I will revert the patch first.
https://codereview.chromium.org/2650973006/#msg6

[horo, 2017-01-26 05:51:48]

The CQ bit was checked by horo@chromium.org

[commit-bot: I haz the power, 2017-01-26 05:52:05]

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-26 05:54:11]

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

[commit-bot: I haz the power, 2017-01-26 05:54:13]

Try jobs failed on following builders:
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[horo, 2017-01-26 05:59:30]

The CQ bit was checked by horo@chromium.org

[commit-bot: I haz the power, 2017-01-26 05:59:45]

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-26 07:49:54]

CQ is committing da patch.

Bot data: {"patchset_id": 330001, "attempt_start_ts": 1485410370131710,
"parent_rev": "1ae83a08f3b0b6831e5dbdaae12635010367b246", "commit_rev":
"16de2c25e724f07d46b68f5758e49fae6f136073"}

[commit-bot: I haz the power, 2017-01-26 07:50:28]

Description was changed from

==========
Revert of Show service worker navigation preload requests in DevTools Network
tab (patchset #10 id:480001 of https://codereview.chromium.org/2620463002/ )

Reason for revert:
Introduced DCHECK crash.

BUG=685493


Original issue's description:
> Show service worker navigation preload requests in DevTools Network tab.
> 
> Demo: https://youtu.be/I-Qe_Y-xYxE
> 
> Navigation Preload requests are initiated from the browser process.
> This is different from the normal network requests which are initiated from
the
> renderer process.
> 
> When the DevTools show the normal requests in the Network tab, DevTool's
> Network events (requestWillBeSent, responseReceived, loadingFinished etc) are
> dispatched via InspectorInstrumentation and InspectorNetworkAgent.
> 
> This CL introduces new DevTool's Network events (navigationPreloadSent,
> navigationPreloadResponseReceived, navigationPreloadFailed,
> navigationPreloadFinished) which are dispatched via InspectorInstrumentation
and
> InspectorNetworkAgent from ServiceWorkerContextClient.
> 
> In the normal requests case, we record the request sending timestamp when the
> renderer process will send the request in InspectorNetworkAgent::
> willSendRequestInternal(). But in the navigation preload case, we record the
> timestamp in the browser process, and send it to the service worker's renderer
> process using FetchEventPreloadHandle.
> 
> 
> BUG=649558
> 
> Review-Url: https://codereview.chromium.org/2620463002
> Cr-Original-Commit-Position: refs/heads/master@{#445630}
> Committed:
https://chromium.googlesource.com/chromium/src/+/4c6b3b0f11b62e069d9d84ba99db...
> Review-Url: https://codereview.chromium.org/2620463002
> Cr-Commit-Position: refs/heads/master@{#445699}
> Committed:
https://chromium.googlesource.com/chromium/src/+/23276c71698583a0142e8213bada...

TBR=falken@chromium.org,pfeldman@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=649558
==========

to

==========
Revert of Show service worker navigation preload requests in DevTools Network
tab (patchset #10 id:480001 of https://codereview.chromium.org/2620463002/ )

Reason for revert:
Introduced DCHECK crash.

BUG=685493


Original issue's description:
> Show service worker navigation preload requests in DevTools Network tab.
> 
> Demo: https://youtu.be/I-Qe_Y-xYxE
> 
> Navigation Preload requests are initiated from the browser process.
> This is different from the normal network requests which are initiated from
the
> renderer process.
> 
> When the DevTools show the normal requests in the Network tab, DevTool's
> Network events (requestWillBeSent, responseReceived, loadingFinished etc) are
> dispatched via InspectorInstrumentation and InspectorNetworkAgent.
> 
> This CL introduces new DevTool's Network events (navigationPreloadSent,
> navigationPreloadResponseReceived, navigationPreloadFailed,
> navigationPreloadFinished) which are dispatched via InspectorInstrumentation
and
> InspectorNetworkAgent from ServiceWorkerContextClient.
> 
> In the normal requests case, we record the request sending timestamp when the
> renderer process will send the request in InspectorNetworkAgent::
> willSendRequestInternal(). But in the navigation preload case, we record the
> timestamp in the browser process, and send it to the service worker's renderer
> process using FetchEventPreloadHandle.
> 
> 
> BUG=649558
> 
> Review-Url: https://codereview.chromium.org/2620463002
> Cr-Original-Commit-Position: refs/heads/master@{#445630}
> Committed:
https://chromium.googlesource.com/chromium/src/+/4c6b3b0f11b62e069d9d84ba99db...
> Review-Url: https://codereview.chromium.org/2620463002
> Cr-Commit-Position: refs/heads/master@{#445699}
> Committed:
https://chromium.googlesource.com/chromium/src/+/23276c71698583a0142e8213bada...

TBR=falken@chromium.org,pfeldman@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=649558

Review-Url: https://codereview.chromium.org/2654433012
Cr-Commit-Position: refs/heads/master@{#446270}
Committed:
https://chromium.googlesource.com/chromium/src/+/16de2c25e724f07d46b68f5758e4...
==========

[commit-bot: I haz the power, 2017-01-26 07:50:30]

Committed patchset #3 (id:330001) as
https://chromium.googlesource.com/chromium/src/+/16de2c25e724f07d46b68f5758e4...