Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(652)

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: cc/layer_tree_host_unittest.cc

Issue 11360066: cc: Remove WebCore rect use from the compositor, except within Region. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Signs Created 8 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « cc/layer_impl.h ('k') | cc/occlusion_tracker.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/layer_tree_host_unittest.cc
diff --git a/cc/layer_tree_host_unittest.cc b/cc/layer_tree_host_unittest.cc
index 4deb60d0fa725d50bc8f9d4ea023005b1834adab..704158ef0fc4c84b55e8687011474900e7744280 100644
--- a/cc/layer_tree_host_unittest.cc
+++ b/cc/layer_tree_host_unittest.cc
@@ -1689,11 +1689,11 @@ public:
m_layerTreeHost->commitComplete();
EXPECT_RECT_EQ(gfx::Rect(), grandChild->occludedScreenSpace().bounds());
- EXPECT_EQ(0u, grandChild->occludedScreenSpace().rects().size());
+ EXPECT_EQ(0u, Region::Iterator(grandChild->occludedScreenSpace()).size());
EXPECT_RECT_EQ(gfx::Rect(30, 40, 170, 160), child->occludedScreenSpace().bounds());
- EXPECT_EQ(1u, child->occludedScreenSpace().rects().size());
+ EXPECT_EQ(1u, Region::Iterator(child->occludedScreenSpace()).size());
EXPECT_RECT_EQ(gfx::Rect(30, 40, 170, 160), rootLayer->occludedScreenSpace().bounds());
- EXPECT_EQ(1u, rootLayer->occludedScreenSpace().rects().size());
+ EXPECT_EQ(1u, Region::Iterator(rootLayer->occludedScreenSpace()).size());
// If the child layer is opaque, then it adds to the occlusion seen by the rootLayer.
setLayerPropertiesForTesting(rootLayer.get(), 0, identityMatrix, gfx::PointF(0, 0), gfx::PointF(0, 0), gfx::Size(200, 200), true);
@@ -1706,11 +1706,11 @@ public:
m_layerTreeHost->commitComplete();
EXPECT_RECT_EQ(gfx::Rect(), grandChild->occludedScreenSpace().bounds());
- EXPECT_EQ(0u, grandChild->occludedScreenSpace().rects().size());
+ EXPECT_EQ(0u, Region::Iterator(grandChild->occludedScreenSpace()).size());
EXPECT_RECT_EQ(gfx::Rect(30, 40, 170, 160), child->occludedScreenSpace().bounds());
- EXPECT_EQ(1u, child->occludedScreenSpace().rects().size());
+ EXPECT_EQ(1u, Region::Iterator(child->occludedScreenSpace()).size());
EXPECT_RECT_EQ(gfx::Rect(30, 30, 170, 170), rootLayer->occludedScreenSpace().bounds());
- EXPECT_EQ(1u, rootLayer->occludedScreenSpace().rects().size());
+ EXPECT_EQ(1u, Region::Iterator(rootLayer->occludedScreenSpace()).size());
// Add a second child to the root layer and the regions should merge
setTestLayerPropertiesForTesting(rootLayer.get(), 0, identityMatrix, gfx::PointF(0, 0), gfx::PointF(0, 0), gfx::Size(200, 200), true);
@@ -1724,13 +1724,13 @@ public:
m_layerTreeHost->commitComplete();
EXPECT_RECT_EQ(gfx::Rect(), grandChild->occludedScreenSpace().bounds());
- EXPECT_EQ(0u, grandChild->occludedScreenSpace().rects().size());
+ EXPECT_EQ(0u, Region::Iterator(grandChild->occludedScreenSpace()).size());
EXPECT_RECT_EQ(gfx::Rect(30, 40, 170, 160), child->occludedScreenSpace().bounds());
- EXPECT_EQ(1u, child->occludedScreenSpace().rects().size());
+ EXPECT_EQ(1u, Region::Iterator(child->occludedScreenSpace()).size());
EXPECT_RECT_EQ(gfx::Rect(30, 30, 170, 170), child2->occludedScreenSpace().bounds());
- EXPECT_EQ(1u, child2->occludedScreenSpace().rects().size());
+ EXPECT_EQ(1u, Region::Iterator(child2->occludedScreenSpace()).size());
EXPECT_RECT_EQ(gfx::Rect(30, 20, 170, 180), rootLayer->occludedScreenSpace().bounds());
- EXPECT_EQ(2u, rootLayer->occludedScreenSpace().rects().size());
+ EXPECT_EQ(2u, Region::Iterator(rootLayer->occludedScreenSpace()).size());
// Move the second child to be sure.
setTestLayerPropertiesForTesting(rootLayer.get(), 0, identityMatrix, gfx::PointF(0, 0), gfx::PointF(0, 0), gfx::Size(200, 200), true);
@@ -1744,13 +1744,13 @@ public:
m_layerTreeHost->commitComplete();
EXPECT_RECT_EQ(gfx::Rect(), grandChild->occludedScreenSpace().bounds());
- EXPECT_EQ(0u, grandChild->occludedScreenSpace().rects().size());
+ EXPECT_EQ(0u, Region::Iterator(grandChild->occludedScreenSpace()).size());
EXPECT_RECT_EQ(gfx::Rect(30, 40, 170, 160), child->occludedScreenSpace().bounds());
- EXPECT_EQ(1u, child->occludedScreenSpace().rects().size());
+ EXPECT_EQ(1u, Region::Iterator(child->occludedScreenSpace()).size());
EXPECT_RECT_EQ(gfx::Rect(30, 30, 170, 170), child2->occludedScreenSpace().bounds());
- EXPECT_EQ(1u, child2->occludedScreenSpace().rects().size());
+ EXPECT_EQ(1u, Region::Iterator(child2->occludedScreenSpace()).size());
EXPECT_RECT_EQ(gfx::Rect(10, 30, 190, 170), rootLayer->occludedScreenSpace().bounds());
- EXPECT_EQ(2u, rootLayer->occludedScreenSpace().rects().size());
+ EXPECT_EQ(2u, Region::Iterator(rootLayer->occludedScreenSpace()).size());
// If the child layer has a mask on it, then it shouldn't contribute to occlusion on stuff below it
setLayerPropertiesForTesting(rootLayer.get(), 0, identityMatrix, gfx::PointF(0, 0), gfx::PointF(0, 0), gfx::Size(200, 200), true);
@@ -1766,13 +1766,13 @@ public:
m_layerTreeHost->commitComplete();
EXPECT_RECT_EQ(gfx::Rect(), grandChild->occludedScreenSpace().bounds());
- EXPECT_EQ(0u, grandChild->occludedScreenSpace().rects().size());
+ EXPECT_EQ(0u, Region::Iterator(grandChild->occludedScreenSpace()).size());
EXPECT_RECT_EQ(gfx::Rect(30, 40, 170, 160), child->occludedScreenSpace().bounds());
- EXPECT_EQ(1u, child->occludedScreenSpace().rects().size());
+ EXPECT_EQ(1u, Region::Iterator(child->occludedScreenSpace()).size());
EXPECT_RECT_EQ(gfx::Rect(), child2->occludedScreenSpace().bounds());
- EXPECT_EQ(0u, child2->occludedScreenSpace().rects().size());
+ EXPECT_EQ(0u, Region::Iterator(child2->occludedScreenSpace()).size());
EXPECT_RECT_EQ(gfx::Rect(10, 70, 190, 130), rootLayer->occludedScreenSpace().bounds());
- EXPECT_EQ(1u, rootLayer->occludedScreenSpace().rects().size());
+ EXPECT_EQ(1u, Region::Iterator(rootLayer->occludedScreenSpace()).size());
// If the child layer with a mask is below child2, then child2 should contribute to occlusion on everything, and child shouldn't contribute to the rootLayer
setLayerPropertiesForTesting(rootLayer.get(), 0, identityMatrix, gfx::PointF(0, 0), gfx::PointF(0, 0), gfx::Size(200, 200), true);
@@ -1788,13 +1788,13 @@ public:
m_layerTreeHost->commitComplete();
EXPECT_RECT_EQ(gfx::Rect(), child2->occludedScreenSpace().bounds());
- EXPECT_EQ(0u, child2->occludedScreenSpace().rects().size());
+ EXPECT_EQ(0u, Region::Iterator(child2->occludedScreenSpace()).size());
EXPECT_RECT_EQ(gfx::Rect(10, 70, 190, 130), grandChild->occludedScreenSpace().bounds());
- EXPECT_EQ(1u, grandChild->occludedScreenSpace().rects().size());
+ EXPECT_EQ(1u, Region::Iterator(grandChild->occludedScreenSpace()).size());
EXPECT_RECT_EQ(gfx::Rect(10, 40, 190, 160), child->occludedScreenSpace().bounds());
- EXPECT_EQ(2u, child->occludedScreenSpace().rects().size());
+ EXPECT_EQ(2u, Region::Iterator(child->occludedScreenSpace()).size());
EXPECT_RECT_EQ(gfx::Rect(10, 70, 190, 130), rootLayer->occludedScreenSpace().bounds());
- EXPECT_EQ(1u, rootLayer->occludedScreenSpace().rects().size());
+ EXPECT_EQ(1u, Region::Iterator(rootLayer->occludedScreenSpace()).size());
// If the child layer has a non-opaque drawOpacity, then it shouldn't contribute to occlusion on stuff below it
setTestLayerPropertiesForTesting(rootLayer.get(), 0, identityMatrix, gfx::PointF(0, 0), gfx::PointF(0, 0), gfx::Size(200, 200), true);
@@ -1811,13 +1811,13 @@ public:
m_layerTreeHost->commitComplete();
EXPECT_RECT_EQ(gfx::Rect(), grandChild->occludedScreenSpace().bounds());
- EXPECT_EQ(0u, grandChild->occludedScreenSpace().rects().size());
+ EXPECT_EQ(0u, Region::Iterator(grandChild->occludedScreenSpace()).size());
EXPECT_RECT_EQ(gfx::Rect(30, 40, 170, 160), child->occludedScreenSpace().bounds());
- EXPECT_EQ(1u, child->occludedScreenSpace().rects().size());
+ EXPECT_EQ(1u, Region::Iterator(child->occludedScreenSpace()).size());
EXPECT_RECT_EQ(gfx::Rect(), child2->occludedScreenSpace().bounds());
- EXPECT_EQ(0u, child2->occludedScreenSpace().rects().size());
+ EXPECT_EQ(0u, Region::Iterator(child2->occludedScreenSpace()).size());
EXPECT_RECT_EQ(gfx::Rect(10, 70, 190, 130), rootLayer->occludedScreenSpace().bounds());
- EXPECT_EQ(1u, rootLayer->occludedScreenSpace().rects().size());
+ EXPECT_EQ(1u, Region::Iterator(rootLayer->occludedScreenSpace()).size());
// If the child layer with non-opaque drawOpacity is below child2, then child2 should contribute to occlusion on everything, and child shouldn't contribute to the rootLayer
setTestLayerPropertiesForTesting(rootLayer.get(), 0, identityMatrix, gfx::PointF(0, 0), gfx::PointF(0, 0), gfx::Size(200, 200), true);
@@ -1834,13 +1834,13 @@ public:
m_layerTreeHost->commitComplete();
EXPECT_RECT_EQ(gfx::Rect(), child2->occludedScreenSpace().bounds());
- EXPECT_EQ(0u, child2->occludedScreenSpace().rects().size());
+ EXPECT_EQ(0u, Region::Iterator(child2->occludedScreenSpace()).size());
EXPECT_RECT_EQ(gfx::Rect(10, 70, 190, 130), grandChild->occludedScreenSpace().bounds());
- EXPECT_EQ(1u, grandChild->occludedScreenSpace().rects().size());
+ EXPECT_EQ(1u, Region::Iterator(grandChild->occludedScreenSpace()).size());
EXPECT_RECT_EQ(gfx::Rect(10, 40, 190, 160), child->occludedScreenSpace().bounds());
- EXPECT_EQ(2u, child->occludedScreenSpace().rects().size());
+ EXPECT_EQ(2u, Region::Iterator(child->occludedScreenSpace()).size());
EXPECT_RECT_EQ(gfx::Rect(10, 70, 190, 130), rootLayer->occludedScreenSpace().bounds());
- EXPECT_EQ(1u, rootLayer->occludedScreenSpace().rects().size());
+ EXPECT_EQ(1u, Region::Iterator(rootLayer->occludedScreenSpace()).size());
// Kill the layerTreeHost immediately.
m_layerTreeHost->setRootLayer(0);
@@ -1897,13 +1897,13 @@ public:
m_layerTreeHost->commitComplete();
EXPECT_RECT_EQ(gfx::Rect(), child2->occludedScreenSpace().bounds());
- EXPECT_EQ(0u, child2->occludedScreenSpace().rects().size());
+ EXPECT_EQ(0u, Region::Iterator(child2->occludedScreenSpace()).size());
EXPECT_RECT_EQ(gfx::Rect(10, 70, 190, 130), grandChild->occludedScreenSpace().bounds());
- EXPECT_EQ(1u, grandChild->occludedScreenSpace().rects().size());
+ EXPECT_EQ(1u, Region::Iterator(grandChild->occludedScreenSpace()).size());
EXPECT_RECT_EQ(gfx::Rect(10, 40, 190, 160), child->occludedScreenSpace().bounds());
- EXPECT_EQ(2u, child->occludedScreenSpace().rects().size());
+ EXPECT_EQ(2u, Region::Iterator(child->occludedScreenSpace()).size());
EXPECT_RECT_EQ(gfx::Rect(10, 70, 190, 130), rootLayer->occludedScreenSpace().bounds());
- EXPECT_EQ(1u, rootLayer->occludedScreenSpace().rects().size());
+ EXPECT_EQ(1u, Region::Iterator(rootLayer->occludedScreenSpace()).size());
// If the child layer has a filter that moves pixels/changes alpha, and is below child2, then child should not inherit occlusion from outside its subtree,
// and should not contribute to the rootLayer
@@ -1924,13 +1924,13 @@ public:
m_layerTreeHost->commitComplete();
EXPECT_RECT_EQ(gfx::Rect(), child2->occludedScreenSpace().bounds());
- EXPECT_EQ(0u, child2->occludedScreenSpace().rects().size());
+ EXPECT_EQ(0u, Region::Iterator(child2->occludedScreenSpace()).size());
EXPECT_RECT_EQ(gfx::Rect(), grandChild->occludedScreenSpace().bounds());
- EXPECT_EQ(0u, grandChild->occludedScreenSpace().rects().size());
+ EXPECT_EQ(0u, Region::Iterator(grandChild->occludedScreenSpace()).size());
EXPECT_RECT_EQ(gfx::Rect(30, 40, 170, 160), child->occludedScreenSpace().bounds());
- EXPECT_EQ(1u, child->occludedScreenSpace().rects().size());
+ EXPECT_EQ(1u, Region::Iterator(child->occludedScreenSpace()).size());
EXPECT_RECT_EQ(gfx::Rect(10, 70, 190, 130), rootLayer->occludedScreenSpace().bounds());
- EXPECT_EQ(1u, rootLayer->occludedScreenSpace().rects().size());
+ EXPECT_EQ(1u, Region::Iterator(rootLayer->occludedScreenSpace()).size());
// Kill the layerTreeHost immediately.
m_layerTreeHost->setRootLayer(0);
@@ -1989,7 +1989,7 @@ public:
gfx::Rect expectedOcclusion(i+1, i+1, 200-i-1, 200-i-1);
EXPECT_RECT_EQ(expectedOcclusion, layers[i]->occludedScreenSpace().bounds());
- EXPECT_EQ(1u, layers[i]->occludedScreenSpace().rects().size());
+ EXPECT_EQ(1u, Region::Iterator(layers[i]->occludedScreenSpace()).size());
}
// Kill the layerTreeHost immediately.
« no previous file with comments | « cc/layer_impl.h ('k') | cc/occlusion_tracker.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698