Get rid of WebLayerTreeViewClient references in WebLayerTreeViewImplForTesting

Get rid of WebLayerTreeViewClient references in WebLayerTreeViewImplForTesting

Follow-up to r185341 / WebKit r144398

The only client this class needs is the DRTLayerTreeViewClient.

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=185662

[jamesr, 2013-02-28 22:21:43]

Depends on https://bugs.webkit.org/show_bug.cgi?id=111017 landing + rolling in

[danakj, 2013-02-28 22:38:38]

On Thu, Feb 28, 2013 at 2:21 PM, <jamesr@chromium.org> wrote:

> Reviewers: enne,
>
> Message:
> Depends on
https://bugs.webkit.org/show_**bug.cgi?id=111017<https://bugs.webkit.org/show...
+ rolling in
>
> Description:
> Get rid of WebLayerTreeViewClient references in
> WebLayerTreeViewImplForTesting
>
> Follow-up to r185341 / WebKit r?????
>

Did you mean to fill that in?


>
> The only client this class needs is the DRTLayerTreeViewClient.
>
>
> Please review this at
https://codereview.chromium.**org/12377029/<https://codereview.chromium.org/1...
>
> SVN Base:
svn://svn.chromium.org/chrome/**trunk/src<http://svn.chromium.org/chrome/trunk/src>
>
> Affected files:
>   M webkit/compositor_bindings/**compositor_bindings_tests.gyp
>   D webkit/compositor_bindings/**test/web_layer_tree_view_test_**common.h
>   M webkit/compositor_bindings/**web_layer_tree_view_impl_for_**testing.h
>   M webkit/compositor_bindings/**web_layer_tree_view_impl_for_**testing.cc
>   D webkit/compositor_bindings/**web_layer_unittest.cc
>   M webkit/support/test_webkit_**platform_support.cc
>   M webkit/support/webkit_support.**h
>   M webkit/support/webkit_support.**cc
>
>
>

[jamesr, 2013-02-28 22:43:07]

On 2013/02/28 22:38:38, danakj wrote:
> On Thu, Feb 28, 2013 at 2:21 PM, <mailto:jamesr@chromium.org> wrote:
> 
> > Reviewers: enne,
> >
> > Message:
> > Depends on
>
https://bugs.webkit.org/show_**bug.cgi?id=111017%3Chttps://bugs.webkit.org/sh...
> + rolling in
> >
> > Description:
> > Get rid of WebLayerTreeViewClient references in
> > WebLayerTreeViewImplForTesting
> >
> > Follow-up to r185341 / WebKit r?????
> >
> 
> Did you mean to fill that in?

Yes, but I can't fill it in (or land this patch) until the WebKit side lands.

[jamesr, 2013-03-01 00:52:38]

WebKit side landed (but not yet rolled): http://trac.webkit.org/changeset/144398

[enne (OOO), 2013-03-01 04:14:37]

lgtm

[commit-bot: I haz the power, 2013-03-01 21:54:22]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jamesr@chromium.org/12377029/1

[jamesr, 2013-03-02 02:19:24]

Committed patchset #1 manually as r185662 (presubmit successful).