Encapsulate the CSSPrimitiveValue UnitType trie bethind a method that takes a StringView.

Encapsulate the CSSPrimitiveValue UnitType trie bethind a method that takes a StringView.

This removes the need for callers to manually branch on is8Bit() and
also avoids having these separate functions in a header that's harder
to discover.

Now CSSPrimitiveValue has both unitTypeToString and stringToUnitType.

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

[esprehn, 2016-06-09 22:56:22]

Description was changed from

==========
Encapsulate the CSSPrimitiveValue UnitType trie bethind a method that takes a
StringView.

This removes the need for callers to manually branch on is8Bit() and
also avoids leaking the detail that we're using a trie here.

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

to

==========
Encapsulate the CSSPrimitiveValue UnitType trie bethind a method that takes a
StringView.

This removes the need for callers to manually branch on is8Bit() and
also avoids having these separate functions in a header that's harder
to discover.

Now CSSPrimitiveValue has both unitTypeToString and stringToUnitType.

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

[esprehn, 2016-06-09 22:56:25]

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

[commit-bot: I haz the power, 2016-06-09 22:56:45]

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

[esprehn, 2016-06-09 23:50:36]

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

[meade_UTC10, 2016-06-10 00:01:41]

lgtm

[esprehn, 2016-06-10 00:02:02]

The CQ bit was unchecked by esprehn@chromium.org

[esprehn, 2016-06-10 00:02:05]

The CQ bit was checked by esprehn@chromium.org

[commit-bot: I haz the power, 2016-06-10 00:02:44]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2051313002/1

[commit-bot: I haz the power, 2016-06-10 00:58:44]

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

[commit-bot: I haz the power, 2016-06-10 00:58:45]

Try jobs failed on following builders:
  mac_chromium_compile_dbg_ng on tryserver.chromium.mac (JOB_TIMED_OUT, no build
URL)
  mac_chromium_gn_rel on tryserver.chromium.mac (JOB_TIMED_OUT, no build URL)
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_TIMED_OUT, no build URL)

[esprehn, 2016-06-10 02:23:17]

The CQ bit was checked by esprehn@chromium.org

[commit-bot: I haz the power, 2016-06-10 02:23:43]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2051313002/1

[commit-bot: I haz the power, 2016-06-10 04:30:50]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-10 04:31:06]

CQ bit was unchecked

[commit-bot: I haz the power, 2016-06-10 04:33:34]

Description was changed from

==========
Encapsulate the CSSPrimitiveValue UnitType trie bethind a method that takes a
StringView.

This removes the need for callers to manually branch on is8Bit() and
also avoids having these separate functions in a header that's harder
to discover.

Now CSSPrimitiveValue has both unitTypeToString and stringToUnitType.

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

to

==========
Encapsulate the CSSPrimitiveValue UnitType trie bethind a method that takes a
StringView.

This removes the need for callers to manually branch on is8Bit() and
also avoids having these separate functions in a header that's harder
to discover.

Now CSSPrimitiveValue has both unitTypeToString and stringToUnitType.

BUG=615174

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

[commit-bot: I haz the power, 2016-06-10 04:33:34]

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