cc: Cleanup class/struct forward declarations

cc: Cleanup class/struct forward declarations

Remove useless class and struct forward declarations, and add
missing ones in cc header files.

BUG=662195
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Committed: https://crrev.com/58f287687a11f1b9f35371aa708e8b93269b3974
Cr-Commit-Position: refs/heads/master@{#432964}

[jbriance, 2016-11-16 08:09:35]

Description was changed from

==========
cc: Cleanup class/struct forward declarations

Remove useless class and struct forward declarations, and add
missing ones in cc header files.

BUG=662195
==========

to

==========
cc: Cleanup class/struct forward declarations

Remove useless class and struct forward declarations, and add
missing ones in cc header files.

BUG=662195
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[jbriance, 2016-11-16 08:12:32]

jbriance@cisco.com changed reviewers:
+ ccameron@chromium.org, danakj@chromium.org

[ccameron, 2016-11-16 21:41:54]

lgtm

[danakj, 2016-11-16 21:44:13]

LGTM

[jbriance, 2016-11-16 21:48:24]

The CQ bit was checked by jbriance@cisco.com

[commit-bot: I haz the power, 2016-11-16 21:49:07]

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-16 22:23:16]

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

[commit-bot: I haz the power, 2016-11-16 22:23:17]

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

[jbriance, 2016-11-16 22:40:09]

jbriance@cisco.com changed reviewers:
+ dcheng@chromium.org, nyquist@chromium.org

[jbriance, 2016-11-16 22:40:10]

+ nyquist for blimp/client/support/compositor/blimp_embedder_compositor.h
+ dcheng for cc/ipc/cc_param_traits.h

[dcheng, 2016-11-16 22:40:50]

ipc rs lgtm

[jbriance, 2016-11-16 22:42:02]

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

[commit-bot: I haz the power, 2016-11-16 22:42:35]

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-11-17 00:45:35]

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

[commit-bot: I haz the power, 2016-11-17 00:45:36]

Dry run: Try jobs failed on following builders:
  linux_trusty_blink_rel on master.tryserver.blink (JOB_TIMED_OUT, no build URL)

[jbriance, 2016-11-17 08:00:08]

The CQ bit was checked by jbriance@cisco.com

[jbriance, 2016-11-17 08:00:09]

The patchset sent to the CQ was uploaded after l-g-t-m from danakj@chromium.org,
ccameron@chromium.org
Link to the patchset: https://codereview.chromium.org/2503283002/#ps20001
(title: "Add missing fwd class decl in blimp_embedder_compositor.h")

[commit-bot: I haz the power, 2016-11-17 08:00:43]

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-17 08:09:53]

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

[commit-bot: I haz the power, 2016-11-17 08:09:54]

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

[jbriance, 2016-11-17 08:17:30]

jbriance@cisco.com changed reviewers:
+ dtrainor@chromium.org, kmarshall@chromium.org, wez@chromium.org

[jbriance, 2016-11-17 08:17:31]

+ dtrainor, kmarshall and wez to be able to submit:

  ** Presubmit ERRORS **
  Missing LGTM from an OWNER for these files:
    blimp/client/support/compositor/blimp_embedder_compositor.h

[Wez, 2016-11-17 19:00:37]

blimp/ LGTM

[jbriance, 2016-11-17 20:52:35]

The CQ bit was checked by jbriance@cisco.com

[commit-bot: I haz the power, 2016-11-17 20:53:31]

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-17 21:38:19]

Description was changed from

==========
cc: Cleanup class/struct forward declarations

Remove useless class and struct forward declarations, and add
missing ones in cc header files.

BUG=662195
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
cc: Cleanup class/struct forward declarations

Remove useless class and struct forward declarations, and add
missing ones in cc header files.

BUG=662195
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Committed: https://crrev.com/58f287687a11f1b9f35371aa708e8b93269b3974
Cr-Commit-Position: refs/heads/master@{#432964}
==========

[commit-bot: I haz the power, 2016-11-17 21:38:21]

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