Add inline capacity to CSSTokenizer::m_blockStack.

Add inline capacity to CSSTokenizer::m_blockStack.

The malloc on the m_blockStack shows up in profiles in Animometer when
parsing transforms since we push and pop from this stack for every
paren and transforms use function syntax like rotate(10deg). We can
instead use inline capacity to skip the malloc when parsing many inline
transforms, variables, or other things that use parens.

I chose 8 since it's unlikely that you'd nest more than 8 parens,
brackets or braces in CSS. We can easily make this number bigger someday
too since CSSParserTokenType is just an enum so the storage is very
small.

BUG=605792
Committed: https://crrev.com/53ab0155b7a113df1fcbeb1050df6fea27581685
Cr-Commit-Position: refs/heads/master@{#402401}

[esprehn, 2016-06-27 22:15:11]

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

[commit-bot: I haz the power, 2016-06-27 22:16:12]

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

[esprehn, 2016-06-27 22:16:17]

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

[esprehn, 2016-06-27 22:16:18]

https://codereview.chromium.org/2105463003/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/css/parser/CSSTokenizer.cpp (left):

https://codereview.chromium.org/2105463003/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/css/parser/CSSTokenizer.cpp:128: static bool
popIfBlockMatches(Vector<CSSParserTokenType>& blockStack, CSSParserTokenType
type)
I merged this into the function so I don't need a typedef or to repeat the
inline capacity. The function also doesn't seem like it makes the code more
clear, it's just hiding the comparison and the removeLast() call.

[meade_UTC10, 2016-06-27 22:22:35]

lgtm

[esprehn, 2016-06-27 23:16:32]

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

[commit-bot: I haz the power, 2016-06-27 23:17:33]

Dry run: 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-06-28 02:08:48]

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

[commit-bot: I haz the power, 2016-06-28 02:08:50]

Dry run: This issue passed the CQ dry run.

[esprehn, 2016-06-28 03:47:38]

The CQ bit was checked by esprehn@chromium.org

[esprehn, 2016-06-28 03:47:39]

The patchset sent to the CQ was uploaded after l-g-t-m from meade@chromium.org
Link to the patchset: https://codereview.chromium.org/2105463003/#ps20001
(title: "Fix typo.")

[commit-bot: I haz the power, 2016-06-28 03:47:59]

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-06-28 03:53:55]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-28 03:56:36]

Description was changed from

==========
Add inline capacity to CSSTokenizer::m_blockStack.

The malloc on the m_blockStack shows up in profiles in Animometer when
parsing transforms since we push and pop from this stack for every
paren and transforms use function syntax like rotate(10deg). We can
instead use inline capacity to skip the malloc when parsing many inline
transforms, variables, or other things that use parens.

I chose 8 since it's unlikely that you'd nest more than 8 parens,
brackets or braces in CSS. We can easily make this number bigger someday
too since CSSParserTokenType is just an enum so the storage is very
small.

BUG=605792
==========

to

==========
Add inline capacity to CSSTokenizer::m_blockStack.

The malloc on the m_blockStack shows up in profiles in Animometer when
parsing transforms since we push and pop from this stack for every
paren and transforms use function syntax like rotate(10deg). We can
instead use inline capacity to skip the malloc when parsing many inline
transforms, variables, or other things that use parens.

I chose 8 since it's unlikely that you'd nest more than 8 parens,
brackets or braces in CSS. We can easily make this number bigger someday
too since CSSParserTokenType is just an enum so the storage is very
small.

BUG=605792

Committed: https://crrev.com/53ab0155b7a113df1fcbeb1050df6fea27581685
Cr-Commit-Position: refs/heads/master@{#402401}
==========

[commit-bot: I haz the power, 2016-06-28 03:56:37]

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