[chromium] Remove WebLayerTreeViewClient and WebLayerTreeView::Settings
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 8 Mar 2013 21:02:00 +0000 (21:02 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 8 Mar 2013 21:02:00 +0000 (21:02 +0000)
https://bugs.webkit.org/show_bug.cgi?id=111632

Patch by James Robinson <jamesr@chromium.org> on 2013-03-08
Reviewed by Adrienne Walker.

Source/Platform:

Neither of these are used any more. All callbacks from the compositor are handled by
the embedder and plumbed (where appropriate) through the WebWidget/WebView APIs. All
settings are handled by the embedder.

* Platform.gypi:
* chromium/public/WebLayerTreeView.h:
(WebKit):
* chromium/public/WebLayerTreeViewClient.h: Removed.
* chromium/public/WebUnitTestSupport.h:
(WebKit):

Source/WebKit/chromium:

* public/WebWidgetClient.h:
(WebWidgetClient):
(WebKit::WebWidgetClient::initializeLayerTreeView):
* src/WebViewImpl.cpp:
(WebKit::WebViewImpl::setIsAcceleratedCompositingActive):
* src/WebViewImpl.h:
* tests/ScrollingCoordinatorChromiumTest.cpp:
(WebKit::FakeWebViewClient::initializeLayerTreeView):

Tools:

* DumpRenderTree/chromium/WebViewHost.cpp:
(WebViewHost::initializeLayerTreeView):
* DumpRenderTree/chromium/WebViewHost.h:
(WebViewHost):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@145258 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 files changed:
Source/Platform/ChangeLog
Source/Platform/Platform.gypi
Source/Platform/chromium/public/WebLayerTreeView.h
Source/Platform/chromium/public/WebLayerTreeViewClient.h [deleted file]
Source/Platform/chromium/public/WebUnitTestSupport.h
Source/WebKit/chromium/ChangeLog
Source/WebKit/chromium/public/WebWidgetClient.h
Source/WebKit/chromium/src/WebViewImpl.cpp
Source/WebKit/chromium/src/WebViewImpl.h
Source/WebKit/chromium/tests/ScrollingCoordinatorChromiumTest.cpp
Tools/ChangeLog
Tools/DumpRenderTree/chromium/WebViewHost.cpp
Tools/DumpRenderTree/chromium/WebViewHost.h

index a541a3a..f0100a5 100644 (file)
@@ -1,3 +1,21 @@
+2013-03-08  James Robinson  <jamesr@chromium.org>
+
+        [chromium] Remove WebLayerTreeViewClient and WebLayerTreeView::Settings
+        https://bugs.webkit.org/show_bug.cgi?id=111632
+
+        Reviewed by Adrienne Walker.
+
+        Neither of these are used any more. All callbacks from the compositor are handled by
+        the embedder and plumbed (where appropriate) through the WebWidget/WebView APIs. All
+        settings are handled by the embedder.
+
+        * Platform.gypi:
+        * chromium/public/WebLayerTreeView.h:
+        (WebKit):
+        * chromium/public/WebLayerTreeViewClient.h: Removed.
+        * chromium/public/WebUnitTestSupport.h:
+        (WebKit):
+
 2013-03-06  James Robinson  <jamesr@chromium.org>
 
         [chromium] Stop using WebTransformationMatrix on WebLayer
index ed2479b..2b8f265 100644 (file)
@@ -82,7 +82,6 @@
             'chromium/public/WebLayer.h',
             'chromium/public/WebLayerScrollClient.h',
             'chromium/public/WebLayerTreeView.h',
-            'chromium/public/WebLayerTreeViewClient.h',
             'chromium/public/WebLocalizedString.h',
             'chromium/public/WebMediaConstraints.h',
             'chromium/public/WebMediaStreamCenter.h',
index e5fc0eb..b15d4dd 100644 (file)
 namespace WebKit {
 class WebGraphicsContext3D;
 class WebLayer;
-class WebLayerTreeViewClient;
 struct WebPoint;
 struct WebRect;
 struct WebRenderingStats;
 
 class WebLayerTreeView {
 public:
-    struct Settings {
-        Settings()
-            : acceleratePainting(false)
-            , showDebugBorders(false)
-            , showFPSCounter(false)
-            , showPlatformLayerTree(false)
-            , showPaintRects(false)
-            , renderVSyncEnabled(true)
-            , renderVSyncNotificationEnabled(false)
-            , perTilePaintingEnabled(false)
-            , partialSwapEnabled(false)
-            , acceleratedAnimationEnabled(true)
-            , pageScalePinchZoomEnabled(false)
-            , recordRenderingStats(false)
-            , refreshRate(0)
-            , defaultTileSize(WebSize(256, 256))
-            , maxUntiledLayerSize(WebSize(512, 512))
-        {
-        }
-
-        bool acceleratePainting;
-        bool showDebugBorders;
-        bool showFPSCounter;
-        bool showPlatformLayerTree;
-        bool showPaintRects;
-        bool renderVSyncEnabled;
-        bool renderVSyncNotificationEnabled;
-        bool perTilePaintingEnabled;
-        bool partialSwapEnabled;
-        bool acceleratedAnimationEnabled;
-        bool pageScalePinchZoomEnabled;
-        bool recordRenderingStats;
-        double refreshRate;
-        WebSize defaultTileSize;
-        WebSize maxUntiledLayerSize;
-    };
-
     virtual ~WebLayerTreeView() { }
 
     // Initialization and lifecycle --------------------------------------
@@ -182,6 +144,9 @@ public:
 
     // FIXME: Remove this.
     virtual void loseCompositorContext(int numTimes) { }
+
+    // DEPRECATED.
+    struct Settings { };
 };
 
 } // namespace WebKit
diff --git a/Source/Platform/chromium/public/WebLayerTreeViewClient.h b/Source/Platform/chromium/public/WebLayerTreeViewClient.h
deleted file mode 100644 (file)
index e4b94b0..0000000
+++ /dev/null
@@ -1,52 +0,0 @@
-/*
- * Copyright (C) 2011 Google Inc. All rights reserved.
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions
- * are met:
- *
- * 1.  Redistributions of source code must retain the above copyright
- *     notice, this list of conditions and the following disclaimer.
- * 2.  Redistributions in binary form must reproduce the above copyright
- *     notice, this list of conditions and the following disclaimer in the
- *     documentation and/or other materials provided with the distribution.
- *
- * THIS SOFTWARE IS PROVIDED BY APPLE AND ITS CONTRIBUTORS "AS IS" AND ANY
- * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED
- * WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
- * DISCLAIMED. IN NO EVENT SHALL APPLE OR ITS CONTRIBUTORS BE LIABLE FOR ANY
- * DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES
- * (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES;
- * LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND
- * ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
- * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
- * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
- */
-
-#ifndef WebLayerTreeViewClient_h
-#define WebLayerTreeViewClient_h
-
-class SkBitmap;
-
-namespace WebKit {
-class WebCompositorOutputSurface;
-class WebInputHandler;
-class WebThread;
-struct WebRect;
-struct WebSize;
-
-class WebLayerTreeViewClient {
-public:
-    // DEPRECATED - WILL BE REMOVED SOON
-    virtual void applyScrollAndScale(const WebSize& scrollDelta, float scaleFactor) { }
-    virtual void updateAnimations(double frameBeginTime) { }
-    virtual void didRecreateOutputSurface(bool success) { }
-    virtual WebInputHandler* createInputHandler() { return 0; }
-
-protected:
-    virtual ~WebLayerTreeViewClient() { }
-};
-
-} // namespace WebKit
-
-#endif // WebLayerTreeViewClient_h
index 5698ed3..43dd0fe 100644 (file)
@@ -32,7 +32,6 @@
 namespace WebKit {
 
 class WebLayerTreeView;
-class WebLayerTreeViewClient;
 class WebURL;
 class WebURLResponse;
 struct WebURLError;
index 96917b3..e583770 100644 (file)
@@ -1,3 +1,19 @@
+2013-03-08  James Robinson  <jamesr@chromium.org>
+
+        [chromium] Remove WebLayerTreeViewClient and WebLayerTreeView::Settings
+        https://bugs.webkit.org/show_bug.cgi?id=111632
+
+        Reviewed by Adrienne Walker.
+
+        * public/WebWidgetClient.h:
+        (WebWidgetClient):
+        (WebKit::WebWidgetClient::initializeLayerTreeView):
+        * src/WebViewImpl.cpp:
+        (WebKit::WebViewImpl::setIsAcceleratedCompositingActive):
+        * src/WebViewImpl.h:
+        * tests/ScrollingCoordinatorChromiumTest.cpp:
+        (WebKit::FakeWebViewClient::initializeLayerTreeView):
+
 2013-03-08  Sheriff Bot  <webkit.review.bot@gmail.com>
 
         Unreviewed.  Rolled Chromium DEPS to r186991.  Requested by
index bf59354..b68c5ec 100644 (file)
@@ -66,10 +66,9 @@ public:
     virtual void didActivateCompositor(int inputHandlerIdentifier) { }
     virtual void didDeactivateCompositor() { }
 
-    // Attempt to initialize compositing for this widget using the given
-    // parameters. If this is successful, layerTreeView() will return a valid
-    // WebLayerTreeView. If not, nothing happens.
-    virtual void initializeLayerTreeView(WebLayerTreeViewClient*, const WebLayer& rootLayer, const WebLayerTreeView::Settings&) { }
+    // Attempt to initialize compositing for this widget. If this is successful,
+    // layerTreeView() will return a valid WebLayerTreeView.
+    virtual void initializeLayerTreeView() { }
 
     // Return a compositing view used for this widget. This is owned by the
     // WebWidgetClient.
index f65c59c..6fc9c40 100644 (file)
@@ -1845,11 +1845,6 @@ void WebViewImpl::animate(double monotonicFrameBeginTime)
     }
 }
 
-void WebViewImpl::updateAnimations(double monotonicFrameBeginTime)
-{
-    animate(monotonicFrameBeginTime);
-}
-
 void WebViewImpl::layout()
 {
     TRACE_EVENT0("webkit", "WebViewImpl::layout");
@@ -4126,29 +4121,14 @@ void WebViewImpl::setIsAcceleratedCompositingActive(bool active)
     } else {
         TRACE_EVENT0("webkit", "WebViewImpl::setIsAcceleratedCompositingActive(true)");
 
-        WebLayerTreeView::Settings layerTreeViewSettings;
-        layerTreeViewSettings.acceleratePainting = page()->settings()->acceleratedDrawingEnabled();
-        layerTreeViewSettings.showDebugBorders = page()->settings()->showDebugBorders();
-        layerTreeViewSettings.showFPSCounter = settingsImpl()->showFPSCounter();
-        layerTreeViewSettings.showPlatformLayerTree = settingsImpl()->showPlatformLayerTree();
-        layerTreeViewSettings.showPaintRects = settingsImpl()->showPaintRects();
-        layerTreeViewSettings.renderVSyncEnabled = settingsImpl()->renderVSyncEnabled();
-        layerTreeViewSettings.renderVSyncNotificationEnabled = settingsImpl()->renderVSyncNotificationEnabled();
-        layerTreeViewSettings.perTilePaintingEnabled = settingsImpl()->perTilePaintingEnabled();
-        layerTreeViewSettings.acceleratedAnimationEnabled = settingsImpl()->acceleratedAnimationEnabled();
-        layerTreeViewSettings.pageScalePinchZoomEnabled = settingsImpl()->applyPageScaleFactorInCompositor();
-        layerTreeViewSettings.recordRenderingStats = settingsImpl()->recordRenderingStats();
-
-        layerTreeViewSettings.defaultTileSize = settingsImpl()->defaultTileSize();
-        layerTreeViewSettings.maxUntiledLayerSize = settingsImpl()->maxUntiledLayerSize();
-
         m_nonCompositedContentHost = NonCompositedContentHost::create(this, graphicsLayerFactory());
         m_nonCompositedContentHost->setShowDebugBorders(page()->settings()->showDebugBorders());
         m_nonCompositedContentHost->setOpaque(!isTransparent());
 
-        m_client->initializeLayerTreeView(this, *m_rootLayer, layerTreeViewSettings);
+        m_client->initializeLayerTreeView();
         m_layerTreeView = m_client->layerTreeView();
         if (m_layerTreeView) {
+            m_layerTreeView->setRootLayer(*m_rootLayer);
             if (m_webSettings->applyDeviceScaleFactorInCompositor() && page()->deviceScaleFactor() != 1)
                 setDeviceScaleFactor(page()->deviceScaleFactor());
 
@@ -4215,12 +4195,6 @@ void WebViewImpl::applyScrollAndScale(const WebSize& scrollDelta, float pageScal
     }
 }
 
-void WebViewImpl::didRecreateOutputSurface(bool success)
-{
-    if (!success)
-        didExitCompositingMode();
-}
-
 void WebViewImpl::didExitCompositingMode()
 {
     ASSERT(m_isAcceleratedCompositingActive);
index de48851..2ab8b7b 100644 (file)
@@ -53,7 +53,6 @@
 #include <public/WebFloatQuad.h>
 #include <public/WebGestureCurveTarget.h>
 #include <public/WebLayer.h>
-#include <public/WebLayerTreeViewClient.h>
 #include <public/WebPoint.h>
 #include <public/WebRect.h>
 #include <public/WebSize.h>
@@ -120,7 +119,6 @@ class WebTouchEvent;
 class WebViewBenchmarkSupport;
 
 class WebViewImpl : public WebView
-    , public WebLayerTreeViewClient
     , public RefCounted<WebViewImpl>
     , public WebGestureCurveTarget
 #if ENABLE(PAGE_POPUP)
@@ -317,10 +315,6 @@ public:
     virtual void setShowFPSCounter(bool);
     virtual void setContinuousPaintingEnabled(bool);
 
-    // WebLayerTreeViewClient
-    virtual void updateAnimations(double monotonicFrameBeginTime);
-    virtual void didRecreateOutputSurface(bool success);
-
     // WebViewImpl
 
     void suppressInvalidations(bool enable);
index efad780..b59cba1 100644 (file)
@@ -51,11 +51,10 @@ namespace {
 
 class FakeWebViewClient : public WebViewClient {
 public:
-    virtual void initializeLayerTreeView(WebLayerTreeViewClient* client, const WebLayer& rootLayer, const WebLayerTreeView::Settings& settings)
+    virtual void initializeLayerTreeView()
     {
         m_layerTreeView = adoptPtr(Platform::current()->unitTestSupport()->createLayerTreeViewForTesting(WebUnitTestSupport::TestViewTypeUnitTest));
         ASSERT(m_layerTreeView);
-        m_layerTreeView->setRootLayer(rootLayer);
     }
 
     virtual WebLayerTreeView* layerTreeView()
index 067b8a2..1379ff7 100644 (file)
@@ -1,3 +1,15 @@
+2013-03-08  James Robinson  <jamesr@chromium.org>
+
+        [chromium] Remove WebLayerTreeViewClient and WebLayerTreeView::Settings
+        https://bugs.webkit.org/show_bug.cgi?id=111632
+
+        Reviewed by Adrienne Walker.
+
+        * DumpRenderTree/chromium/WebViewHost.cpp:
+        (WebViewHost::initializeLayerTreeView):
+        * DumpRenderTree/chromium/WebViewHost.h:
+        (WebViewHost):
+
 2013-03-08  Benjamin Poulain  <benjamin@webkit.org>
 
         [Mac] Add a feature flag for 'view-mode' Media Feature, disable it on Mac
index 6dd11a9..ff27053 100644 (file)
@@ -276,7 +276,7 @@ private:
     WebViewHost* m_host;
 };
 
-void WebViewHost::initializeLayerTreeView(WebLayerTreeViewClient* client, const WebLayer& rootLayer, const WebLayerTreeView::Settings& settings)
+void WebViewHost::initializeLayerTreeView()
 {
     m_layerTreeViewClient = adoptPtr(new WebViewHostDRTLayerTreeViewClient(this));
     if (m_shell->softwareCompositingEnabled()) {
@@ -293,7 +293,6 @@ void WebViewHost::initializeLayerTreeView(WebLayerTreeViewClient* client, const
 
     ASSERT(m_layerTreeView);
     updateViewportSize();
-    m_layerTreeView->setRootLayer(rootLayer);
     m_layerTreeView->setSurfaceReady();
 }
 
index ff2eda7..fdbed2f 100644 (file)
@@ -157,7 +157,7 @@ class WebViewHost : public WebKit::WebViewClient, public WebKit::WebFrameClient,
 
     // WebKit::WebWidgetClient
     virtual void didAutoResize(const WebKit::WebSize& newSize);
-    virtual void initializeLayerTreeView(WebKit::WebLayerTreeViewClient*, const WebKit::WebLayer& rootLayer, const WebKit::WebLayerTreeView::Settings&);
+    virtual void initializeLayerTreeView();
     virtual WebKit::WebLayerTreeView* layerTreeView();
     virtual void scheduleAnimation();
     virtual void didFocus();