cc: Give TilingData a Rect instead of a Size

cc/test/fake_picture_layer_impl.cc

 // Copyright 2013 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/test/fake_picture_layer_impl.h"
 
 #include <vector>
 #include "cc/resources/tile.h"
 #include "cc/trees/layer_tree_impl.h"
 
 namespace cc {
 
 FakePictureLayerImpl::FakePictureLayerImpl(
     LayerTreeImpl* tree_impl,
     int id,
-    scoped_refptr<PicturePileImpl> pile)
+    scoped_refptr<PicturePileImpl> pile,
+    const gfx::Size& layer_bounds)
     : PictureLayerImpl(tree_impl, id),
       append_quads_count_(0) {
   pile_ = pile;
-  SetBounds(pile_->size());
+  SetBounds(layer_bounds);

[enne (OOO), 2014/04/11 20:19:51]

Why do you need the explicit bounds here? Can you make bounds from the pile, as
before, and maybe provide another ctor for new test cases?

[ernstm, 2014/04/11 23:43:51]

Done. This was intended for new test cases.

 }
 
 FakePictureLayerImpl::FakePictureLayerImpl(LayerTreeImpl* tree_impl, int id)
     : PictureLayerImpl(tree_impl, id), append_quads_count_(0) {}
 
 scoped_ptr<LayerImpl> FakePictureLayerImpl::CreateLayerImpl(
     LayerTreeImpl* tree_impl) {
   return make_scoped_ptr(
       new FakePictureLayerImpl(tree_impl, id())).PassAs<LayerImpl>();
 }
@@ skipping 99 matching lines @@
     DCHECK_EQ(tilings()->tiling_at(0)->resolution(), HIGH_RESOLUTION);
     DCHECK_EQ(tilings()->tiling_at(1)->resolution(), LOW_RESOLUTION);
     HighResTiling()->CreateAllTilesForTesting();
     LowResTiling()->CreateAllTilesForTesting();
   } else {
     DCHECK_EQ(tilings()->num_tilings(), 0u);
   }
 }
 
 }  // namespace cc