Don't include LayoutObject-derived headers where not needed.

Don't include LayoutObject-derived headers where not needed.

Or, if a LayoutObject-derived header is still required, pick the most generic
one possible.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Committed: https://crrev.com/8990614c565f4158ee1e0109fb48f1575760b877
Cr-Commit-Position: refs/heads/master@{#430165}

[mstensho (USE GERRIT), 2016-11-02 09:11:30]

Description was changed from

==========
Don't include LayoutObject-derived headers where not needed.

Or, if a LayoutObject-derived header is still required, pick the most generic
one possible.
==========

to

==========
Don't include LayoutObject-derived headers where not needed.

Or, if a LayoutObject-derived header is still required, pick the most generic
one possible.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[mstensho (USE GERRIT), 2016-11-02 09:17:41]

mstensho@opera.com changed reviewers:
+ aboxhall@chromium.org, wangxianzhu@chromium.org

[mstensho (USE GERRIT), 2016-11-02 09:17:42]

aboxhall for modules/accessibility/
wangxianzhu for the rest :)

This is the third and final CL in a series of CLs that in total will eliminate
the need for LayoutObject in 24 compilation units, and LayoutBox in 5
compilation units. No huge gain, in other words, but still worthwhile to shave
off some seconds of recompilation time when changing those headers.

[mstensho (USE GERRIT), 2016-11-02 09:49:17]

On 2016/11/02 09:17:42, mstensho wrote:
> aboxhall for modules/accessibility/
> wangxianzhu for the rest :)
> 
> This is the third and final CL in a series of CLs that in total will eliminate
> the need for LayoutObject in 24 compilation units, and LayoutBox in 5
> compilation units. No huge gain, in other words, but still worthwhile to shave
> off some seconds of recompilation time when changing those headers.

Forgot to link to the two other CLs:
https://codereview.chromium.org/2468073002/
https://codereview.chromium.org/2472573002/

[Xianzhu, 2016-11-02 15:37:27]

lgtm

[mstensho (USE GERRIT), 2016-11-04 20:29:39]

@aboxhall ping

[dmazzoni, 2016-11-04 22:22:29]

dmazzoni@chromium.org changed reviewers:
+ dmazzoni@chromium.org

[dmazzoni, 2016-11-04 22:22:30]

lgtm for modules/accessibility

In the future please feel free to TBR accessibility
owners for refactoring changes as long as some
Blink core owner has already approved

[mstensho (USE GERRIT), 2016-11-05 12:15:28]

The CQ bit was checked by mstensho@opera.com

[commit-bot: I haz the power, 2016-11-05 12:15:33]

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-11-05 13:52:37]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-05 13:55:12]

Description was changed from

==========
Don't include LayoutObject-derived headers where not needed.

Or, if a LayoutObject-derived header is still required, pick the most generic
one possible.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
Don't include LayoutObject-derived headers where not needed.

Or, if a LayoutObject-derived header is still required, pick the most generic
one possible.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Committed: https://crrev.com/8990614c565f4158ee1e0109fb48f1575760b877
Cr-Commit-Position: refs/heads/master@{#430165}
==========

[commit-bot: I haz the power, 2016-11-05 13:55:14]

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