Don't include LayoutObject.h from FrameView.h

Don't include LayoutObject.h from FrameView.h

This reduces the LayoutObject.h dependency by about 50 compilation units.

Needed to move data of DepthOrderedLayoutObjectList into a forward-declared
struct, since HashSet members need their T defined, even if it's a pointer.
This due to oilpan. See https://codereview.chromium.org/1999343002 for details.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Committed: https://crrev.com/ed98e051cdbac8bec50e76ff68394fb39404f5eb
Cr-Commit-Position: refs/heads/master@{#419728}

[mstensho (USE GERRIT), 2016-09-16 20:20:04]

Description was changed from

==========
Don't include LayoutObject.h from FrameView.h

This reduces the LayoutObject.h dependency by about 50 compilation units.

Needed to move data of DepthOrderedLayoutObjectList into a forward-declared
struct, since HashSet members need their T defined, even if it's a pointer.
This due to oilpan. See https://codereview.chromium.org/1999343002 for details.
==========

to

==========
Don't include LayoutObject.h from FrameView.h

This reduces the LayoutObject.h dependency by about 50 compilation units.

Needed to move data of DepthOrderedLayoutObjectList into a forward-declared
struct, since HashSet members need their T defined, even if it's a pointer.
This due to oilpan. See https://codereview.chromium.org/1999343002 for details.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[mstensho (USE GERRIT), 2016-09-16 20:20:14]

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

[commit-bot: I haz the power, 2016-09-16 20:20:36]

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

[mstensho (USE GERRIT), 2016-09-16 20:20:48]

Description was changed from

==========
Don't include LayoutObject.h from FrameView.h

This reduces the LayoutObject.h dependency by about 50 compilation units.

Needed to move data of DepthOrderedLayoutObjectList into a forward-declared
struct, since HashSet members need their T defined, even if it's a pointer.
This due to oilpan. See https://codereview.chromium.org/1999343002 for details.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
Don't include LayoutObject.h from FrameView.h

This reduces the LayoutObject.h dependency by about 50 compilation units.

Needed to move data of DepthOrderedLayoutObjectList into a forward-declared
struct, since HashSet members need their T defined, even if it's a pointer.
This due to oilpan. See https://codereview.chromium.org/1999343002 for details.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[mstensho (USE GERRIT), 2016-09-16 20:21:23]

mstensho@opera.com changed reviewers:
+ eae@chromium.org, mkwst@chromium.org

[eae, 2016-09-16 20:24:22]

RS LGTM

[pdr., 2016-09-16 20:38:27]

Could Opera setup a bot to help track these sorts of issues for non-goma'ers,
such as a non-goma incremental compile time bot? Most folks are blissfully
unaware of these sorts of issues but I bet they would be happy to fix them if
bugs were autofiled, or something.

[commit-bot: I haz the power, 2016-09-16 21:38:15]

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

[commit-bot: I haz the power, 2016-09-16 21:38:16]

Dry run: This issue passed the CQ dry run.

[mstensho (USE GERRIT), 2016-09-20 07:12:56]

mstensho@opera.com changed reviewers:
+ aelias@chromium.org

[mstensho (USE GERRIT), 2016-09-20 07:17:15]

mstensho@opera.com changed reviewers:
+ bokan@chromium.org, haraken@chromium.org, pdr@chromium.org

[mstensho (USE GERRIT), 2016-09-20 07:17:16]

Adding a bunch of reviewers for Source/web/ (since the one I initially picked
went OOO) - landing this CL is blocked on a tiny change there.

[haraken, 2016-09-20 07:29:24]

LGTM

[mstensho (USE GERRIT), 2016-09-20 10:00:05]

The CQ bit was checked by mstensho@opera.com

[commit-bot: I haz the power, 2016-09-20 10:00:17]

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-09-20 12:29:24]

Description was changed from

==========
Don't include LayoutObject.h from FrameView.h

This reduces the LayoutObject.h dependency by about 50 compilation units.

Needed to move data of DepthOrderedLayoutObjectList into a forward-declared
struct, since HashSet members need their T defined, even if it's a pointer.
This due to oilpan. See https://codereview.chromium.org/1999343002 for details.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
Don't include LayoutObject.h from FrameView.h

This reduces the LayoutObject.h dependency by about 50 compilation units.

Needed to move data of DepthOrderedLayoutObjectList into a forward-declared
struct, since HashSet members need their T defined, even if it's a pointer.
This due to oilpan. See https://codereview.chromium.org/1999343002 for details.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[commit-bot: I haz the power, 2016-09-20 12:29:26]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-20 12:31:59]

Description was changed from

==========
Don't include LayoutObject.h from FrameView.h

This reduces the LayoutObject.h dependency by about 50 compilation units.

Needed to move data of DepthOrderedLayoutObjectList into a forward-declared
struct, since HashSet members need their T defined, even if it's a pointer.
This due to oilpan. See https://codereview.chromium.org/1999343002 for details.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
Don't include LayoutObject.h from FrameView.h

This reduces the LayoutObject.h dependency by about 50 compilation units.

Needed to move data of DepthOrderedLayoutObjectList into a forward-declared
struct, since HashSet members need their T defined, even if it's a pointer.
This due to oilpan. See https://codereview.chromium.org/1999343002 for details.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

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

[commit-bot: I haz the power, 2016-09-20 12:32:01]

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