Tidy CrossThreadCopier.h inclusion.

Tidy CrossThreadCopier.h inclusion.

This header file declares a set of types as being cross-copiable; avoid
bringing in all of Oilpan for its two cross-thread persistent types.

Tidy up some downstream header files which were implicitly depending on
Oilpan being included here.

R=
BUG=597856, 624419
Committed: https://crrev.com/f2b88d1e91a969a91bc4ca9bdebb4dd1bd0edfdd
Cr-Commit-Position: refs/heads/master@{#402823}

[sof, 2016-06-29 10:57:36]

Description was changed from

==========
Tidy up CrossThreadCopier.h inclusion.

This header file declares a set of types as being cross-copiable; avoid
bringing in all of Oilpan for its two cross-thread persistent types.

Tidy up some downstream header files which were implicitly depending on
Oilpan being included here.

R=
BUG=597856
==========

to

==========
Tidy CrossThreadCopier.h inclusion.

This header file declares a set of types as being cross-copiable; avoid
bringing in all of Oilpan for its two cross-thread persistent types.

Tidy up some downstream header files which were implicitly depending on
Oilpan being included here.

R=
BUG=597856
==========

[sof, 2016-06-29 11:02:06]

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

[sof, 2016-06-29 11:02:07]

just something noticed while looking at r402738 - not sure it is worth it.
CrossThreadCopier.h could now be moved to wtf/, if that's a goal.

[haraken, 2016-06-29 11:39:42]

LGTM

[hiroshige, 2016-06-29 14:35:25]

lgtm!

> CrossThreadCopier.h could now be moved to wtf/, if that's a goal.
It would be nice because we can move threadSafeBind() to wtf/ (currently bind()
is in wtf/ and threadSafeBind() is in platform).

[sof, 2016-06-29 14:41:22]

On 2016/06/29 14:35:25, hiroshige wrote:
> lgtm!
> 
> > CrossThreadCopier.h could now be moved to wtf/, if that's a goal.
> It would be nice because we can move threadSafeBind() to wtf/ (currently
bind()
> is in wtf/ and threadSafeBind() is in platform).

ok, let's land this small step towards that.

[sof, 2016-06-29 14:41:29]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2016-06-29 14:41:43]

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

[hiroshige, 2016-06-29 14:44:30]

Created Issue 624419 for tracking move of CrossThreadCopier. Could you add it to
BUG= field of the CL description?

[sof, 2016-06-29 14:46:05]

The CQ bit was unchecked by sigbjornf@opera.com

[sof, 2016-06-29 14:46:22]

Description was changed from

==========
Tidy CrossThreadCopier.h inclusion.

This header file declares a set of types as being cross-copiable; avoid
bringing in all of Oilpan for its two cross-thread persistent types.

Tidy up some downstream header files which were implicitly depending on
Oilpan being included here.

R=
BUG=597856
==========

to

==========
Tidy CrossThreadCopier.h inclusion.

This header file declares a set of types as being cross-copiable; avoid
bringing in all of Oilpan for its two cross-thread persistent types.

Tidy up some downstream header files which were implicitly depending on
Oilpan being included here.

R=
BUG=597856, 624419
==========

[sof, 2016-06-29 14:46:29]

The CQ bit was checked by sigbjornf@opera.com

[sof, 2016-06-29 14:46:38]

On 2016/06/29 14:44:30, hiroshige wrote:
> Created Issue 624419 for tracking move of CrossThreadCopier. Could you add it
to
> BUG= field of the CL description?

thanks, done.

[commit-bot: I haz the power, 2016-06-29 14:46:49]

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, 2016-06-29 15:54:28]

Description was changed from

==========
Tidy CrossThreadCopier.h inclusion.

This header file declares a set of types as being cross-copiable; avoid
bringing in all of Oilpan for its two cross-thread persistent types.

Tidy up some downstream header files which were implicitly depending on
Oilpan being included here.

R=
BUG=597856, 624419
==========

to

==========
Tidy CrossThreadCopier.h inclusion.

This header file declares a set of types as being cross-copiable; avoid
bringing in all of Oilpan for its two cross-thread persistent types.

Tidy up some downstream header files which were implicitly depending on
Oilpan being included here.

R=
BUG=597856, 624419
==========

[commit-bot: I haz the power, 2016-06-29 15:54:29]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-29 15:57:24]

Description was changed from

==========
Tidy CrossThreadCopier.h inclusion.

This header file declares a set of types as being cross-copiable; avoid
bringing in all of Oilpan for its two cross-thread persistent types.

Tidy up some downstream header files which were implicitly depending on
Oilpan being included here.

R=
BUG=597856, 624419
==========

to

==========
Tidy CrossThreadCopier.h inclusion.

This header file declares a set of types as being cross-copiable; avoid
bringing in all of Oilpan for its two cross-thread persistent types.

Tidy up some downstream header files which were implicitly depending on
Oilpan being included here.

R=
BUG=597856, 624419

Committed: https://crrev.com/f2b88d1e91a969a91bc4ca9bdebb4dd1bd0edfdd
Cr-Commit-Position: refs/heads/master@{#402823}
==========

[commit-bot: I haz the power, 2016-06-29 15:57:25]

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