cc: Stop using contents_scale_x() in PictureLayerImpl since it's 1.

cc: Stop using contents_scale_x() in PictureLayerImpl since it's 1.

We are still passing it around and scaling a rect by it but it is
always 1 so just make that explicit.

R=vmpstr

Committed: https://crrev.com/f502e86b9352ab1cffff5c76ff9d5e33c8e6dcbc
Cr-Commit-Position: refs/heads/master@{#294730}

[vmpstr, 2014-09-12 22:34:21]

lgtm

https://codereview.chromium.org/566023004/diff/20001/cc/layers/picture_layer_...
File cc/layers/picture_layer_impl.cc (right):

https://codereview.chromium.org/566023004/diff/20001/cc/layers/picture_layer_...
cc/layers/picture_layer_impl.cc:527:
visible_rect_in_content_space.Intersect(visible_content_rect());
This seems unrelated to removing contents_scale_x? I'm ok with it, but is there
a particular reason for the change or just a cleanup?

https://codereview.chromium.org/566023004/diff/20001/cc/layers/picture_layer_...
cc/layers/picture_layer_impl.cc:796: gfx::Rect rect =
GetViewportForTilePriorityInContentSpace();
And this.

[danakj, 2014-09-13 02:03:49]

https://codereview.chromium.org/566023004/diff/20001/cc/layers/picture_layer_...
File cc/layers/picture_layer_impl.cc (right):

https://codereview.chromium.org/566023004/diff/20001/cc/layers/picture_layer_...
cc/layers/picture_layer_impl.cc:527:
visible_rect_in_content_space.Intersect(visible_content_rect());
On 2014/09/12 22:34:20, vmpstr wrote:
> This seems unrelated to removing contents_scale_x? I'm ok with it, but is
there
> a particular reason for the change or just a cleanup?

oops this is https://codereview.chromium.org/571653003/, i'll upload without it.

[danakj, 2014-09-13 02:04:35]

Patchset #2 (id:20001) has been deleted

[danakj, 2014-09-13 02:04:48]

The CQ bit was checked by danakj@chromium.org

[danakj, 2014-09-13 02:04:56]

The CQ bit was unchecked by danakj@chromium.org

[danakj, 2014-09-13 03:11:32]

The CQ bit was checked by danakj@chromium.org

[commit-bot: I haz the power, 2014-09-13 03:11:58]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patchset/566023004/40001

[commit-bot: I haz the power, 2014-09-13 04:27:30]

Committed patchset #2 (id:40001) as cc53b4e6f5072cf8c57e2a13f1152dc6a1811229

[commit-bot: I haz the power, 2014-09-13 04:29:21]

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