Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(224)

Side by Side Diff: third_party/WebKit/Source/core/html/parser/HTMLTreeBuilderSimulator.cpp

Issue 2386893002: Reformat comments in core/html/parser (Closed)
Patch Set: self review Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2013 Google, Inc. All Rights Reserved. 2 * Copyright (C) 2013 Google, Inc. All Rights Reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions 5 * modification, are permitted provided that the following conditions
6 * are met: 6 * are met:
7 * 1. Redistributions of source code must retain the above copyright 7 * 1. Redistributions of source code must retain the above copyright
8 * notice, this list of conditions and the following disclaimer. 8 * notice, this list of conditions and the following disclaimer.
9 * 2. Redistributions in binary form must reproduce the above copyright 9 * 2. Redistributions in binary form must reproduce the above copyright
10 * notice, this list of conditions and the following disclaimer in the 10 * notice, this list of conditions and the following disclaimer in the
(...skipping 19 matching lines...) Expand all
30 #include "core/SVGNames.h" 30 #include "core/SVGNames.h"
31 #include "core/html/parser/HTMLParserIdioms.h" 31 #include "core/html/parser/HTMLParserIdioms.h"
32 #include "core/html/parser/HTMLTokenizer.h" 32 #include "core/html/parser/HTMLTokenizer.h"
33 #include "core/html/parser/HTMLTreeBuilder.h" 33 #include "core/html/parser/HTMLTreeBuilder.h"
34 34
35 namespace blink { 35 namespace blink {
36 36
37 using namespace HTMLNames; 37 using namespace HTMLNames;
38 38
39 static bool tokenExitsForeignContent(const CompactHTMLToken& token) { 39 static bool tokenExitsForeignContent(const CompactHTMLToken& token) {
40 // FIXME: This is copied from HTMLTreeBuilder::processTokenInForeignContent an d changed to use threadSafeHTMLNamesMatch. 40 // FIXME: This is copied from HTMLTreeBuilder::processTokenInForeignContent
41 // and changed to use threadSafeHTMLNamesMatch.
41 const String& tagName = token.data(); 42 const String& tagName = token.data();
42 return threadSafeMatch(tagName, bTag) || threadSafeMatch(tagName, bigTag) || 43 return threadSafeMatch(tagName, bTag) || threadSafeMatch(tagName, bigTag) ||
43 threadSafeMatch(tagName, blockquoteTag) || 44 threadSafeMatch(tagName, blockquoteTag) ||
44 threadSafeMatch(tagName, bodyTag) || threadSafeMatch(tagName, brTag) || 45 threadSafeMatch(tagName, bodyTag) || threadSafeMatch(tagName, brTag) ||
45 threadSafeMatch(tagName, centerTag) || 46 threadSafeMatch(tagName, centerTag) ||
46 threadSafeMatch(tagName, codeTag) || threadSafeMatch(tagName, ddTag) || 47 threadSafeMatch(tagName, codeTag) || threadSafeMatch(tagName, ddTag) ||
47 threadSafeMatch(tagName, divTag) || threadSafeMatch(tagName, dlTag) || 48 threadSafeMatch(tagName, divTag) || threadSafeMatch(tagName, dlTag) ||
48 threadSafeMatch(tagName, dtTag) || threadSafeMatch(tagName, emTag) || 49 threadSafeMatch(tagName, dtTag) || threadSafeMatch(tagName, emTag) ||
49 threadSafeMatch(tagName, embedTag) || 50 threadSafeMatch(tagName, embedTag) ||
50 threadSafeMatch(tagName, h1Tag) || threadSafeMatch(tagName, h2Tag) || 51 threadSafeMatch(tagName, h1Tag) || threadSafeMatch(tagName, h2Tag) ||
(...skipping 16 matching lines...) Expand all
67 threadSafeMatch(tagName, tableTag) || 68 threadSafeMatch(tagName, tableTag) ||
68 threadSafeMatch(tagName, ttTag) || threadSafeMatch(tagName, uTag) || 69 threadSafeMatch(tagName, ttTag) || threadSafeMatch(tagName, uTag) ||
69 threadSafeMatch(tagName, ulTag) || threadSafeMatch(tagName, varTag) || 70 threadSafeMatch(tagName, ulTag) || threadSafeMatch(tagName, varTag) ||
70 (threadSafeMatch(tagName, fontTag) && 71 (threadSafeMatch(tagName, fontTag) &&
71 (token.getAttributeItem(colorAttr) || 72 (token.getAttributeItem(colorAttr) ||
72 token.getAttributeItem(faceAttr) || 73 token.getAttributeItem(faceAttr) ||
73 token.getAttributeItem(sizeAttr))); 74 token.getAttributeItem(sizeAttr)));
74 } 75 }
75 76
76 static bool tokenExitsSVG(const CompactHTMLToken& token) { 77 static bool tokenExitsSVG(const CompactHTMLToken& token) {
77 // FIXME: It's very fragile that we special case foreignObject here to be case -insensitive. 78 // FIXME: It's very fragile that we special case foreignObject here to be
79 // case-insensitive.
78 return equalIgnoringCase(token.data(), 80 return equalIgnoringCase(token.data(),
79 SVGNames::foreignObjectTag.localName()); 81 SVGNames::foreignObjectTag.localName());
80 } 82 }
81 83
82 static bool tokenExitsMath(const CompactHTMLToken& token) { 84 static bool tokenExitsMath(const CompactHTMLToken& token) {
83 // FIXME: This is copied from HTMLElementStack::isMathMLTextIntegrationPoint a nd changed to use threadSafeMatch. 85 // FIXME: This is copied from HTMLElementStack::isMathMLTextIntegrationPoint
86 // and changed to use threadSafeMatch.
84 const String& tagName = token.data(); 87 const String& tagName = token.data();
85 return threadSafeMatch(tagName, MathMLNames::miTag) || 88 return threadSafeMatch(tagName, MathMLNames::miTag) ||
86 threadSafeMatch(tagName, MathMLNames::moTag) || 89 threadSafeMatch(tagName, MathMLNames::moTag) ||
87 threadSafeMatch(tagName, MathMLNames::mnTag) || 90 threadSafeMatch(tagName, MathMLNames::mnTag) ||
88 threadSafeMatch(tagName, MathMLNames::msTag) || 91 threadSafeMatch(tagName, MathMLNames::msTag) ||
89 threadSafeMatch(tagName, MathMLNames::mtextTag); 92 threadSafeMatch(tagName, MathMLNames::mtextTag);
90 } 93 }
91 94
92 HTMLTreeBuilderSimulator::HTMLTreeBuilderSimulator( 95 HTMLTreeBuilderSimulator::HTMLTreeBuilderSimulator(
93 const HTMLParserOptions& options) 96 const HTMLParserOptions& options)
(...skipping 31 matching lines...) Expand 10 before | Expand all | Expand 10 after
125 if (threadSafeMatch(tagName, SVGNames::svgTag)) 128 if (threadSafeMatch(tagName, SVGNames::svgTag))
126 m_namespaceStack.append(SVG); 129 m_namespaceStack.append(SVG);
127 if (threadSafeMatch(tagName, MathMLNames::mathTag)) 130 if (threadSafeMatch(tagName, MathMLNames::mathTag))
128 m_namespaceStack.append(MathML); 131 m_namespaceStack.append(MathML);
129 if (inForeignContent() && tokenExitsForeignContent(token)) 132 if (inForeignContent() && tokenExitsForeignContent(token))
130 m_namespaceStack.removeLast(); 133 m_namespaceStack.removeLast();
131 if ((m_namespaceStack.last() == SVG && tokenExitsSVG(token)) || 134 if ((m_namespaceStack.last() == SVG && tokenExitsSVG(token)) ||
132 (m_namespaceStack.last() == MathML && tokenExitsMath(token))) 135 (m_namespaceStack.last() == MathML && tokenExitsMath(token)))
133 m_namespaceStack.append(HTML); 136 m_namespaceStack.append(HTML);
134 if (!inForeignContent()) { 137 if (!inForeignContent()) {
135 // FIXME: This is just a copy of Tokenizer::updateStateFor which uses thre adSafeMatches. 138 // FIXME: This is just a copy of Tokenizer::updateStateFor which uses
139 // threadSafeMatches.
136 if (threadSafeMatch(tagName, textareaTag) || 140 if (threadSafeMatch(tagName, textareaTag) ||
137 threadSafeMatch(tagName, titleTag)) { 141 threadSafeMatch(tagName, titleTag)) {
138 tokenizer->setState(HTMLTokenizer::RCDATAState); 142 tokenizer->setState(HTMLTokenizer::RCDATAState);
139 } else if (threadSafeMatch(tagName, plaintextTag)) { 143 } else if (threadSafeMatch(tagName, plaintextTag)) {
140 tokenizer->setState(HTMLTokenizer::PLAINTEXTState); 144 tokenizer->setState(HTMLTokenizer::PLAINTEXTState);
141 } else if (threadSafeMatch(tagName, scriptTag)) { 145 } else if (threadSafeMatch(tagName, scriptTag)) {
142 tokenizer->setState(HTMLTokenizer::ScriptDataState); 146 tokenizer->setState(HTMLTokenizer::ScriptDataState);
143 simulatedToken = ScriptStart; 147 simulatedToken = ScriptStart;
144 } else if (threadSafeMatch(tagName, styleTag) || 148 } else if (threadSafeMatch(tagName, styleTag) ||
145 threadSafeMatch(tagName, iframeTag) || 149 threadSafeMatch(tagName, iframeTag) ||
(...skipping 26 matching lines...) Expand all
172 } 176 }
173 } 177 }
174 178
175 // FIXME: Also setForceNullCharacterReplacement when in text mode. 179 // FIXME: Also setForceNullCharacterReplacement when in text mode.
176 tokenizer->setForceNullCharacterReplacement(inForeignContent()); 180 tokenizer->setForceNullCharacterReplacement(inForeignContent());
177 tokenizer->setShouldAllowCDATA(inForeignContent()); 181 tokenizer->setShouldAllowCDATA(inForeignContent());
178 return simulatedToken; 182 return simulatedToken;
179 } 183 }
180 184
181 } // namespace blink 185 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698