blink: Cleanup core class forward declarations

blink: Cleanup core class forward declarations

Remove useless class forward declarations and add missing
ones in blink core sources.

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

Committed: https://crrev.com/9c8325596dae53105c1e1428d6843c305bbb2d9f
Cr-Commit-Position: refs/heads/master@{#428780}

[jbriance, 2016-10-31 16:20:37]

Description was changed from

==========
blink: Cleanup core class forward declarations

Remove useless class forward declarations and add missing
ones in blink core sources.

BUG=none
==========

to

==========
blink: Cleanup core class forward declarations

Remove useless class forward declarations and add missing
ones in blink core sources.

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

[jbriance, 2016-10-31 16:24:34]

jbriance@cisco.com changed reviewers:
+ haraken@chromium.org

[jbriance, 2016-10-31 16:24:35]

Following https://codereview.chromium.org/2460243002/, I found lots of forward
class decl that were not up-to-date in core sources of blink.

As I don't know the appropriate reviewers to target here, please feel free to
add other reviewers

[haraken, 2016-10-31 16:55:38]

LGTM

How are you finding these unused forward decls?

https://codereview.chromium.org/2463703003/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/animation/SampledEffect.h (right):

https://codereview.chromium.org/2463703003/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/animation/SampledEffect.h:16: 

Remove the redundant empty line. The same comment for other files in this CL.

[jbriance, 2016-10-31 17:39:45]

On 2016/10/31 16:55:38, haraken wrote:
> How are you finding these unused forward decls?

I made a quick script to check "class xxx;" lines in header files, and remove
the line if no other occurence of xxx is found in the file.

> Remove the redundant empty line. The same comment for other files in this CL.

Done in patch set 2

[jbriance, 2016-10-31 17:40:08]

The CQ bit was checked by jbriance@cisco.com to run a CQ dry run

[commit-bot: I haz the power, 2016-10-31 17:40:39]

Dry run: 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-10-31 19:12:20]

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

[commit-bot: I haz the power, 2016-10-31 19:12:21]

Dry run: This issue passed the CQ dry run.

[jbriance, 2016-10-31 19:58:01]

The CQ bit was checked by jbriance@cisco.com

[jbriance, 2016-10-31 19:58:02]

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/2463703003/#ps20001
(title: "Remove redundant empty lines")

[commit-bot: I haz the power, 2016-10-31 19:58:34]

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-10-31 20:09:43]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-31 20:11:34]

Description was changed from

==========
blink: Cleanup core class forward declarations

Remove useless class forward declarations and add missing
ones in blink core sources.

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

to

==========
blink: Cleanup core class forward declarations

Remove useless class forward declarations and add missing
ones in blink core sources.

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

Committed: https://crrev.com/9c8325596dae53105c1e1428d6843c305bbb2d9f
Cr-Commit-Position: refs/heads/master@{#428780}
==========

[commit-bot: I haz the power, 2016-10-31 20:11:36]

Patchset 2 (id:??) landed as
https://crrev.com/9c8325596dae53105c1e1428d6843c305bbb2d9f
Cr-Commit-Position: refs/heads/master@{#428780}

[jbriance, 2016-11-03 22:15:54]

Meta bug created: https://bugs.chromium.org/p/chromium/issues/detail?id=662195