Title: [111815] trunk/Source/WebKit/chromium
Revision
111815
Author
[email protected]
Date
2012-03-22 19:49:45 -0700 (Thu, 22 Mar 2012)

Log Message

[chromium] Removing failing unit test - CCLayerTreeHostImplTest.visibilityChangeResetsDamage
https://bugs.webkit.org/show_bug.cgi?id=82003

Patch by Michal Mocny <[email protected]> on 2012-03-22
Reviewed by Adrienne Walker.

* tests/CCLayerTreeHostImplTest.cpp:
(WebKitTests::TEST_F):

Modified Paths

Diff

Modified: trunk/Source/WebKit/chromium/ChangeLog (111814 => 111815)


--- trunk/Source/WebKit/chromium/ChangeLog	2012-03-23 02:42:10 UTC (rev 111814)
+++ trunk/Source/WebKit/chromium/ChangeLog	2012-03-23 02:49:45 UTC (rev 111815)
@@ -1,3 +1,13 @@
+2012-03-22  Michal Mocny  <[email protected]>
+
+        [chromium] Removing failing unit test - CCLayerTreeHostImplTest.visibilityChangeResetsDamage
+        https://bugs.webkit.org/show_bug.cgi?id=82003
+
+        Reviewed by Adrienne Walker.
+
+        * tests/CCLayerTreeHostImplTest.cpp:
+        (WebKitTests::TEST_F):
+
 2012-03-22  Robert Kroeger  <[email protected]>
 
         [chromium] Halt in progress flings on receipt of key events

Modified: trunk/Source/WebKit/chromium/tests/CCLayerTreeHostImplTest.cpp (111814 => 111815)


--- trunk/Source/WebKit/chromium/tests/CCLayerTreeHostImplTest.cpp	2012-03-23 02:42:10 UTC (rev 111814)
+++ trunk/Source/WebKit/chromium/tests/CCLayerTreeHostImplTest.cpp	2012-03-23 02:49:45 UTC (rev 111815)
@@ -980,54 +980,6 @@
     EXPECT_EQ(expectedSwapRect.height(), actualSwapRect.height());
 }
 
-// Make sure that we reset damage tracking on visibility change because the
-// state of the front buffer that we push to with PostSubBuffer is undefined.
-TEST_F(CCLayerTreeHostImplTest, visibilityChangeResetsDamage)
-{
-    PartialSwapTrackerContext* partialSwapTracker = new PartialSwapTrackerContext();
-    RefPtr<GraphicsContext3D> context = GraphicsContext3DPrivate::createGraphicsContextFromWebContext(adoptPtr(partialSwapTracker), GraphicsContext3D::RenderDirectlyToHostWindow);
-
-    // This test creates its own CCLayerTreeHostImpl, so
-    // that we can force partial swap enabled.
-    CCSettings settings;
-    settings.partialSwapEnabled = true;
-    OwnPtr<CCLayerTreeHostImpl> layerTreeHostImpl = CCLayerTreeHostImpl::create(settings, this);
-    layerTreeHostImpl->initializeLayerRenderer(context);
-    layerTreeHostImpl->setViewportSize(IntSize(500, 500));
-
-    CCLayerImpl* root = new FakeDrawableCCLayerImpl(1);
-    root->setAnchorPoint(FloatPoint(0, 0));
-    root->setBounds(IntSize(500, 500));
-    root->setDrawsContent(true);
-    layerTreeHostImpl->setRootLayer(adoptPtr(root));
-
-    CCLayerTreeHostImpl::FrameData frame;
-
-    // First frame: ignore.
-    EXPECT_TRUE(layerTreeHostImpl->prepareToDraw(frame));
-    layerTreeHostImpl->drawLayers(frame);
-    layerTreeHostImpl->swapBuffers();
-
-    // Second frame: nothing has changed --- so we souldn't push anything with partial swap.
-    EXPECT_TRUE(layerTreeHostImpl->prepareToDraw(frame));
-    layerTreeHostImpl->drawLayers(frame);
-    layerTreeHostImpl->swapBuffers();
-    EXPECT_TRUE(partialSwapTracker->partialSwapRect().isEmpty());
-
-    // Third frame: visibility change --- so we should push a full frame with partial swap.
-    layerTreeHostImpl->setVisible(false);
-    layerTreeHostImpl->setVisible(true);
-    EXPECT_TRUE(layerTreeHostImpl->prepareToDraw(frame));
-    layerTreeHostImpl->drawLayers(frame);
-    layerTreeHostImpl->swapBuffers();
-    IntRect actualSwapRect = partialSwapTracker->partialSwapRect();
-    IntRect expectedSwapRect = IntRect(IntPoint::zero(), IntSize(500, 500));
-    EXPECT_EQ(expectedSwapRect.x(), actualSwapRect.x());
-    EXPECT_EQ(expectedSwapRect.y(), actualSwapRect.y());
-    EXPECT_EQ(expectedSwapRect.width(), actualSwapRect.width());
-    EXPECT_EQ(expectedSwapRect.height(), actualSwapRect.height());
-}
-
 // Make sure that context lost notifications are propagated through the tree.
 class ContextLostNotificationCheckLayer : public CCLayerImpl {
 public:
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to