Revert of Expose GlobalRequestID in NavigationHandle and ResourceRequestInfo.

Revert of Expose GlobalRequestID in NavigationHandle and ResourceRequestInfo. (patchset #14 id:400001 of https://codereview.chromium.org/2413663003/ )

Reason for revert:
Looks like this is causing a test to crash on Win7 debug:

https://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%28dbg%29%281%29/builds/54581

https://uberchromegw.corp.google.com/i/chromium.win/builders/Win7%20Tests%20%28dbg%29%281%29/builds/54581/steps/browser_tests%20on%20Windows-7-SP1/logs/PrerenderBrowserTestWithExtensions.StreamsTest

So I'm reverting it, after double-checking w/ ryansturm@. Sorry!

Original issue's description:
> Expose GlobalRequestID in NavigationHandle and ResourceRequestInfo.
>
> Create DataUseRecorder instances associated with page loads.
> Instances are associated with either pending navigations or render
> frame hosts depending on the state of the page load. Use
> GlobalRequestID to associate MAIN_FRAME URLRequests with their
> corresponding pending navigation. Complete life-cycle of the
> DataUseRecorders will be implemented in a forthcoming cl.
>
> BUG=660065
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
>
> Committed: https://crrev.com/763c3fe382f8ae7c39cfa28191eab55b1036e4e6
> Cr-Commit-Position: refs/heads/master@{#431282}

TBR=clamy@chromium.org,rajendrant@chromium.org,ryansturm@chromium.org,kundaji@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=660065
Committed: https://crrev.com/093c9fbda78ab82fd22e1b0c0ae94942eff2660b
Cr-Commit-Position: refs/heads/master@{#431368}

[Dirk Pranke, 2016-11-10 21:48:36]

The CQ bit was checked by dpranke@chromium.org

[Dirk Pranke, 2016-11-10 21:48:37]

Created Revert of Expose GlobalRequestID in NavigationHandle and
ResourceRequestInfo.

[commit-bot: I haz the power, 2016-11-10 21:49:17]

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-11-10 21:51:36]

Description was changed from

==========
Revert of Expose GlobalRequestID in NavigationHandle and ResourceRequestInfo.
(patchset #14 id:400001 of https://codereview.chromium.org/2413663003/ )

Reason for revert:
Looks like this is causing a test to crash on Win7 debug:

https://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%28dbg%29%2...

https://uberchromegw.corp.google.com/i/chromium.win/builders/Win7%20Tests%20%...

So I'm reverting it, after double-checking w/ ryansturm@. Sorry!

Original issue's description:
> Expose GlobalRequestID in NavigationHandle and ResourceRequestInfo.
> 
> Create DataUseRecorder instances associated with page loads.
> Instances are associated with either pending navigations or render
> frame hosts depending on the state of the page load. Use
> GlobalRequestID to associate MAIN_FRAME URLRequests with their
> corresponding pending navigation. Complete life-cycle of the
> DataUseRecorders will be implemented in a forthcoming cl.
> 
> BUG=660065
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
> 
> Committed: https://crrev.com/763c3fe382f8ae7c39cfa28191eab55b1036e4e6
> Cr-Commit-Position: refs/heads/master@{#431282}

TBR=clamy@chromium.org,rajendrant@chromium.org,ryansturm@chromium.org,kundaji...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=660065
==========

to

==========
Revert of Expose GlobalRequestID in NavigationHandle and ResourceRequestInfo.
(patchset #14 id:400001 of https://codereview.chromium.org/2413663003/ )

Reason for revert:
Looks like this is causing a test to crash on Win7 debug:

https://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%28dbg%29%2...

https://uberchromegw.corp.google.com/i/chromium.win/builders/Win7%20Tests%20%...

So I'm reverting it, after double-checking w/ ryansturm@. Sorry!

Original issue's description:
> Expose GlobalRequestID in NavigationHandle and ResourceRequestInfo.
> 
> Create DataUseRecorder instances associated with page loads.
> Instances are associated with either pending navigations or render
> frame hosts depending on the state of the page load. Use
> GlobalRequestID to associate MAIN_FRAME URLRequests with their
> corresponding pending navigation. Complete life-cycle of the
> DataUseRecorders will be implemented in a forthcoming cl.
> 
> BUG=660065
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
> 
> Committed: https://crrev.com/763c3fe382f8ae7c39cfa28191eab55b1036e4e6
> Cr-Commit-Position: refs/heads/master@{#431282}

TBR=clamy@chromium.org,rajendrant@chromium.org,ryansturm@chromium.org,kundaji...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=660065
==========

[commit-bot: I haz the power, 2016-11-10 21:51:37]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-10 23:09:57]

Description was changed from

==========
Revert of Expose GlobalRequestID in NavigationHandle and ResourceRequestInfo.
(patchset #14 id:400001 of https://codereview.chromium.org/2413663003/ )

Reason for revert:
Looks like this is causing a test to crash on Win7 debug:

https://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%28dbg%29%2...

https://uberchromegw.corp.google.com/i/chromium.win/builders/Win7%20Tests%20%...

So I'm reverting it, after double-checking w/ ryansturm@. Sorry!

Original issue's description:
> Expose GlobalRequestID in NavigationHandle and ResourceRequestInfo.
> 
> Create DataUseRecorder instances associated with page loads.
> Instances are associated with either pending navigations or render
> frame hosts depending on the state of the page load. Use
> GlobalRequestID to associate MAIN_FRAME URLRequests with their
> corresponding pending navigation. Complete life-cycle of the
> DataUseRecorders will be implemented in a forthcoming cl.
> 
> BUG=660065
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
> 
> Committed: https://crrev.com/763c3fe382f8ae7c39cfa28191eab55b1036e4e6
> Cr-Commit-Position: refs/heads/master@{#431282}

TBR=clamy@chromium.org,rajendrant@chromium.org,ryansturm@chromium.org,kundaji...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=660065
==========

to

==========
Revert of Expose GlobalRequestID in NavigationHandle and ResourceRequestInfo.
(patchset #14 id:400001 of https://codereview.chromium.org/2413663003/ )

Reason for revert:
Looks like this is causing a test to crash on Win7 debug:

https://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%28dbg%29%2...

https://uberchromegw.corp.google.com/i/chromium.win/builders/Win7%20Tests%20%...

So I'm reverting it, after double-checking w/ ryansturm@. Sorry!

Original issue's description:
> Expose GlobalRequestID in NavigationHandle and ResourceRequestInfo.
>
> Create DataUseRecorder instances associated with page loads.
> Instances are associated with either pending navigations or render
> frame hosts depending on the state of the page load. Use
> GlobalRequestID to associate MAIN_FRAME URLRequests with their
> corresponding pending navigation. Complete life-cycle of the
> DataUseRecorders will be implemented in a forthcoming cl.
>
> BUG=660065
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
>
> Committed: https://crrev.com/763c3fe382f8ae7c39cfa28191eab55b1036e4e6
> Cr-Commit-Position: refs/heads/master@{#431282}

TBR=clamy@chromium.org,rajendrant@chromium.org,ryansturm@chromium.org,kundaji...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=660065

Committed: https://crrev.com/093c9fbda78ab82fd22e1b0c0ae94942eff2660b
Cr-Commit-Position: refs/heads/master@{#431368}
==========

[commit-bot: I haz the power, 2016-11-10 23:09:58]

Patchset 1 (id:??) landed as
https://crrev.com/093c9fbda78ab82fd22e1b0c0ae94942eff2660b
Cr-Commit-Position: refs/heads/master@{#431368}