| Index: cc/layers/picture_layer_impl.cc
|
| diff --git a/cc/layers/picture_layer_impl.cc b/cc/layers/picture_layer_impl.cc
|
| index 6ec1f0a1b616f9694a366a247f4f56838291c09e..54cfac88b32ec722abac90809ad0c5c9486e7731 100644
|
| --- a/cc/layers/picture_layer_impl.cc
|
| +++ b/cc/layers/picture_layer_impl.cc
|
| @@ -39,13 +39,6 @@ const float kMaxScaleRatioDuringPinch = 2.0f;
|
| // tiling's scale if the desired scale is within this ratio.
|
| const float kSnapToExistingTilingRatio = 1.2f;
|
|
|
| -// Estimate skewport 60 frames ahead for pre-rasterization on the CPU.
|
| -const float kCpuSkewportTargetTimeInFrames = 60.0f;
|
| -
|
| -// Don't pre-rasterize on the GPU (except for kBackflingGuardDistancePixels in
|
| -// TileManager::BinFromTilePriority).
|
| -const float kGpuSkewportTargetTimeInFrames = 0.0f;
|
| -
|
| // Even for really wide viewports, at some point GPU raster should use
|
| // less than 4 tiles to fill the viewport. This is set to 256 as a
|
| // sane minimum for now, but we might want to tune this for low-end.
|
| @@ -630,12 +623,8 @@ scoped_refptr<Tile> PictureLayerImpl::CreateTile(PictureLayerTiling* tiling,
|
|
|
| int flags = 0;
|
|
|
| - // TODO(vmpstr): Revisit this. For now, enabling analysis means that we get as
|
| - // much savings on memory as we can. However, for some cases like ganesh or
|
| - // small layers, the amount of time we spend analyzing might not justify
|
| - // memory savings that we can get. Note that we don't handle solid color
|
| - // masks, so we shouldn't bother analyzing those.
|
| - // Bugs: crbug.com/397198, crbug.com/396908
|
| + // We don't handle solid color masks, so we shouldn't bother analyzing those.
|
| + // Otherwise, always analyze to maximize memory savings.
|
| if (!is_mask_)
|
| flags = Tile::USE_PICTURE_ANALYSIS;
|
|
|
| @@ -684,13 +673,9 @@ size_t PictureLayerImpl::GetMaxTilesForInterestArea() const {
|
| }
|
|
|
| float PictureLayerImpl::GetSkewportTargetTimeInSeconds() const {
|
| - float skewport_target_time_in_frames =
|
| - layer_tree_impl()->use_gpu_rasterization()
|
| - ? kGpuSkewportTargetTimeInFrames
|
| - : kCpuSkewportTargetTimeInFrames;
|
| - return skewport_target_time_in_frames *
|
| - layer_tree_impl()->begin_impl_frame_interval().InSecondsF() *
|
| - layer_tree_impl()->settings().skewport_target_time_multiplier;
|
| + return layer_tree_impl()->use_gpu_rasterization()
|
| + ? 0.f
|
| + : layer_tree_impl()->settings().skewport_target_time_in_seconds;
|
| }
|
|
|
| int PictureLayerImpl::GetSkewportExtrapolationLimitInContentPixels() const {
|
|
|