Simplify WorkerGlobalScope::m_eventListeners.

Simplify WorkerGlobalScope::m_eventListeners.

Switch to using a 'mere' HashSet<> for these event listeners. Mainly
done to diagnose GC instability, but removing the use of an involved
abstraction like HeapListHashSet<> is something wanted regardless.

R=
BUG=677654
Committed: https://crrev.com/68a05927d5b6f315e26229deacabdabd5f754bf1
Cr-Commit-Position: refs/heads/master@{#441364}

[sof, 2017-01-04 11:25:02]

The CQ bit was checked by sigbjornf@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2017-01-04 11:25:14]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[sof, 2017-01-04 11:25:33]

Description was changed from

==========
Simplify WorkerGlobalScope::m_eventListeners.

Switch to using a 'mere' HashSet<> for these event listeners. Mainly
done to diagnose GC instability, but removing the use of an involved
abstraction ListHashSet<> is something wanted regardless.

R=
BUG=677654
==========

to

==========
Simplify WorkerGlobalScope::m_eventListeners.

Switch to using a 'mere' HashSet<> for these event listeners. Mainly
done to diagnose GC instability, but removing the use of an involved
abstraction like HeapListHashSet<> is something wanted regardless.

R=
BUG=677654
==========

[commit-bot: I haz the power, 2017-01-04 11:52:32]

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

[commit-bot: I haz the power, 2017-01-04 11:52:33]

Dry run: 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...)

[sof, 2017-01-04 12:04:36]

The CQ bit was checked by sigbjornf@opera.com to run a CQ dry run

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

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[sof, 2017-01-04 13:10:28]

sigbjornf@opera.com changed reviewers:
+ haraken@chromium.org, jochen@chromium.org

[sof, 2017-01-04 13:10:29]

please take a look (and a happy new year :) )

There's a general increase in GC related bugs at the moment, at least it appears
so from here. This is another attempt at diagnosing one such. iow, speculative.

[jochen (gone - plz use gerrit), 2017-01-04 13:19:26]

lgtm

[commit-bot: I haz the power, 2017-01-04 13:33:58]

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

[commit-bot: I haz the power, 2017-01-04 13:33:58]

Dry run: This issue passed the CQ dry run.

[sof, 2017-01-04 13:35:52]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2017-01-04 13:36:10]

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-04 13:39:57]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1483536952074770,
"parent_rev": "4c0c514281e0d5ccfecad158a8f965068f705c42", "commit_rev":
"fd829ed7a3102bc73a151cee5872be6ee71cc597"}

[commit-bot: I haz the power, 2017-01-04 13:40:25]

Description was changed from

==========
Simplify WorkerGlobalScope::m_eventListeners.

Switch to using a 'mere' HashSet<> for these event listeners. Mainly
done to diagnose GC instability, but removing the use of an involved
abstraction like HeapListHashSet<> is something wanted regardless.

R=
BUG=677654
==========

to

==========
Simplify WorkerGlobalScope::m_eventListeners.

Switch to using a 'mere' HashSet<> for these event listeners. Mainly
done to diagnose GC instability, but removing the use of an involved
abstraction like HeapListHashSet<> is something wanted regardless.

R=
BUG=677654

Review-Url: https://codereview.chromium.org/2609413002
==========

[commit-bot: I haz the power, 2017-01-04 13:40:26]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2017-01-04 13:42:31]

Description was changed from

==========
Simplify WorkerGlobalScope::m_eventListeners.

Switch to using a 'mere' HashSet<> for these event listeners. Mainly
done to diagnose GC instability, but removing the use of an involved
abstraction like HeapListHashSet<> is something wanted regardless.

R=
BUG=677654

Review-Url: https://codereview.chromium.org/2609413002
==========

to

==========
Simplify WorkerGlobalScope::m_eventListeners.

Switch to using a 'mere' HashSet<> for these event listeners. Mainly
done to diagnose GC instability, but removing the use of an involved
abstraction like HeapListHashSet<> is something wanted regardless.

R=
BUG=677654

Committed: https://crrev.com/68a05927d5b6f315e26229deacabdabd5f754bf1
Cr-Commit-Position: refs/heads/master@{#441364}
==========

[commit-bot: I haz the power, 2017-01-04 13:42:32]

Patchset 3 (id:??) landed as
https://crrev.com/68a05927d5b6f315e26229deacabdabd5f754bf1
Cr-Commit-Position: refs/heads/master@{#441364}

[haraken, 2017-01-04 14:29:02]

LGTM