compositor-worker: Add CompositorProxyClient worker client of CompositorWorker.

third_party/WebKit/Source/web/ChromeClientImpl.cpp

 /*
  * Copyright (C) 2009 Google Inc. All rights reserved.
  * Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies).
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
@@ skipping 1055 matching lines @@
 {
     if (m_webView->rootGraphicsLayer() && m_webView->layerTreeView())
         m_webView->page()->frameHost().visualViewport().registerLayersWithTreeView(m_webView->layerTreeView());
 }
 
 void ChromeClientImpl::didUpdateTopControls() const
 {
     m_webView->didUpdateTopControls();
 }
 
+CompositorProxyClient* ChromeClientImpl::createCompositorProxyClient()
+{
+    return m_webView->createCompositorProxyClient();

[dcheng, 2016/05/12 16:56:15]

Why does this have to go through WebView? Why can't this just new
CompositorProxyClient directly?

[flackr, 2016/05/12 17:00:51]

In this patch it absolutely could - I can move it there for this patch. In
https://codereview.chromium.org/1895873006 when we hook it up to the
CompositorMutatorImpl we need WebViewImpl to pass the constructed
CompositorMutatorImpl through WebLayerTreeView to the LayerTreeHostImpl.

[dcheng, 2016/05/12 17:09:29]

I don't think that patch will work well with OOPI: the WebView won't have a
WebLayerTreeView if the main frame is a remote frame. Maybe it should be plumbed
through WebFrameWidget instead (we'll likely need to special case the main frame
case to fall back to WebViewImpl for now, since we're still in the process of
factoring out the WebWidget portions of WebView).

+}
+
 void ChromeClientImpl::registerPopupOpeningObserver(PopupOpeningObserver* observer)
 {
     DCHECK(observer);
     m_popupOpeningObservers.append(observer);
 }
 
 void ChromeClientImpl::unregisterPopupOpeningObserver(PopupOpeningObserver* observer)
 {
     size_t index = m_popupOpeningObservers.find(observer);
     DCHECK_NE(index, kNotFound);
@@ skipping 22 matching lines @@
 {
     return adoptPtr(m_webView->scheduler()->createFrameScheduler(blameContext).release());
 }
 
 double ChromeClientImpl::lastFrameTimeMonotonic() const
 {
     return m_webView->lastFrameTimeMonotonic();
 }
 
 } // namespace blink