Suppress dumping number of unload handlers when LayoutTest ends

Suppress dumping number of unload handlers when LayoutTest ends

Before this CL, content shell (in DRT mode) dumped number of onunload
handlers when test end.
This feature was added 5 years ago in WebKit for undisclosed Safari bug:
https://blink.lc/blink/commit/?id=4463bfd9a3882721f2252d3b380d60213a66dd37

This unload handler count dump isn't useful in current Blink, and
it is also leading to flakiness of LayoutTests results after
Blink Scheduler / Threaded Data Parser changes, as they are not deterministic
if onunload handler on iframe is registered before/after its navigation.

This CL removes this feature, and should unblock "Threaded Parser Data
Receiver" implementation in Blink.

BUG=277124
Committed: https://crrev.com/0a2aaff2a36dbb2634af9eaeeb3cd44d6be77e95
Cr-Commit-Position: refs/heads/master@{#332588}

[kouhei (in TOK), 2015-06-03 05:12:11]

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

[kouhei (in TOK), 2015-06-03 05:13:50]

tkent: Do you have any thoughts on this change?

I'd like to chat about how I should land this if you think this is ok.
- the change is content/shell, but rebaseline must be done in third_party/WebKit

[tkent, 2015-06-03 05:49:12]

On 2015/06/03 05:13:50, kouhei wrote:
> tkent: Do you have any thoughts on this change?
> 
> I'd like to chat about how I should land this if you think this is ok.
> - the change is content/shell, but rebaseline must be done in
third_party/WebKit

lgtm.

You need to mark affected layout tests as NeedsManualRebaseline before landing
this CL, and need to change it to NeedsRebaseline after landing this CL.

[kouhei (in TOK), 2015-06-03 05:50:32]

On 2015/06/03 05:49:12, tkent wrote:
> On 2015/06/03 05:13:50, kouhei wrote:
> > tkent: Do you have any thoughts on this change?
> > 
> > I'd like to chat about how I should land this if you think this is ok.
> > - the change is content/shell, but rebaseline must be done in
> third_party/WebKit
> 
> lgtm.
> 
> You need to mark affected layout tests as NeedsManualRebaseline before landing
> this CL, and need to change it to NeedsRebaseline after landing this CL.

Will do. Thanks!

[kouhei (in TOK), 2015-06-03 10:01:24]

The CQ bit was checked by kouhei@chromium.org

[commit-bot: I haz the power, 2015-06-03 10:02:28]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1168553003/1

[commit-bot: I haz the power, 2015-06-03 10:09:57]

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

[commit-bot: I haz the power, 2015-06-03 10:09:58]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)
  win8_chromium_rel on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win8_chromium_rel...)

[kouhei (in TOK), 2015-06-03 10:36:19]

The CQ bit was checked by kouhei@chromium.org

[kouhei (in TOK), 2015-06-03 10:36:19]

The patchset sent to the CQ was uploaded after l-g-t-m from tkent@chromium.org
Link to the patchset: https://codereview.chromium.org/1168553003/#ps20001
(title: "rebased")

[commit-bot: I haz the power, 2015-06-03 10:36:36]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1168553003/20001

[commit-bot: I haz the power, 2015-06-03 11:10:19]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-06-03 11:11:12]

Patchset 2 (id:??) landed as
https://crrev.com/0a2aaff2a36dbb2634af9eaeeb3cd44d6be77e95
Cr-Commit-Position: refs/heads/master@{#332588}