Index: cc/surfaces/surface_aggregator_unittest.cc |
diff --git a/cc/surfaces/surface_aggregator_unittest.cc b/cc/surfaces/surface_aggregator_unittest.cc |
index 2e47708415c8d8a21beac71bfe4596176f0ce614..1daff52e0d2475e9c118c06fda7b1cf8e0dfeb00 100644 |
--- a/cc/surfaces/surface_aggregator_unittest.cc |
+++ b/cc/surfaces/surface_aggregator_unittest.cc |
@@ -59,7 +59,7 @@ class SurfaceAggregatorTest : public testing::Test { |
TEST_F(SurfaceAggregatorTest, ValidSurfaceNoFrame) { |
SurfaceId one_id(7); |
- factory_.Create(one_id, SurfaceSize()); |
+ factory_.Create(one_id); |
scoped_ptr<CompositorFrame> frame = aggregator_.Aggregate(one_id); |
EXPECT_FALSE(frame); |
factory_.Destroy(one_id); |
@@ -72,7 +72,7 @@ class SurfaceAggregatorValidSurfaceTest : public SurfaceAggregatorTest { |
virtual void SetUp() { |
SurfaceAggregatorTest::SetUp(); |
root_surface_id_ = allocator_.GenerateId(); |
- factory_.Create(root_surface_id_, SurfaceSize()); |
+ factory_.Create(root_surface_id_); |
} |
virtual void TearDown() { |
@@ -157,7 +157,7 @@ TEST_F(SurfaceAggregatorValidSurfaceTest, SimpleFrame) { |
TEST_F(SurfaceAggregatorValidSurfaceTest, OpacityCopied) { |
SurfaceId embedded_surface_id = allocator_.GenerateId(); |
- factory_.Create(embedded_surface_id, SurfaceSize()); |
+ factory_.Create(embedded_surface_id); |
test::Quad embedded_quads[] = {test::Quad::SolidColorQuad(SK_ColorGREEN)}; |
test::Pass embedded_passes[] = { |
@@ -190,9 +190,9 @@ TEST_F(SurfaceAggregatorValidSurfaceTest, OpacityCopied) { |
TEST_F(SurfaceAggregatorValidSurfaceTest, OpacityCombinedWithNesting) { |
SurfaceId surface_id1 = allocator_.GenerateId(); |
- factory_.Create(surface_id1, SurfaceSize()); |
+ factory_.Create(surface_id1); |
SurfaceId surface_id2 = allocator_.GenerateId(); |
- factory_.Create(surface_id2, SurfaceSize()); |
+ factory_.Create(surface_id2); |
// |surface_id1| is color quad. |
{ |
@@ -258,7 +258,7 @@ TEST_F(SurfaceAggregatorValidSurfaceTest, MultiPassSimpleFrame) { |
// color quad should be aggregated into the final frame. |
TEST_F(SurfaceAggregatorValidSurfaceTest, SimpleSurfaceReference) { |
SurfaceId embedded_surface_id = allocator_.GenerateId(); |
- factory_.Create(embedded_surface_id, SurfaceSize()); |
+ factory_.Create(embedded_surface_id); |
test::Quad embedded_quads[] = {test::Quad::SolidColorQuad(SK_ColorGREEN)}; |
test::Pass embedded_passes[] = { |
@@ -287,7 +287,7 @@ TEST_F(SurfaceAggregatorValidSurfaceTest, SimpleSurfaceReference) { |
TEST_F(SurfaceAggregatorValidSurfaceTest, CopyRequest) { |
SurfaceId embedded_surface_id = allocator_.GenerateId(); |
- factory_.Create(embedded_surface_id, SurfaceSize()); |
+ factory_.Create(embedded_surface_id); |
test::Quad embedded_quads[] = {test::Quad::SolidColorQuad(SK_ColorGREEN)}; |
test::Pass embedded_passes[] = { |
@@ -344,7 +344,7 @@ TEST_F(SurfaceAggregatorValidSurfaceTest, CopyRequest) { |
// Root surface may contain copy requests. |
TEST_F(SurfaceAggregatorValidSurfaceTest, RootCopyRequest) { |
SurfaceId embedded_surface_id = allocator_.GenerateId(); |
- factory_.Create(embedded_surface_id, SurfaceSize()); |
+ factory_.Create(embedded_surface_id); |
test::Quad embedded_quads[] = {test::Quad::SolidColorQuad(SK_ColorGREEN)}; |
test::Pass embedded_passes[] = { |
@@ -432,7 +432,7 @@ TEST_F(SurfaceAggregatorValidSurfaceTest, RootCopyRequest) { |
// This tests referencing a surface that has multiple render passes. |
TEST_F(SurfaceAggregatorValidSurfaceTest, MultiPassSurfaceReference) { |
SurfaceId embedded_surface_id = child_allocator_.GenerateId(); |
- factory_.Create(embedded_surface_id, SurfaceSize()); |
+ factory_.Create(embedded_surface_id); |
RenderPassId pass_ids[] = {RenderPassId(1, 1), RenderPassId(1, 2), |
RenderPassId(1, 3)}; |
@@ -590,7 +590,7 @@ TEST_F(SurfaceAggregatorValidSurfaceTest, InvalidSurfaceReference) { |
// should also just be dropped. |
TEST_F(SurfaceAggregatorValidSurfaceTest, ValidSurfaceReferenceWithNoFrame) { |
SurfaceId surface_with_no_frame_id = allocator_.GenerateId(); |
- factory_.Create(surface_with_no_frame_id, gfx::Size(5, 5)); |
+ factory_.Create(surface_with_no_frame_id); |
test::Quad quads[] = {test::Quad::SolidColorQuad(SK_ColorGREEN), |
test::Quad::SurfaceQuad(surface_with_no_frame_id, 1.f), |
test::Quad::SolidColorQuad(SK_ColorBLUE)}; |
@@ -628,7 +628,7 @@ TEST_F(SurfaceAggregatorValidSurfaceTest, SimpleCyclicalReference) { |
// Tests a more complex cycle with one intermediate surface. |
TEST_F(SurfaceAggregatorValidSurfaceTest, TwoSurfaceCyclicalReference) { |
SurfaceId child_surface_id = allocator_.GenerateId(); |
- factory_.Create(child_surface_id, SurfaceSize()); |
+ factory_.Create(child_surface_id); |
test::Quad parent_quads[] = {test::Quad::SolidColorQuad(SK_ColorBLUE), |
test::Quad::SurfaceQuad(child_surface_id, 1.f), |
@@ -667,7 +667,7 @@ TEST_F(SurfaceAggregatorValidSurfaceTest, TwoSurfaceCyclicalReference) { |
// namespace and update RenderPassDrawQuad's id references to match. |
TEST_F(SurfaceAggregatorValidSurfaceTest, RenderPassIdMapping) { |
SurfaceId child_surface_id = allocator_.GenerateId(); |
- factory_.Create(child_surface_id, SurfaceSize()); |
+ factory_.Create(child_surface_id); |
RenderPassId child_pass_id[] = {RenderPassId(1, 1), RenderPassId(1, 2)}; |
test::Quad child_quad[][1] = {{test::Quad::SolidColorQuad(SK_ColorGREEN)}, |
@@ -795,7 +795,7 @@ TEST_F(SurfaceAggregatorValidSurfaceTest, AggregateSharedQuadStateProperties) { |
RenderPassId pass_id(1, 1); |
SurfaceId grandchild_surface_id = allocator_.GenerateId(); |
- factory_.Create(grandchild_surface_id, SurfaceSize()); |
+ factory_.Create(grandchild_surface_id); |
scoped_ptr<RenderPass> grandchild_pass = RenderPass::Create(); |
gfx::Rect output_rect(SurfaceSize()); |
gfx::Rect damage_rect(SurfaceSize()); |
@@ -807,7 +807,7 @@ TEST_F(SurfaceAggregatorValidSurfaceTest, AggregateSharedQuadStateProperties) { |
QueuePassAsFrame(grandchild_pass.Pass(), grandchild_surface_id); |
SurfaceId child_one_surface_id = allocator_.GenerateId(); |
- factory_.Create(child_one_surface_id, SurfaceSize()); |
+ factory_.Create(child_one_surface_id); |
scoped_ptr<RenderPass> child_one_pass = RenderPass::Create(); |
child_one_pass->SetNew( |
@@ -825,7 +825,7 @@ TEST_F(SurfaceAggregatorValidSurfaceTest, AggregateSharedQuadStateProperties) { |
QueuePassAsFrame(child_one_pass.Pass(), child_one_surface_id); |
SurfaceId child_two_surface_id = allocator_.GenerateId(); |
- factory_.Create(child_two_surface_id, SurfaceSize()); |
+ factory_.Create(child_two_surface_id); |
scoped_ptr<RenderPass> child_two_pass = RenderPass::Create(); |
child_two_pass->SetNew( |
@@ -903,7 +903,7 @@ TEST_F(SurfaceAggregatorValidSurfaceTest, AggregateSharedQuadStateProperties) { |
// affected. |
TEST_F(SurfaceAggregatorValidSurfaceTest, AggregateMultiplePassWithTransform) { |
SurfaceId child_surface_id = allocator_.GenerateId(); |
- factory_.Create(child_surface_id, SurfaceSize()); |
+ factory_.Create(child_surface_id); |
RenderPassId child_pass_id[] = {RenderPassId(1, 1), RenderPassId(1, 2)}; |
test::Quad child_quads[][1] = { |
{test::Quad::SolidColorQuad(SK_ColorGREEN)}, |
@@ -1038,7 +1038,7 @@ TEST_F(SurfaceAggregatorValidSurfaceTest, AggregateMultiplePassWithTransform) { |
// Tests that damage rects are aggregated correctly when surfaces change. |
TEST_F(SurfaceAggregatorValidSurfaceTest, AggregateDamageRect) { |
SurfaceId child_surface_id = allocator_.GenerateId(); |
- factory_.Create(child_surface_id, SurfaceSize()); |
+ factory_.Create(child_surface_id); |
RenderPassId child_pass_id = RenderPassId(1, 1); |
test::Quad child_quads[] = {test::Quad::RenderPassQuad(child_pass_id)}; |
test::Pass child_passes[] = { |
@@ -1300,7 +1300,7 @@ TEST_F(SurfaceAggregatorWithResourcesTest, TakeResourcesOneSurface) { |
ResourceTrackingSurfaceFactoryClient client; |
SurfaceFactory factory(&manager_, &client); |
SurfaceId surface_id(7u); |
- factory.Create(surface_id, SurfaceSize()); |
+ factory.Create(surface_id); |
ResourceProvider::ResourceId ids[] = {11, 12, 13}; |
SubmitFrameWithResources(ids, arraysize(ids), &factory, surface_id); |
@@ -1328,9 +1328,9 @@ TEST_F(SurfaceAggregatorWithResourcesTest, TwoSurfaces) { |
ResourceTrackingSurfaceFactoryClient client; |
SurfaceFactory factory(&manager_, &client); |
SurfaceId surface_id(7u); |
- factory.Create(surface_id, SurfaceSize()); |
+ factory.Create(surface_id); |
SurfaceId surface_id2(8u); |
- factory.Create(surface_id2, SurfaceSize()); |
+ factory.Create(surface_id2); |
ResourceProvider::ResourceId ids[] = {11, 12, 13}; |
SubmitFrameWithResources(ids, arraysize(ids), &factory, surface_id); |