Turn ResourceLoaderSet into a part object.

Turn ResourceLoaderSet into a part object.

The abstraction is really a derived HashSet, so make it
a part object.

(Change done in part to diagnose what appears to be a
heap overwrite involving ResourceLoaderSet.)

R=
BUG=615673
Committed: https://crrev.com/3302313bb0d52bdfaa8796204fc3ced1c64a8197
Cr-Commit-Position: refs/heads/master@{#396657}

[sof, 2016-05-29 07:33:01]

Description was changed from

==========
Turn ResourceLoaderSet into a part object.

The abstraction is really a derived HashSet, so make it
a part object.

(Change done in part to diagnose what appears to be a
heap overwrite involving ResourceLoaderSet.)

R=
BUG=
==========

to

==========
Turn ResourceLoaderSet into a part object.

The abstraction is really a derived HashSet, so make it
a part object.

(Change done in part to diagnose what appears to be a
heap overwrite involving ResourceLoaderSet.)

R=
BUG=615673
==========

[sof, 2016-05-29 07:37:35]

sigbjornf@opera.com changed reviewers:
+ oilpan-reviews@chromium.org

[sof, 2016-05-29 07:37:36]

please take a look.

Trying to perturb what seems to be a heap overwrite/mutation over
ResourceLoaderSet on the associated bug, moving it inside ResourceFetcher's
allocation, but it makes some sense to make this change in representation
regardless.

(meta: it is somewhat tiresome for Oilpan to be perpetually implicated in these
issues, with there being no known sign that it is the infrastructure that's
misbehaving.)

[haraken, 2016-05-29 09:22:52]

LGTM

[sof, 2016-05-29 10:00:37]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2016-05-29 10:00:45]

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

[commit-bot: I haz the power, 2016-05-29 10:52:57]

Description was changed from

==========
Turn ResourceLoaderSet into a part object.

The abstraction is really a derived HashSet, so make it
a part object.

(Change done in part to diagnose what appears to be a
heap overwrite involving ResourceLoaderSet.)

R=
BUG=615673
==========

to

==========
Turn ResourceLoaderSet into a part object.

The abstraction is really a derived HashSet, so make it
a part object.

(Change done in part to diagnose what appears to be a
heap overwrite involving ResourceLoaderSet.)

R=
BUG=615673
==========

[commit-bot: I haz the power, 2016-05-29 10:52:58]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-05-29 10:54:56]

Description was changed from

==========
Turn ResourceLoaderSet into a part object.

The abstraction is really a derived HashSet, so make it
a part object.

(Change done in part to diagnose what appears to be a
heap overwrite involving ResourceLoaderSet.)

R=
BUG=615673
==========

to

==========
Turn ResourceLoaderSet into a part object.

The abstraction is really a derived HashSet, so make it
a part object.

(Change done in part to diagnose what appears to be a
heap overwrite involving ResourceLoaderSet.)

R=
BUG=615673

Committed: https://crrev.com/3302313bb0d52bdfaa8796204fc3ced1c64a8197
Cr-Commit-Position: refs/heads/master@{#396657}
==========

[commit-bot: I haz the power, 2016-05-29 10:54:57]

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