Revert "Subpixel precision for clientWidth, offsetWidth, scrollTop et al"

Revert "Subpixel precision for clientWidth, offsetWidth, scrollTop et al"

Revert back to full pixel precision for the clientHeight, clientWidth,
clientLeft, clientTop, scrollTop, scrollLeft, scrollWidth, scrollHeight,
offsetWidth, offsetHeight, offsetTop, and offsetLeft properties.

Turns out the world isn't quite ready for these to return values with subpixel
precision.

This reverts r173426 [1].

BUG=378052, 382889, 380146, 386500
R=pdr@chromium.org

1: https://src.chromium.org/viewvc/blink?revision=173426&view=revision

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

[eae, 2014-06-19 00:23:44]

This is a fix for three blocking bugs so a review at some point would be great.

[pdr., 2014-06-19 00:33:40]

On 2014/06/19 00:23:44, eae wrote:
> This is a fix for three blocking bugs so a review at some point would be
great.

Japhet is out on paternity leave and I've been in meetings all afternoon. I will
review this by EOD.

[eae, 2014-06-19 00:36:37]

Thanks pdr!

To unsubscribe from this group and stop receiving emails from it, send an email
to blink-reviews+unsubscribe@chromium.org.

[pdr., 2014-06-19 03:43:59]

On 2014/06/19 00:36:37, eae wrote:
> Thanks pdr!
> 
> To unsubscribe from this group and stop receiving emails from it, send an
email
> to mailto:blink-reviews+unsubscribe@chromium.org.

Sorry for the delay. Was on my phone before and didn't realize that this is a
straightforward revert.

LGTM


Could you please update the blink-dev thread[1] and the entry on the feature
dashboard saying that this has been reverted?

[1]
https://groups.google.com/a/chromium.org/forum/#!msg/blink-dev/_Q7A4AQBFKY/S4...
[2] http://www.chromestatus.com/features/5497402177880064

[pdr., 2014-06-19 03:44:16]

The CQ bit was checked by pdr@chromium.org

[commit-bot: I haz the power, 2014-06-19 03:45:29]

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

[commit-bot: I haz the power, 2014-06-19 04:38:08]

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

[commit-bot: I haz the power, 2014-06-19 04:38:09]

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

[pdr., 2014-06-19 04:44:04]

+cc haraken and/or nbarth for Source/bindings

[eseidel, 2014-06-19 17:02:11]

rslgtm

[eae, 2014-06-19 17:07:25]

The CQ bit was checked by eae@chromium.org

[commit-bot: I haz the power, 2014-06-19 17:08:32]

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

[commit-bot: I haz the power, 2014-06-19 18:43:40]

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

[commit-bot: I haz the power, 2014-06-19 18:43:42]

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

[eae, 2014-06-19 19:42:40]

The CQ bit was checked by eae@chromium.org

[commit-bot: I haz the power, 2014-06-19 19:42:54]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/eae@chromium.org/340903002/20001

[commit-bot: I haz the power, 2014-06-19 20:48:44]

Change committed as 176545