Avoid truncation to int when setting background-position coordinates

Avoid truncation to int when setting background-position coordinates

This error was part of the original
https://bugs.webkit.org/show_bug.cgi?id=103440 implementation of
background-position but then only for non-repeating backgrounds. When
also used for repeating backgrounds through
https://codereview.chromium.org/1287293002 it has been noticed that
the background-position was sometimes a pixel off when zooming.

BUG=535504
Committed: https://crrev.com/25015ec55dfc1a4719c4f7e1c8448dcac6eea9f1
Cr-Commit-Position: refs/heads/master@{#359101}

[davve, 2015-11-10 11:19:04]

The CQ bit was checked by davve@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2015-11-10 11:19:15]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1431103005/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1431103005/1

[davve, 2015-11-10 11:20:09]

davve@opera.com changed reviewers:
+ fs@opera.com, pdr@chromium.org

[davve, 2015-11-10 11:20:09]

See, I get to do fun stuff while you're slaving away at BlinkOn. Have a good
one. PTAL.

[commit-bot: I haz the power, 2015-11-10 12:11:37]

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

[commit-bot: I haz the power, 2015-11-10 12:11:37]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[davve, 2015-11-10 12:47:53]

On 2015/11/10 12:11:37, commit-bot: I haz the power wrote:
> Dry run: Try jobs failed on following builders:
>   win_chromium_rel_ng on tryserver.chromium.win (JOB_FAILED,
>
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

Found an issue with the no-repeat case. Will re-upload in a while.

[davve, 2015-11-10 14:13:48]

The CQ bit was checked by davve@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2015-11-10 14:13:54]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1431103005/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1431103005/20001

[commit-bot: I haz the power, 2015-11-10 15:31:55]

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

[commit-bot: I haz the power, 2015-11-10 15:31:55]

Dry run: Try jobs failed on following builders:
  win_chromium_x64_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[davve, 2015-11-11 08:39:57]

On 2015/11/10 15:31:55, commit-bot: I haz the power wrote:
> Dry run: Try jobs failed on following builders:
>   win_chromium_x64_rel_ng on tryserver.chromium.win (JOB_FAILED,
>
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

win_chromium_x64_rel_ng seems to feel better now.

[fs, 2015-11-11 13:40:20]

lgtm

[davve, 2015-11-11 15:01:18]

Description was changed from

==========
Avoid truncation to int when setting background-position coordinates

This error was part of the original
https://bugs.webkit.org/show_bug.cgi?id=103440 implementation of
background-position but then only for non-repeating backgrounds. When
also used for repeating backgrounds through
https://codereview.chromium.org/1287293002 it has been noticed that
the background-position was sometimes off.

BUG=535504
==========

to

==========
Avoid truncation to int when setting background-position coordinates

This error was part of the original
https://bugs.webkit.org/show_bug.cgi?id=103440 implementation of
background-position but then only for non-repeating backgrounds. When
also used for repeating backgrounds through
https://codereview.chromium.org/1287293002 it has been noticed that
the background-position was sometimes a pixel off when zooming.

BUG=535504
==========

[davve, 2015-11-11 15:01:33]

The CQ bit was checked by davve@opera.com

[commit-bot: I haz the power, 2015-11-11 15:01:42]

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

[commit-bot: I haz the power, 2015-11-11 16:57:15]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-11-12 19:58:02]

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