Add inline capacity to CSSTokenizer::m_blockStack.

third_party/WebKit/Source/core/css/parser/CSSTokenizer.cpp

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "core/css/parser/CSSTokenizer.h"
 
 namespace blink {
 #include "core/CSSTokenizerCodepoints.cpp"
 }
 
@@ skipping 107 matching lines @@
 {
     m_input.advance(offset);
 }
 
 CSSParserToken CSSTokenizer::whiteSpace(UChar cc)
 {
     m_input.advanceUntilNonWhitespace();
     return CSSParserToken(WhitespaceToken);
 }
 
-static bool popIfBlockMatches(Vector<CSSParserTokenType>& blockStack, CSSParserTokenType type)

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

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.

-{
-    if (!blockStack.isEmpty() && blockStack.last() == type) {
-        blockStack.removeLast();
-        return true;
-    }
-    return false;
-}
-
 CSSParserToken CSSTokenizer::blockStart(CSSParserTokenType type)
 {
     m_blockStack.append(type);
     return CSSParserToken(type, CSSParserToken::BlockStart);
 }
 
 CSSParserToken CSSTokenizer::blockStart(CSSParserTokenType blockType, CSSParserTokenType type, StringView name)
 {
     m_blockStack.append(blockType);
     return CSSParserToken(type, name, CSSParserToken::BlockStart);
 }
 
 CSSParserToken CSSTokenizer::blockEnd(CSSParserTokenType type, CSSParserTokenType startType)
 {
-    if (popIfBlockMatches(m_blockStack, startType))
+    if (!m_blockStack.isEmpty() && m_blockStack.last() == type) {
+        m_blockStack.removeLast();
         return CSSParserToken(type, CSSParserToken::BlockEnd);
+    }
     return CSSParserToken(type);
 }
 
 CSSParserToken CSSTokenizer::leftParenthesis(UChar cc)
 {
     return blockStart(LeftParenthesisToken);
 }
 
 CSSParserToken CSSTokenizer::rightParenthesis(UChar cc)
 {
@@ skipping 602 matching lines @@
     return areIdentifier;
 }
 
 StringView CSSTokenizer::registerString(const String& string)
 {
     m_scope.storeString(string);
     return string;
 }
 
 } // namespace blink