Cache viewport rect for tile priority in UpdateTiles

cc/layers/picture_layer_impl.h

 // Copyright 2012 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.
 
 #ifndef CC_LAYERS_PICTURE_LAYER_IMPL_H_
 #define CC_LAYERS_PICTURE_LAYER_IMPL_H_
 
 #include <set>
 #include <string>
 #include <vector>
@@ skipping 101 matching lines @@
   void RemoveAllTilings();
   void AddTilingsForRasterScale();
   bool UpdateTilePriorities(const Occlusion& occlusion_in_content_space);
   virtual bool ShouldAdjustRasterScale() const;
   virtual void RecalculateRasterScales();
   void CleanUpTilingsOnActiveLayer(
       std::vector<PictureLayerTiling*> used_tilings);
   float MinimumContentsScale() const;
   float MaximumContentsScale() const;
   void ResetRasterScale();
-  gfx::Rect GetViewportForTilePriorityInContentSpace() const;
+  gfx::Rect viewport_rect_for_tile_priority_in_content_space() const {

[danakj, 2015/01/21 22:32:18]

I meant you can have this accessor on the FakePictureLayerImpl. You don't need
to make it public here.

[hush (inactive), 2015/01/21 22:40:46]

Oh. I see. Done.

+    return viewport_rect_for_tile_priority_in_content_space_;
+  }
+  void CacheViewportForTilePriorityInContentSpace();
   PictureLayerImpl* GetRecycledTwinLayer() const;
 
   void SanityCheckTilingState() const;
   // Checks if all tiles required for a certain action (e.g. activation) are
   // ready to draw.  is_tile_required_callback gets called on all candidate
   // tiles and returns true if the tile is required for the action.
   bool AllTilesRequiredAreReadyToDraw(
       TileRequirementCheck is_tile_required_callback) const;
 
   bool ShouldAdjustRasterScaleDuringScaleAnimations() const;
@@ skipping 33 matching lines @@
 
   bool nearest_neighbor_;
 
   // Any draw properties derived from |transform|, |viewport|, and |clip|
   // parameters in LayerTreeHostImpl::SetExternalDrawConstraints are not valid
   // for prioritizing tiles during resourceless software draws. This is because
   // resourceless software draws can have wildly different transforms/viewports
   // from regular draws. Save a copy of the required draw properties of the last
   // frame that has a valid viewport for prioritizing tiles.
   gfx::Rect visible_rect_for_tile_priority_;
+  gfx::Rect viewport_rect_for_tile_priority_in_content_space_;
 
   DISALLOW_COPY_AND_ASSIGN(PictureLayerImpl);
 };
 
 }  // namespace cc
 
 #endif  // CC_LAYERS_PICTURE_LAYER_IMPL_H_