[webkit-changes] [142569] trunk/Source

Mon, 11 Feb 2013 20:15:43 -0800

Title: [142569] trunk/Source
Revision
142569
Author
[email protected]
Date
2013-02-11 20:17:40 -0800 (Mon, 11 Feb 2013)

Log Message

Unreviewed, rolling out r142568.
http://trac.webkit.org/changeset/142568
https://bugs.webkit.org/show_bug.cgi?id=109541

Broke the build, won't compile. (Requested by alancutter on
#webkit).

Patch by Sheriff Bot <[email protected]> on 2013-02-11

Source/Platform:

* chromium/public/WebCompositorSupport.h:
(WebCompositorSupport):
(WebKit::WebCompositorSupport::createLayerTreeView):
* chromium/public/WebUnitTestSupport.h:

Source/WebKit/chromium:

* tests/GraphicsLayerChromiumTest.cpp:
(WebKit::GraphicsLayerChromiumTest::GraphicsLayerChromiumTest):
* tests/ScrollingCoordinatorChromiumTest.cpp:
(WebKit::FakeWebViewClient::initializeLayerTreeView):

Modified Paths

Diff

Modified: trunk/Source/Platform/ChangeLog (142568 => 142569)


--- trunk/Source/Platform/ChangeLog	2013-02-12 02:50:58 UTC (rev 142568)
+++ trunk/Source/Platform/ChangeLog	2013-02-12 04:17:40 UTC (rev 142569)
@@ -1,3 +1,17 @@
+2013-02-11  Sheriff Bot  <[email protected]>
+
+        Unreviewed, rolling out r142568.
+        http://trac.webkit.org/changeset/142568
+        https://bugs.webkit.org/show_bug.cgi?id=109541
+
+        Broke the build, won't compile. (Requested by alancutter on
+        #webkit).
+
+        * chromium/public/WebCompositorSupport.h:
+        (WebCompositorSupport):
+        (WebKit::WebCompositorSupport::createLayerTreeView):
+        * chromium/public/WebUnitTestSupport.h:
+
 2013-02-11  James Robinson  <[email protected]>
 
         [chromium] Add WebUnitTestSupport::createLayerTreeViewForTesting for webkit_unit_tests

Modified: trunk/Source/Platform/chromium/public/WebCompositorSupport.h (142568 => 142569)


--- trunk/Source/Platform/chromium/public/WebCompositorSupport.h	2013-02-12 02:50:58 UTC (rev 142568)
+++ trunk/Source/Platform/chromium/public/WebCompositorSupport.h	2013-02-12 04:17:40 UTC (rev 142569)
@@ -69,6 +69,9 @@
     // after shutdown.
     virtual void shutdown() { }
 
+    // May return 0 if initialization fails.
+    virtual WebLayerTreeView* createLayerTreeView(WebLayerTreeViewClient*, const WebLayer& root, const WebLayerTreeView::Settings&) { return 0; }
+
     // Creates an output surface for the compositor backed by a 3d context.
     virtual WebCompositorOutputSurface* createOutputSurfaceFor3D(WebKit::WebGraphicsContext3D*) { return 0; }
 
@@ -104,13 +107,6 @@
 
     virtual WebTransformOperations* createTransformOperations() { return 0; }
 
-
-    // Testing ------------------------------------------------------
-
-    // DEPRECATED
-    // May return 0 if initialization fails.
-    virtual WebLayerTreeView* createLayerTreeView(WebLayerTreeViewClient*, const WebLayer& root, const WebLayerTreeView::Settings&) { return 0; }
-
 protected:
     virtual ~WebCompositorSupport() { }
 };

Modified: trunk/Source/Platform/chromium/public/WebUnitTestSupport.h (142568 => 142569)


--- trunk/Source/Platform/chromium/public/WebUnitTestSupport.h	2013-02-12 02:50:58 UTC (rev 142568)
+++ trunk/Source/Platform/chromium/public/WebUnitTestSupport.h	2013-02-12 04:17:40 UTC (rev 142569)
@@ -31,7 +31,6 @@
 
 namespace WebKit {
 
-class WebLayerTreeView;
 class WebURL;
 class WebURLResponse;
 struct WebURLError;
@@ -53,18 +52,6 @@
 
     // Returns the root directory of the WebKit code.
     virtual WebString webKitRootDir() { return WebString(); }
-
-    // Constructs a WebLayerTreeView set up with reasonable defaults for
-    // testing. A LayerTreeTypeUnitTest view can initialize and perform most
-    // operations, but is not capable of rendering pixels. A
-    // LayerTreeTypeLayoutTest view can render.
-
-#define HAVE_CREATELAYERTREEVIEWFORTESTING 1
-    enum TestViewType {
-        TestViewTypeUnitTest,
-        TestViewTypeLayoutTest
-    };
-    virtual WebLayerTreeView* createLayerTreeViewForTesting(TestViewType type) { return 0; }
 };
 
 }

Modified: trunk/Source/WebKit/chromium/ChangeLog (142568 => 142569)


--- trunk/Source/WebKit/chromium/ChangeLog	2013-02-12 02:50:58 UTC (rev 142568)
+++ trunk/Source/WebKit/chromium/ChangeLog	2013-02-12 04:17:40 UTC (rev 142569)
@@ -1,3 +1,17 @@
+2013-02-11  Sheriff Bot  <[email protected]>
+
+        Unreviewed, rolling out r142568.
+        http://trac.webkit.org/changeset/142568
+        https://bugs.webkit.org/show_bug.cgi?id=109541
+
+        Broke the build, won't compile. (Requested by alancutter on
+        #webkit).
+
+        * tests/GraphicsLayerChromiumTest.cpp:
+        (WebKit::GraphicsLayerChromiumTest::GraphicsLayerChromiumTest):
+        * tests/ScrollingCoordinatorChromiumTest.cpp:
+        (WebKit::FakeWebViewClient::initializeLayerTreeView):
+
 2013-02-11  James Robinson  <[email protected]>
 
         [chromium] Add WebUnitTestSupport::createLayerTreeViewForTesting for webkit_unit_tests

Modified: trunk/Source/WebKit/chromium/tests/GraphicsLayerChromiumTest.cpp (142568 => 142569)


--- trunk/Source/WebKit/chromium/tests/GraphicsLayerChromiumTest.cpp	2013-02-12 02:50:58 UTC (rev 142568)
+++ trunk/Source/WebKit/chromium/tests/GraphicsLayerChromiumTest.cpp	2013-02-12 04:17:40 UTC (rev 142569)
@@ -35,11 +35,11 @@
 #include "WebLayerTreeViewTestCommon.h"
 #include <gtest/gtest.h>
 #include <public/Platform.h>
+#include <public/WebCompositorSupport.h>
 #include <public/WebFloatAnimationCurve.h>
 #include <public/WebGraphicsContext3D.h>
 #include <public/WebLayerTreeView.h>
 #include <public/WebTransformationMatrix.h>
-#include <public/WebUnitTestSupport.h>
 #include <wtf/PassOwnPtr.h>
 
 using namespace WebCore;
@@ -62,9 +62,7 @@
         Platform::current()->compositorSupport()->initialize(0);
         m_graphicsLayer = adoptPtr(new GraphicsLayerChromium(&m_client));
         m_platformLayer = m_graphicsLayer->platformLayer();
-        m_layerTreeView = adoptPtr(Platform::current()->unitTestSupport()->createLayerTreeViewForTesting(WebUnitTestSupport::TestViewTypeUnitTest));
-        ASSERT_TRUE(m_layerTreeView);
-        m_layerTreeView->setRootLayer(*m_platformLayer);
+        m_layerTreeView = adoptPtr(Platform::current()->compositorSupport()->createLayerTreeView(&m_layerTreeViewClient, *m_platformLayer, WebLayerTreeView::Settings()));
         m_layerTreeView->setViewportSize(WebSize(1, 1), WebSize(1, 1));
     }

Modified: trunk/Source/WebKit/chromium/tests/ScrollingCoordinatorChromiumTest.cpp (142568 => 142569)


--- trunk/Source/WebKit/chromium/tests/ScrollingCoordinatorChromiumTest.cpp	2013-02-12 02:50:58 UTC (rev 142568)
+++ trunk/Source/WebKit/chromium/tests/ScrollingCoordinatorChromiumTest.cpp	2013-02-12 04:17:40 UTC (rev 142569)
@@ -59,9 +59,8 @@
 
     virtual void initializeLayerTreeView(WebLayerTreeViewClient* client, const WebLayer& rootLayer, const WebLayerTreeView::Settings& settings)
     {
-        m_layerTreeView = adoptPtr(Platform::current()->unitTestSupport()->createLayerTreeViewForTesting(WebUnitTestSupport::TestViewTypeUnitTest));
+        m_layerTreeView = adoptPtr(Platform::current()->compositorSupport()->createLayerTreeView(client, rootLayer, settings));
         ASSERT(m_layerTreeView);
-        m_layerTreeView->setRootLayer(rootLayer);
     }
 
     virtual WebLayerTreeView* layerTreeView()

_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to