offset-rotate: Clamp rotation to float prior to assignment

offset-rotate: Clamp rotation to float prior to assignment

This change moves a clampTo<float>() call to before float assignment
rather than after. This is intended to keep float overflow detection
happy and has no behavioural changes.

BUG=675558
Committed: https://crrev.com/ea473ab14a34a428db2ff8bbcb75e3378530e3a4
Cr-Commit-Position: refs/heads/master@{#439962}

[alancutter (OOO until 2018), 2016-12-20 03:29:02]

Description was changed from

==========
offset-rotate: Clamp rotation to float prior to assignment

This change moves a clampTo<float>() call to before float assignment
rather than after. This is intended to keep float overflow detection
happy and has not behavioural changes.

BUG=675558
==========

to

==========
offset-rotate: Clamp rotation to float prior to assignment

This change moves a clampTo<float>() call to before float assignment
rather than after. This is intended to keep float overflow detection
happy and has no behavioural changes.

BUG=675558
==========

[alancutter (OOO until 2018), 2016-12-20 03:29:03]

alancutter@chromium.org changed reviewers:
+ sashab@chromium.org

[sashab, 2016-12-20 22:36:44]

Nice!! LGTM

https://codereview.chromium.org/2594493002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/css3/motion-path/motion-rotation-overflow-crash.html
(right):

https://codereview.chromium.org/2594493002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/css3/motion-path/motion-rotation-overflow-crash.html:5:
offset: path("M 2 9223372036854775640 h 112 v 18446744073709551478")
6772159099343311147963407119496477758278323944392504792520726726528445908706753060030057770863072007690525803528990028771578716997626504007399536732362342681220306922677405220428480102113641824539771557937974021684448585322739937495322648385679803197878862537591342648461579499923703493989594208073045976755073456313663842125268129997886257177874866252782866713636942669286588447097343808037614382288341687498451651245152153424054000300383726062592046846049867203248174598084843154rad
reverse 44px;
good choice of number. yup

[alancutter (OOO until 2018), 2016-12-20 23:42:07]

The CQ bit was checked by alancutter@chromium.org

[commit-bot: I haz the power, 2016-12-20 23:42:29]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-21 01:16:58]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1482277327412790, "parent_rev":
"7a29bdee72aa7574163ee3da90536edba55646eb", "commit_rev":
"6f7c4e18f4b435eca8c31af0daf15f53162d85a5"}

[commit-bot: I haz the power, 2016-12-21 01:17:45]

Description was changed from

==========
offset-rotate: Clamp rotation to float prior to assignment

This change moves a clampTo<float>() call to before float assignment
rather than after. This is intended to keep float overflow detection
happy and has no behavioural changes.

BUG=675558
==========

to

==========
offset-rotate: Clamp rotation to float prior to assignment

This change moves a clampTo<float>() call to before float assignment
rather than after. This is intended to keep float overflow detection
happy and has no behavioural changes.

BUG=675558

Review-Url: https://codereview.chromium.org/2594493002
==========

[commit-bot: I haz the power, 2016-12-21 01:17:45]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-21 01:19:24]

Description was changed from

==========
offset-rotate: Clamp rotation to float prior to assignment

This change moves a clampTo<float>() call to before float assignment
rather than after. This is intended to keep float overflow detection
happy and has no behavioural changes.

BUG=675558

Review-Url: https://codereview.chromium.org/2594493002
==========

to

==========
offset-rotate: Clamp rotation to float prior to assignment

This change moves a clampTo<float>() call to before float assignment
rather than after. This is intended to keep float overflow detection
happy and has no behavioural changes.

BUG=675558

Committed: https://crrev.com/ea473ab14a34a428db2ff8bbcb75e3378530e3a4
Cr-Commit-Position: refs/heads/master@{#439962}
==========

[commit-bot: I haz the power, 2016-12-21 01:19:24]

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