Revert of Keep track in the browser of which frames have onunload and onbeforeunload handlers.

Revert of Keep track in the browser of which frames have onunload and onbeforeunload handlers. (patchset #10 id:180001 of https://codereview.chromium.org/2783723002/ )

Reason for revert:
Exposed races with FrameHostMsg_DidStopLoading

BUG=710062
Original issue's description:
> Keep track in the browser of which frames have onunload and onbeforeunload handlers.
> 
> This allows PlzNavigate to only pause fetching the request if it knows that the page has an onbeforeunload handler, where today it currently always goes to the renderer. This avoids delaying the network requests in the 95% cases that don't onbeforeunload handler on a process hop to the renderer.
> 
> BUG=365039, 705559
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
> R=dcheng@chromium.org, nasko@chromium.org
> 
> Review-Url: https://codereview.chromium.org/2783723002 .
> Cr-Commit-Position: refs/heads/master@{#460581}
> Committed: https://chromium.googlesource.com/chromium/src/+/a2c84a7664eceb5cbeaae5bf1ec32da0060e3097

TBR=nasko@chromium.org,dcheng@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=365039, 705559
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

Review-Url: https://codereview.chromium.org/2812743002
Cr-Commit-Position: refs/heads/master@{#463510}
Committed: https://chromium.googlesource.com/chromium/src/+/4044498152ad71d05c57aee559cba6a97db82375

[jam, 2017-04-10 22:11:13]

The CQ bit was checked by jam@chromium.org

[jam, 2017-04-10 22:11:14]

Created Revert of Keep track in the browser of which frames have onunload and
onbeforeunload handlers.

[commit-bot: I haz the power, 2017-04-10 22:12:11]

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-04-10 22:15:43]

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

[commit-bot: I haz the power, 2017-04-10 22:15:44]

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

[jam, 2017-04-10 22:52:17]

Description was changed from

==========
Revert of Keep track in the browser of which frames have onunload and
onbeforeunload handlers. (patchset #10 id:180001 of
https://codereview.chromium.org/2783723002/ )

Reason for revert:
Exposed races with FrameHostMsg_DidStopLoading

BUG=710062


Original issue's description:
> Keep track in the browser of which frames have onunload and onbeforeunload
handlers.
> 
> This allows PlzNavigate to only pause fetching the request if it knows that
the page has an onbeforeunload handler, where today it currently always goes to
the renderer. This avoids delaying the network requests in the 95% cases that
don't onbeforeunload handler on a process hop to the renderer.
> 
> BUG=365039,705559
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
> R=dcheng@chromium.org, nasko@chromium.org
> 
> Review-Url: https://codereview.chromium.org/2783723002 .
> Cr-Commit-Position: refs/heads/master@{#460581}
> Committed:
https://chromium.googlesource.com/chromium/src/+/a2c84a7664eceb5cbeaae5bf1ec3...

TBR=nasko@chromium.org,dcheng@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=365039,705559
==========

to

==========
Revert of Keep track in the browser of which frames have onunload and
onbeforeunload handlers. (patchset #10 id:180001 of
https://codereview.chromium.org/2783723002/ )

Reason for revert:
Exposed races with FrameHostMsg_DidStopLoading

BUG=710062


Original issue's description:
> Keep track in the browser of which frames have onunload and onbeforeunload
handlers.
> 
> This allows PlzNavigate to only pause fetching the request if it knows that
the page has an onbeforeunload handler, where today it currently always goes to
the renderer. This avoids delaying the network requests in the 95% cases that
don't onbeforeunload handler on a process hop to the renderer.
> 
> BUG=365039,705559
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
> R=dcheng@chromium.org, nasko@chromium.org
> 
> Review-Url: https://codereview.chromium.org/2783723002 .
> Cr-Commit-Position: refs/heads/master@{#460581}
> Committed:
https://chromium.googlesource.com/chromium/src/+/a2c84a7664eceb5cbeaae5bf1ec3...

TBR=nasko@chromium.org,dcheng@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=365039,705559
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

[jam, 2017-04-10 22:52:38]

The CQ bit was checked by jam@chromium.org

[commit-bot: I haz the power, 2017-04-10 22:53:20]

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-04-10 23:04:16]

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

[commit-bot: I haz the power, 2017-04-10 23:04:17]

Try jobs failed on following builders:
  cast_shell_linux on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[jam, 2017-04-11 00:04:19]

The CQ bit was checked by jam@chromium.org

[commit-bot: I haz the power, 2017-04-11 00:04:49]

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-04-11 00:19:20]

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

[commit-bot: I haz the power, 2017-04-11 00:19:22]

Try jobs failed on following builders:
  cast_shell_android on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/cast_shell_a...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[jam, 2017-04-11 00:24:02]

The CQ bit was checked by jam@chromium.org

[commit-bot: I haz the power, 2017-04-11 00:24:49]

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

CQ is committing da patch.

Bot data: {"patchset_id": 300001, "attempt_start_ts": 1491870242056400,
"parent_rev": "0dea0097ea24907ed4c0f30fb4180191f378f00e", "commit_rev":
"4044498152ad71d05c57aee559cba6a97db82375"}

[commit-bot: I haz the power, 2017-04-11 03:02:36]

Description was changed from

==========
Revert of Keep track in the browser of which frames have onunload and
onbeforeunload handlers. (patchset #10 id:180001 of
https://codereview.chromium.org/2783723002/ )

Reason for revert:
Exposed races with FrameHostMsg_DidStopLoading

BUG=710062


Original issue's description:
> Keep track in the browser of which frames have onunload and onbeforeunload
handlers.
> 
> This allows PlzNavigate to only pause fetching the request if it knows that
the page has an onbeforeunload handler, where today it currently always goes to
the renderer. This avoids delaying the network requests in the 95% cases that
don't onbeforeunload handler on a process hop to the renderer.
> 
> BUG=365039,705559
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
> R=dcheng@chromium.org, nasko@chromium.org
> 
> Review-Url: https://codereview.chromium.org/2783723002 .
> Cr-Commit-Position: refs/heads/master@{#460581}
> Committed:
https://chromium.googlesource.com/chromium/src/+/a2c84a7664eceb5cbeaae5bf1ec3...

TBR=nasko@chromium.org,dcheng@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=365039,705559
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Revert of Keep track in the browser of which frames have onunload and
onbeforeunload handlers. (patchset #10 id:180001 of
https://codereview.chromium.org/2783723002/ )

Reason for revert:
Exposed races with FrameHostMsg_DidStopLoading

BUG=710062


Original issue's description:
> Keep track in the browser of which frames have onunload and onbeforeunload
handlers.
> 
> This allows PlzNavigate to only pause fetching the request if it knows that
the page has an onbeforeunload handler, where today it currently always goes to
the renderer. This avoids delaying the network requests in the 95% cases that
don't onbeforeunload handler on a process hop to the renderer.
> 
> BUG=365039,705559
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
> R=dcheng@chromium.org, nasko@chromium.org
> 
> Review-Url: https://codereview.chromium.org/2783723002 .
> Cr-Commit-Position: refs/heads/master@{#460581}
> Committed:
https://chromium.googlesource.com/chromium/src/+/a2c84a7664eceb5cbeaae5bf1ec3...

TBR=nasko@chromium.org,dcheng@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=365039,705559
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

Review-Url: https://codereview.chromium.org/2812743002
Cr-Commit-Position: refs/heads/master@{#463510}
Committed:
https://chromium.googlesource.com/chromium/src/+/4044498152ad71d05c57aee559cb...
==========

[commit-bot: I haz the power, 2017-04-11 03:02:37]

Committed patchset #4 (id:300001) as
https://chromium.googlesource.com/chromium/src/+/4044498152ad71d05c57aee559cb...