Simplify AnimationTestHelper.h

Simplify AnimationTestHelper.h

We don't need these extra helper functions and we can use StringView to
avoid allocating temporary strings.

BUG=615174
Committed: https://crrev.com/572944fef3414bbbc946e732afca05e80813ba95
Cr-Commit-Position: refs/heads/master@{#415064}

[esprehn, 2016-08-29 19:36:24]

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

[commit-bot: I haz the power, 2016-08-29 20:25:24]

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

[esprehn, 2016-08-29 22:26:27]

esprehn@chromium.org changed reviewers:
+ haraken@chromium.org, suzyh@chromium.org

[esprehn, 2016-08-29 22:26:36]

Description was changed from

==========
Simplify AnimationTestHelper.h

BUG=
==========

to

==========
Simplify AnimationTestHelper.h
==========

[esprehn, 2016-08-29 22:27:26]

Description was changed from

==========
Simplify AnimationTestHelper.h
==========

to

==========
Simplify AnimationTestHelper.h

We don't need these extra helper functions and we can use StringView to
avoid allocating temporary strings.
==========

[esprehn, 2016-08-29 22:27:42]

Description was changed from

==========
Simplify AnimationTestHelper.h

We don't need these extra helper functions and we can use StringView to
avoid allocating temporary strings.
==========

to

==========
Simplify AnimationTestHelper.h

We don't need these extra helper functions and we can use StringView to
avoid allocating temporary strings.

BUG=615174
==========

[haraken, 2016-08-29 22:27:50]

LGTM

[esprehn, 2016-08-29 22:28:08]

The CQ bit was unchecked by esprehn@chromium.org

[esprehn, 2016-08-29 22:28:09]

The CQ bit was checked by esprehn@chromium.org

[commit-bot: I haz the power, 2016-08-30 02:11:07]

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-08-30 03:57:21]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-30 03:59:56]

Description was changed from

==========
Simplify AnimationTestHelper.h

We don't need these extra helper functions and we can use StringView to
avoid allocating temporary strings.

BUG=615174
==========

to

==========
Simplify AnimationTestHelper.h

We don't need these extra helper functions and we can use StringView to
avoid allocating temporary strings.

BUG=615174

Committed: https://crrev.com/572944fef3414bbbc946e732afca05e80813ba95
Cr-Commit-Position: refs/heads/master@{#415064}
==========

[commit-bot: I haz the power, 2016-08-30 03:59:57]

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

[suzyh_UTC10 (ex-contributor), 2016-08-30 04:00:56]

Looks like I'm just too late to the party. But FWIW, lgtm, although I did wonder
whether the wtf/text/WTFString.h include in the header was still necessary.