cc: Avoid pointless gfx::Point/Size contructors when making Rects.

cc/layers/render_surface_unittest.cc

 // Copyright 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "cc/base/scoped_ptr_vector.h"
 #include "cc/layers/append_quads_data.h"
 #include "cc/layers/layer_impl.h"
 #include "cc/layers/render_pass_sink.h"
 #include "cc/layers/render_surface_impl.h"
 #include "cc/quads/shared_quad_state.h"
@@ skipping 24 matching lines @@
   // This test checks that SurfacePropertyChanged() has the correct behavior.
   //
 
   FakeImplProxy proxy;
   FakeLayerTreeHostImpl host_impl(&proxy);
   scoped_ptr<LayerImpl> owning_layer =
       LayerImpl::Create(host_impl.active_tree(), 1);
   owning_layer->CreateRenderSurface();
   ASSERT_TRUE(owning_layer->render_surface());
   RenderSurfaceImpl* render_surface = owning_layer->render_surface();
-  gfx::Rect test_rect = gfx::Rect(gfx::Point(3, 4), gfx::Size(5, 6));
+  gfx::Rect test_rect(3, 4, 5, 6);
   owning_layer->ResetAllChangeTrackingForSubtree();
 
   // Currently, the content_rect, clip_rect, and
   // owning_layer->layerPropertyChanged() are the only sources of change.
   EXECUTE_AND_VERIFY_SURFACE_CHANGED(render_surface->SetClipRect(test_rect));
   EXECUTE_AND_VERIFY_SURFACE_CHANGED(render_surface->SetContentRect(test_rect));
 
   owning_layer->SetOpacity(0.5f);
   EXPECT_TRUE(render_surface->SurfacePropertyChanged());
   owning_layer->ResetAllChangeTrackingForSubtree();
@@ skipping 30 matching lines @@
 
   scoped_ptr<LayerImpl> owning_layer =
       LayerImpl::Create(host_impl.active_tree(), 2);
   owning_layer->CreateRenderSurface();
   ASSERT_TRUE(owning_layer->render_surface());
   owning_layer->draw_properties().render_target = owning_layer.get();
   RenderSurfaceImpl* render_surface = owning_layer->render_surface();
 
   root_layer->AddChild(owning_layer.Pass());
 
-  gfx::Rect content_rect = gfx::Rect(gfx::Point(), gfx::Size(50, 50));
-  gfx::Rect clip_rect = gfx::Rect(gfx::Point(5, 5), gfx::Size(40, 40));
+  gfx::Rect content_rect(0, 0, 50, 50);
+  gfx::Rect clip_rect(5, 5, 40, 40);
   gfx::Transform origin;
 
   origin.Translate(30, 40);
 
   render_surface->SetDrawTransform(origin);
   render_surface->SetContentRect(content_rect);
   render_surface->SetClipRect(clip_rect);
   render_surface->SetDrawOpacity(1.f);
 
   QuadList quad_list;
@@ skipping 41 matching lines @@
 
   scoped_ptr<LayerImpl> owning_layer =
       LayerImpl::Create(host_impl.active_tree(), 2);
   owning_layer->CreateRenderSurface();
   ASSERT_TRUE(owning_layer->render_surface());
   owning_layer->draw_properties().render_target = owning_layer.get();
   RenderSurfaceImpl* render_surface = owning_layer->render_surface();
 
   root_layer->AddChild(owning_layer.Pass());
 
-  gfx::Rect content_rect = gfx::Rect(gfx::Point(), gfx::Size(50, 50));
+  gfx::Rect content_rect(0, 0, 50, 50);
   gfx::Transform origin;
   origin.Translate(30.0, 40.0);
 
   render_surface->SetScreenSpaceTransform(origin);
   render_surface->SetContentRect(content_rect);
 
   TestRenderPassSink pass_sink;
 
   render_surface->AppendRenderPasses(&pass_sink);
 
   ASSERT_EQ(1u, pass_sink.RenderPasses().size());
   RenderPass* pass = pass_sink.RenderPasses()[0];
 
   EXPECT_EQ(RenderPass::Id(2, 0), pass->id);
   EXPECT_RECT_EQ(content_rect, pass->output_rect);
   EXPECT_EQ(origin, pass->transform_to_root_target);
 }
 
 }  // namespace
 }  // namespace cc