Remove enclosingIntRect(const IntRect&) and adjust all call sites.

Remove enclosingIntRect(const IntRect&) and adjust all call sites.

BUG=247934
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=152087

[pdr., 2013-06-09 02:48:03]

On 2013/06/09 02:38:35, jbroman wrote:

LGTM

[danakj, 2013-06-09 06:14:47]

Personally, I'd rather see the explicit enclosingIntRect stay, and instead
remove the implicit float->int conversion so that it is clear at all call
sites what is happening. This would match what we've done with gfx::RectF
and gfx::Rect.
On Jun 9, 2013 4:48 AM, <pdr@chromium.org> wrote:

> On 2013/06/09 02:38:35, jbroman wrote:
>
> LGTM
>
>
https://codereview.chromium.**org/16700002/<https://codereview.chromium.org/1...
>

[danakj1, 2013-06-09 06:16:12]

oh, from the bug I see this is about enclosingIntRect(IntRect) so please
disregard my last comment. :)
On Jun 9, 2013 8:14 AM, "Dana Jansens" <danakj@chromium.org> wrote:

> Personally, I'd rather see the explicit enclosingIntRect stay, and instead
> remove the implicit float->int conversion so that it is clear at all call
> sites what is happening. This would match what we've done with gfx::RectF
> and gfx::Rect.
> On Jun 9, 2013 4:48 AM, <pdr@chromium.org> wrote:
>
>> On 2013/06/09 02:38:35, jbroman wrote:
>>
>> LGTM
>>
>>
https://codereview.chromium.**org/16700002/<https://codereview.chromium.org/1...
>>
>

[commit-bot: I haz the power, 2013-06-09 13:45:07]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jbroman@chromium.org/16700002/2001

[commit-bot: I haz the power, 2013-06-09 17:15:03]

Change committed as 152087