Clean forward declaration in public

Clean forward declaration in public

These forward references are not needed. After this
patch, public/ dirctory has no unneeded forward references anymore.

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=186556

[zhaoze.zhou, 2014-12-02 23:26:38]

zhaoze.zhou@partner.samsung.com changed reviewers:
+ adamk@chromium.org

[zhaoze.zhou, 2014-12-02 23:28:37]

On 2014/12/02 23:26:38, zhaoze.zhou wrote:
> mailto:zhaoze.zhou@partner.samsung.com changed reviewers:
> + mailto:adamk@chromium.org

Hi, there's several forward declaration that only used in
chromium side, https://codereview.chromium.org/772183003/ .
I am not sure to keep these forward declarations in blink side,
or move them to the correct chromium-side file. 

Thanks. :)

[adamk, 2014-12-02 23:29:28]

I'm not an OWNER in public/, so I won't be much help for this patch.

[zhaoze.zhou, 2014-12-02 23:36:23]

zhaoze.zhou@partner.samsung.com changed reviewers:
+ mkwst@chromium.org, rob.buis@samsung.com, tkent@chromium.org

[zhaoze.zhou, 2014-12-02 23:36:43]

On 2014/12/02 23:29:28, adamk wrote:
> I'm not an OWNER in public/, so I won't be much help for this patch.

OK.. Also thanks...

[tkent, 2014-12-03 00:04:44]

The CQ bit was checked by tkent@chromium.org

[tkent, 2014-12-03 00:04:45]

lgtm

[commit-bot: I haz the power, 2014-12-03 00:05:37]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/776693002/20001

[commit-bot: I haz the power, 2014-12-03 00:16:01]

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

[commit-bot: I haz the power, 2014-12-03 00:16:02]

Try jobs failed on following builders:
  linux_chromium_gn_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_chromium_gn_rel/bu...)
  mac_blink_compile_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/mac_blink_compile_dbg/bu...)
  mac_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/mac_blink_rel/builds/33513)

[zhaoze.zhou, 2014-12-03 14:43:35]

On 2014/12/03 00:04:45, tkent wrote:
> lgtm

Sorry that didn't make it clear to you. 
There's several forward declaration that only used in
chromium side, https://codereview.chromium.org/772183003/ .

I am not sure to keep these forward declarations in blink side,
or move them to the correct chromium-side files.

Keep these declarations in blink make life easier, but it's more logically
to put the declarations to where really need it. 

Thanks.

[tkent, 2014-12-03 23:41:43]

I think this CL does the right thing.  Chromium code should have forward class
declarations or #includes.

[zhaoze.zhou, 2014-12-03 23:53:56]

On 2014/12/03 23:41:43, tkent wrote:
> I think this CL does the right thing.  Chromium code should have forward class
> declarations or #includes.

thanks. so we need to make the chromium patch landed first. :)

[zhaoze.zhou, 2014-12-04 22:12:08]

Patchset #4 (id:60001) has been deleted

[zhaoze.zhou, 2014-12-04 22:42:41]

Patchset #4 (id:80001) has been deleted

[zhaoze.zhou, 2014-12-04 23:28:30]

On 2014/12/03 23:53:56, zhaoze.zhou wrote:
> On 2014/12/03 23:41:43, tkent wrote:
> > I think this CL does the right thing.  Chromium code should have forward
class
> > declarations or #includes.
> 
> thanks. so we need to make the chromium patch landed first. :)

chromium-side patch landed. will commit this patch after all try-bots
successfully
build and test. :)

[zhaoze.zhou, 2014-12-04 23:36:47]

The CQ bit was checked by zhaoze.zhou@partner.samsung.com

[commit-bot: I haz the power, 2014-12-04 23:37:35]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/776693002/120001

[commit-bot: I haz the power, 2014-12-05 00:22:23]

Committed patchset #5 (id:120001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=186556