cc: Don't clip tile priority visible rect to content bounds.

cc/layers/picture_layer_impl_unittest.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/layers/picture_layer_impl.h"
 
 #include <algorithm>
 #include <limits>
 #include <set>
 #include <utility>
@@ skipping 387 matching lines @@
   EXPECT_TRUE(success);
 
   viewport_rect_for_tile_priority_in_view_space =
       gfx::ToEnclosingRect(MathUtil::ProjectClippedRect(
           screen_to_view, viewport_rect_for_tile_priority));
 
   // Verify the viewport rect for tile priority is used in PictureLayerImpl.
   EXPECT_EQ(active_layer_->viewport_rect_for_tile_priority(),
             viewport_rect_for_tile_priority_in_view_space);
 
-  // Interset viewport_rect_for_tile_priority_in_view_space with the layer

[danakj, 2014/09/18 23:19:36]

It would be really easy to add back that intersect and just say "oh this test
must be wrong". Can you add a comment explaining here why the rect should be
outside the layer bounds?

[vmpstr, 2014/09/18 23:58:46]

Done.

-  // bounds and the result should be used in PictureLayerTiling.
-  viewport_rect_for_tile_priority_in_view_space.Intersect(
-      gfx::Rect(layer_bounds));
   tilings = active_layer_->tilings();
   for (size_t i = 0; i < tilings->num_tilings(); i++) {
     PictureLayerTiling* tiling = tilings->tiling_at(i);
     EXPECT_EQ(
         tiling->GetCurrentVisibleRectForTesting(),
         gfx::ScaleToEnclosingRect(viewport_rect_for_tile_priority_in_view_space,
                                   tiling->contents_scale()));
   }
 }
 
@@ skipping 3795 matching lines @@
   ActivateTree();
   EXPECT_TRUE(active_layer_->GetRecycledTwinLayer());
   EXPECT_EQ(old_pending_layer_, active_layer_->GetRecycledTwinLayer());
 
   host_impl_.ResetRecycleTreeForTesting();
   EXPECT_FALSE(active_layer_->GetRecycledTwinLayer());
 }
 
 }  // namespace
 }  // namespace cc