The selection bounds should be clipped to the viewport

The selection bounds should be clipped to the viewport

This is another regression from
https://codereview.chromium.org/570763003 that wasn't
solved by https://codereview.chromium.org/598113003
as it missed more subtleties: most callers want the
clipped selection bounds. Only one caller actually
cared about the unclipped bounds:
FrameSelection::revealSelection as it needs to know
where the outside-the-viewport-selection is.

This change splits bounds() in 2, depending on
the intended behavior. While touching the code,
switched it to use LayoutRect as none of the
callers cares about FloatRect.

This change fixes
fast/transforms/selection-bounds-in-transformed-view.html
that got broken by using a fractional scroll offset.

Unfortunately no extra test as the issue requires
generating a gigantic drag-image that
DRT doesn't do.

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

[Julien - ping for review, 2014-10-09 15:03:35]

jchaffraix@chromium.org changed reviewers:
+ dsinclair@chromium.org, leviw@chromium.org, xianzhunzhu@chromium.org

[Julien - ping for review, 2014-10-09 15:03:35]

1-liner for the motivated!

[dsinclair, 2014-10-09 15:09:15]

The CQ bit was checked by dsinclair@chromium.org

[dsinclair, 2014-10-09 15:09:23]

lgtm

[commit-bot: I haz the power, 2014-10-09 15:10:33]

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

[commit-bot: I haz the power, 2014-10-09 16:31:38]

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

[commit-bot: I haz the power, 2014-10-09 16:31:39]

Try jobs failed on following builders:
  linux_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/2...)

[Julien - ping for review, 2014-10-14 20:03:35]

New change up for review, there has been quite some extra changes added as I
started to understand the code better.

[dsinclair, 2014-10-15 13:10:19]

The CQ bit was checked by dsinclair@chromium.org

[dsinclair, 2014-10-15 13:10:20]

lgtm

[commit-bot: I haz the power, 2014-10-15 13:10:54]

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

[commit-bot: I haz the power, 2014-10-15 13:52:03]

Committed patchset #4 (id:160001) as 183749