Makes GetClipBounds() return the enclosing rect

Makes GetClipBounds() return the enclosing rect

The current code floors, which means we may not paint enough, an in
fact you can see this in views at certain scales.

BUG=none
TEST=none
R=danakj@chromium.org

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

[danakj, 2014-09-10 18:27:16]

LGTM

https://codereview.chromium.org/557953004/diff/1/ui/gfx/canvas_unittest.cc
File ui/gfx/canvas_unittest.cc (right):

https://codereview.chromium.org/557953004/diff/1/ui/gfx/canvas_unittest.cc#ne...
ui/gfx/canvas_unittest.cc:55:
canvas.sk_canvas()->clipRect(RectFToSkRect(gfx::RectF(100, 0, 20, 1.7f)));
nit: 1.2 to show it's not just rounding?

[sky, 2014-09-10 18:29:51]

https://codereview.chromium.org/557953004/diff/1/ui/gfx/canvas_unittest.cc
File ui/gfx/canvas_unittest.cc (right):

https://codereview.chromium.org/557953004/diff/1/ui/gfx/canvas_unittest.cc#ne...
ui/gfx/canvas_unittest.cc:55:
canvas.sk_canvas()->clipRect(RectFToSkRect(gfx::RectF(100, 0, 20, 1.7f)));
On 2014/09/10 18:27:16, danakj wrote:
> nit: 1.2 to show it's not just rounding?

Actually, the 1.2 isn't the interesting part here. It's the 100,20, because of
scaling.

[sky, 2014-09-10 18:29:55]

The CQ bit was checked by sky@chromium.org

[danakj, 2014-09-10 18:32:13]

https://codereview.chromium.org/557953004/diff/1/ui/gfx/canvas_unittest.cc
File ui/gfx/canvas_unittest.cc (right):

https://codereview.chromium.org/557953004/diff/1/ui/gfx/canvas_unittest.cc#ne...
ui/gfx/canvas_unittest.cc:55:
canvas.sk_canvas()->clipRect(RectFToSkRect(gfx::RectF(100, 0, 20, 1.7f)));
On 2014/09/10 18:29:51, sky wrote:
> On 2014/09/10 18:27:16, danakj wrote:
> > nit: 1.2 to show it's not just rounding?
> 
> Actually, the 1.2 isn't the interesting part here. It's the 100,20, because of
> scaling.

Ooh didn't see the scaling, ok cool :)

[commit-bot: I haz the power, 2014-09-10 18:38:26]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/sky@chromium.org/557953004/1

[commit-bot: I haz the power, 2014-09-10 21:05:52]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-09-10 21:05:52]

Exceeded time limit waiting for builds to trigger.

[sky, 2014-09-10 21:15:28]

The CQ bit was checked by sky@chromium.org

[commit-bot: I haz the power, 2014-09-10 21:21:37]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/sky@chromium.org/557953004/1

[commit-bot: I haz the power, 2014-09-10 22:20:37]

Committed patchset #1 (id:1) as b02ef65069406732afaf800d76890fae5f7c9875

[commit-bot: I haz the power, 2014-09-10 22:26:31]

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