Split out Members, Persistents and SelfKeepAlive in separate headers.

Split out Members, Persistents and SelfKeepAlive in separate headers.

Complete the migration of definitions out of Handle.h, adding new
header files for the family of Member<> types that Oilpan supports.
Similarly for Persistent<> and its variants, and the SelfKeepAlive<>
abstraction.

To further prune Handle.h, let go of the unused ScopedDisposal (light)
abstraction.

R=
BUG=

Committed: https://crrev.com/3e5bcb132b1af136ba6faa1c152dcba2c298549f
Cr-Commit-Position: refs/heads/master@{#395318}

[sof, 2016-05-22 14:43:19]

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

[sof, 2016-05-22 14:43:20]

please take a look.

[haraken, 2016-05-22 14:46:53]

I'm fine with this change but is there any reason you want to split out
Handle.h?

[sof, 2016-05-22 15:01:13]

On 2016/05/22 14:46:53, haraken wrote:
> I'm fine with this change but is there any reason you want to split out
> Handle.h?

Cleaner & consistent with the codebase in general to have related classes in
header files of their own + 'going forward', I think it is preferable if we try
to minimize Handle.h sprawl.

Verified locally that this doesn't regress compile times.

[haraken, 2016-05-23 01:23:47]

On 2016/05/22 15:01:13, sof wrote:
> On 2016/05/22 14:46:53, haraken wrote:
> > I'm fine with this change but is there any reason you want to split out
> > Handle.h?
> 
> Cleaner & consistent with the codebase in general to have related classes in
> header files of their own + 'going forward', I think it is preferable if we
try
> to minimize Handle.h sprawl.
> 
> Verified locally that this doesn't regress compile times.

LGTM

[sof, 2016-05-23 07:03:45]

The CQ bit was checked by sigbjornf@opera.com

[sof, 2016-05-23 07:03:46]

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

[commit-bot: I haz the power, 2016-05-23 07:04:00]

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

[commit-bot: I haz the power, 2016-05-23 09:32:23]

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

[commit-bot: I haz the power, 2016-05-23 09:32:24]

Try jobs failed on following builders:
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[sof, 2016-05-23 10:39:12]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2016-05-23 10:39:32]

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

[sof, 2016-05-23 13:07:41]

The CQ bit was unchecked by sigbjornf@opera.com

[sof, 2016-05-23 13:08:00]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2016-05-23 13:08:13]

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

[commit-bot: I haz the power, 2016-05-23 14:28:27]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-05-23 14:30:24]

Description was changed from

==========
Split out Members, Persistents and SelfKeepAlive in separate headers.

Complete the migration of definitions out of Handle.h, adding new
header files for the family of Member<> types that Oilpan supports.
Similarly for Persistent<> and its variants, and the SelfKeepAlive<>
abstraction.

To further prune Handle.h, let go of the unused ScopedDisposal (light)
abstraction.

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

to

==========
Split out Members, Persistents and SelfKeepAlive in separate headers.

Complete the migration of definitions out of Handle.h, adding new
header files for the family of Member<> types that Oilpan supports.
Similarly for Persistent<> and its variants, and the SelfKeepAlive<>
abstraction.

To further prune Handle.h, let go of the unused ScopedDisposal (light)
abstraction.

R=
BUG=

Committed: https://crrev.com/3e5bcb132b1af136ba6faa1c152dcba2c298549f
Cr-Commit-Position: refs/heads/master@{#395318}
==========

[commit-bot: I haz the power, 2016-05-23 14:30:26]

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