Fix whitespace handling in parseSimpleTransformList

Fix whitespace handling in parseSimpleTransformList

The loop would break if there was any trailing whitespace after the
args of the transform operation function, but this meant that if you
put a space between the functions like 'translateX(1px) translateY(1px)'
you'd fall out of the fast path when hitting the space between the
two functions. Any trailing space would similarly fall out of the fast
path and result in wasting all of the work building the transform list
only to discard it.

This patch instead skips whitespace between the functions and also adds
a bunch of tests for the fast paths.

BUG=606211
Committed: https://crrev.com/5f88403bf3e53a955cee48e172269d0ed94f2595
Cr-Commit-Position: refs/heads/master@{#395610}

[esprehn, 2016-05-24 05:09:04]

The CQ bit was checked by esprehn@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-05-24 05:09:18]

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

[esprehn, 2016-05-24 05:10:08]

The CQ bit was unchecked by esprehn@chromium.org

[esprehn, 2016-05-24 05:11:17]

The CQ bit was checked by esprehn@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-05-24 05:11:35]

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

[esprehn, 2016-05-24 06:00:57]

esprehn@chromium.org changed reviewers:
+ meade@chromium.org, timloh@chromium.org

[Timothy Loh, 2016-05-24 06:13:46]

On 2016/05/24 06:00:58, esprehn wrote:

lgtm

[commit-bot: I haz the power, 2016-05-24 09:20:31]

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

[commit-bot: I haz the power, 2016-05-24 09:20:32]

Dry run: This issue passed the CQ dry run.

[esprehn, 2016-05-24 16:02:15]

The CQ bit was checked by esprehn@chromium.org

[commit-bot: I haz the power, 2016-05-24 16:02:32]

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

[esprehn, 2016-05-24 16:02:35]

The CQ bit was unchecked by esprehn@chromium.org

[esprehn, 2016-05-24 16:03:14]

Description was changed from

==========
Fix whitespace handling in parseSimpleTransformList

The loop would break if there was any trailing whitespace after the
body of the transform operation function, but this meant that if you
put a space between the functions like 'translateX(1px) translateY(1px)'
you'd fall out of the fast path when hitting the space between the
two functions. Any trailing space would similarly fall out of the fast
path and result in wasting all of the work building the transform list
only to discard it.

This patch instead skips whitespace between the functions and also adds
a bunch of tests for the fast paths.

BUG=606211
==========

to

==========
Fix whitespace handling in parseSimpleTransformList

The loop would break if there was any trailing whitespace after the
args of the transform operation function, but this meant that if you
put a space between the functions like 'translateX(1px) translateY(1px)'
you'd fall out of the fast path when hitting the space between the
two functions. Any trailing space would similarly fall out of the fast
path and result in wasting all of the work building the transform list
only to discard it.

This patch instead skips whitespace between the functions and also adds
a bunch of tests for the fast paths.

BUG=606211
==========

[esprehn, 2016-05-24 16:03:43]

The CQ bit was checked by esprehn@chromium.org

[commit-bot: I haz the power, 2016-05-24 16:04:00]

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

[commit-bot: I haz the power, 2016-05-24 16:10:14]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-24 16:12:16]

Description was changed from

==========
Fix whitespace handling in parseSimpleTransformList

The loop would break if there was any trailing whitespace after the
args of the transform operation function, but this meant that if you
put a space between the functions like 'translateX(1px) translateY(1px)'
you'd fall out of the fast path when hitting the space between the
two functions. Any trailing space would similarly fall out of the fast
path and result in wasting all of the work building the transform list
only to discard it.

This patch instead skips whitespace between the functions and also adds
a bunch of tests for the fast paths.

BUG=606211
==========

to

==========
Fix whitespace handling in parseSimpleTransformList

The loop would break if there was any trailing whitespace after the
args of the transform operation function, but this meant that if you
put a space between the functions like 'translateX(1px) translateY(1px)'
you'd fall out of the fast path when hitting the space between the
two functions. Any trailing space would similarly fall out of the fast
path and result in wasting all of the work building the transform list
only to discard it.

This patch instead skips whitespace between the functions and also adds
a bunch of tests for the fast paths.

BUG=606211

Committed: https://crrev.com/5f88403bf3e53a955cee48e172269d0ed94f2595
Cr-Commit-Position: refs/heads/master@{#395610}
==========

[commit-bot: I haz the power, 2016-05-24 16:12:17]

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