Revert of Dispatch unload event for swapped-out subframes.

Revert of Dispatch unload event for swapped-out subframes. (patchset #2 id:20001 of https://codereview.chromium.org/2941783002/ )

Reason for revert:
This CL is responsible for https://crbug.com/733940

Original issue's description:
> Avoid unnecessary direct dispatching of unload events from RFI::OnSwapOut.
> 
> There is no need to call DispatchUnloadEvent directly from
> RenderFrameImpl::OnSwapOut, because the unload events will
> be dispatched even without this call - this will happen via
> the following call chain:
> - Document::DispatchUnloadEvents
> - FrameLoader::DispatchUnloadEvent
> - LocalFrame::Detach
> - WebFrame::Swap
> - RenderFrameImpl::OnSwapOut
> 
> This also means that the old TODO from the code doesn't need fixing - we
> already dispatch unload events for swapped out subframes.
> 
> BUG=733314
> TEST=Manualy set main frame's window.onunload and verified it still fires.
> 
> Review-Url: https://codereview.chromium.org/2941783002
> Cr-Commit-Position: refs/heads/master@{#479538}
> Committed: https://chromium.googlesource.com/chromium/src/+/6e8a61ba16f529f02841fb75a50f15ec45726411

TBR=creis@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=733314
Review-Url: https://codereview.chromium.org/2938423002
Cr-Commit-Position: refs/heads/master@{#480133}
Committed: https://chromium.googlesource.com/chromium/src/+/bfda1c11ee19e7ca8fe2c73d33db7ab9be9f5a36

[Łukasz Anforowicz, 2017-06-16 18:34:27]

The CQ bit was checked by lukasza@chromium.org

[Łukasz Anforowicz, 2017-06-16 18:34:27]

Created Revert of Dispatch unload event for swapped-out subframes.

[commit-bot: I haz the power, 2017-06-16 18:34:43]

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-06-16 19:55:11]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1497638067043940, "parent_rev":
"dc4c405b2c520742e753637cc527c14d2f7fb4c3", "commit_rev":
"bfda1c11ee19e7ca8fe2c73d33db7ab9be9f5a36"}

[commit-bot: I haz the power, 2017-06-16 19:55:24]

Description was changed from

==========
Revert of Dispatch unload event for swapped-out subframes. (patchset #2 id:20001
of https://codereview.chromium.org/2941783002/ )

Reason for revert:
This CL is responsible for https://crbug.com/733940

Original issue's description:
> Avoid unnecessary direct dispatching of unload events from RFI::OnSwapOut.
> 
> There is no need to call DispatchUnloadEvent directly from
> RenderFrameImpl::OnSwapOut, because the unload events will
> be dispatched even without this call - this will happen via
> the following call chain:
> - Document::DispatchUnloadEvents
> - FrameLoader::DispatchUnloadEvent
> - LocalFrame::Detach
> - WebFrame::Swap
> - RenderFrameImpl::OnSwapOut
> 
> This also means that the old TODO from the code doesn't need fixing - we
> already dispatch unload events for swapped out subframes.
> 
> BUG=733314
> TEST=Manualy set main frame's window.onunload and verified it still fires.
> 
> Review-Url: https://codereview.chromium.org/2941783002
> Cr-Commit-Position: refs/heads/master@{#479538}
> Committed:
https://chromium.googlesource.com/chromium/src/+/6e8a61ba16f529f02841fb75a50f...

TBR=creis@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=733314
==========

to

==========
Revert of Dispatch unload event for swapped-out subframes. (patchset #2 id:20001
of https://codereview.chromium.org/2941783002/ )

Reason for revert:
This CL is responsible for https://crbug.com/733940

Original issue's description:
> Avoid unnecessary direct dispatching of unload events from RFI::OnSwapOut.
> 
> There is no need to call DispatchUnloadEvent directly from
> RenderFrameImpl::OnSwapOut, because the unload events will
> be dispatched even without this call - this will happen via
> the following call chain:
> - Document::DispatchUnloadEvents
> - FrameLoader::DispatchUnloadEvent
> - LocalFrame::Detach
> - WebFrame::Swap
> - RenderFrameImpl::OnSwapOut
> 
> This also means that the old TODO from the code doesn't need fixing - we
> already dispatch unload events for swapped out subframes.
> 
> BUG=733314
> TEST=Manualy set main frame's window.onunload and verified it still fires.
> 
> Review-Url: https://codereview.chromium.org/2941783002
> Cr-Commit-Position: refs/heads/master@{#479538}
> Committed:
https://chromium.googlesource.com/chromium/src/+/6e8a61ba16f529f02841fb75a50f...

TBR=creis@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=733314

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

[commit-bot: I haz the power, 2017-06-16 19:55:26]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/bfda1c11ee19e7ca8fe2c73d33db...