cc: Ensure that raster contents scale doesn't go below min scale.

cc: Ensure that raster contents scale doesn't go below min scale.

This patch ensures that we can the raster scale to min contents scale.
We already did this, but later could modify the value again to be lower
than MinContentsScale. Now, the clamp happens near the end of the
function.

R=danakj
BUG=429397
Committed: https://crrev.com/2eb362d23158c4897a9346edae4e630c6c9f2702
Cr-Commit-Position: refs/heads/master@{#302636}

[vmpstr, 2014-11-04 00:36:59]

PTAL

[danakj, 2014-11-04 16:45:02]

https://codereview.chromium.org/699053003/diff/1/cc/layers/picture_layer_impl.cc
File cc/layers/picture_layer_impl.cc (right):

https://codereview.chromium.org/699053003/diff/1/cc/layers/picture_layer_impl...
cc/layers/picture_layer_impl.cc:1141: DCHECK_LE(low_res_raster_contents_scale_,
raster_contents_scale_);
can you DCHECK GE MinContentsScale() for both of them as well?

[danakj, 2014-11-04 16:45:25]

https://codereview.chromium.org/699053003/diff/1/cc/layers/picture_layer_impl.cc
File cc/layers/picture_layer_impl.cc (right):

https://codereview.chromium.org/699053003/diff/1/cc/layers/picture_layer_impl...
cc/layers/picture_layer_impl.cc:1141: DCHECK_LE(low_res_raster_contents_scale_,
raster_contents_scale_);
On 2014/11/04 16:45:01, danakj wrote:
> can you DCHECK GE MinContentsScale() for both of them as well?

Well, I guess for for low res is enough since this implies the rest.

[vmpstr, 2014-11-04 17:24:02]

PTAL

https://codereview.chromium.org/699053003/diff/1/cc/layers/picture_layer_impl.cc
File cc/layers/picture_layer_impl.cc (right):

https://codereview.chromium.org/699053003/diff/1/cc/layers/picture_layer_impl...
cc/layers/picture_layer_impl.cc:1141: DCHECK_LE(low_res_raster_contents_scale_,
raster_contents_scale_);
On 2014/11/04 16:45:25, danakj wrote:
> On 2014/11/04 16:45:01, danakj wrote:
> > can you DCHECK GE MinContentsScale() for both of them as well?
> 
> Well, I guess for for low res is enough since this implies the rest.

Done.

[danakj, 2014-11-04 17:34:40]

LGTM

[danakj, 2014-11-04 17:34:44]

The CQ bit was checked by danakj@chromium.org

[commit-bot: I haz the power, 2014-11-04 17:35:43]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/699053003/20001

[commit-bot: I haz the power, 2014-11-04 19:04:21]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2014-11-04 19:04:58]

Patchset 2 (id:??) landed as
https://crrev.com/2eb362d23158c4897a9346edae4e630c6c9f2702
Cr-Commit-Position: refs/heads/master@{#302636}