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

Side by Side Diff: Source/core/dom/QualifiedName.cpp

Issue 485053002: Account for MathML tags / attributes in staticQualifiedNamesCount (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Sort alphabetically Created 6 years, 4 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 | Annotate | Revision Log
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2005, 2006, 2009 Apple Inc. All rights reserved. 2 * Copyright (C) 2005, 2006, 2009 Apple Inc. All rights reserved.
3 * 3 *
4 * This library is free software; you can redistribute it and/or 4 * This library is free software; you can redistribute it and/or
5 * modify it under the terms of the GNU Library General Public 5 * modify it under the terms of the GNU Library General Public
6 * License as published by the Free Software Foundation; either 6 * License as published by the Free Software Foundation; either
7 * version 2 of the License, or (at your option) any later version. 7 * version 2 of the License, or (at your option) any later version.
8 * 8 *
9 * This library is distributed in the hope that it will be useful, 9 * This library is distributed in the hope that it will be useful,
10 * but WITHOUT ANY WARRANTY; without even the implied warranty of 10 * but WITHOUT ANY WARRANTY; without even the implied warranty of
11 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU 11 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
12 * Library General Public License for more details. 12 * Library General Public License for more details.
13 * 13 *
14 * You should have received a copy of the GNU Library General Public License 14 * You should have received a copy of the GNU Library General Public License
15 * along with this library; see the file COPYING.LIB. If not, write to 15 * along with this library; see the file COPYING.LIB. If not, write to
16 * the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor, 16 * the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
17 * Boston, MA 02110-1301, USA. 17 * Boston, MA 02110-1301, USA.
18 */ 18 */
19 19
20 #include "config.h" 20 #include "config.h"
21 21
22 #include "core/dom/QualifiedName.h" 22 #include "core/dom/QualifiedName.h"
23 23
24 #include "core/HTMLNames.h" 24 #include "core/HTMLNames.h"
25 #include "core/MathMLNames.h"
25 #include "core/SVGNames.h" 26 #include "core/SVGNames.h"
26 #include "core/XLinkNames.h" 27 #include "core/XLinkNames.h"
27 #include "core/XMLNSNames.h" 28 #include "core/XMLNSNames.h"
28 #include "core/XMLNames.h" 29 #include "core/XMLNames.h"
29 #include "wtf/Assertions.h" 30 #include "wtf/Assertions.h"
30 #include "wtf/HashSet.h" 31 #include "wtf/HashSet.h"
31 #include "wtf/MainThread.h" 32 #include "wtf/MainThread.h"
32 #include "wtf/StaticConstructors.h" 33 #include "wtf/StaticConstructors.h"
33 34
34 namespace blink { 35 namespace blink {
35 36
36 static const int staticQualifiedNamesCount = HTMLNames::HTMLTagsCount + HTMLName s::HTMLAttrsCount 37 static const int staticQualifiedNamesCount = HTMLNames::HTMLTagsCount + HTMLName s::HTMLAttrsCount
38 + MathMLNames::MathMLTagsCount + MathMLNames::MathMLAttrsCount
eseidel 2014/08/20 17:06:23 This seems pretty brittle. Can we add an ASSERT s
37 + SVGNames::SVGTagsCount + SVGNames::SVGAttrsCount 39 + SVGNames::SVGTagsCount + SVGNames::SVGAttrsCount
38 + XLinkNames::XLinkAttrsCount 40 + XLinkNames::XLinkAttrsCount
39 + XMLNSNames::XMLNSAttrsCount 41 + XMLNSNames::XMLNSAttrsCount
40 + XMLNames::XMLAttrsCount; 42 + XMLNames::XMLAttrsCount;
41 43
42 struct QualifiedNameHashTraits : public HashTraits<QualifiedName::QualifiedNameI mpl*> { 44 struct QualifiedNameHashTraits : public HashTraits<QualifiedName::QualifiedNameI mpl*> {
43 static const unsigned minimumTableSize = WTF::HashTableCapacityForSize<stati cQualifiedNamesCount>::value; 45 static const unsigned minimumTableSize = WTF::HashTableCapacityForSize<stati cQualifiedNamesCount>::value;
44 }; 46 };
45 47
46 typedef HashSet<QualifiedName::QualifiedNameImpl*, QualifiedNameHash, QualifiedN ameHashTraits> QualifiedNameCache; 48 typedef HashSet<QualifiedName::QualifiedNameImpl*, QualifiedNameHash, QualifiedN ameHashTraits> QualifiedNameCache;
(...skipping 87 matching lines...) Expand 10 before | Expand all | Expand 10 after
134 { 136 {
135 new (targetAddress) QualifiedName(nullAtom, AtomicString(name), nameNamespac e, true); 137 new (targetAddress) QualifiedName(nullAtom, AtomicString(name), nameNamespac e, true);
136 } 138 }
137 139
138 void QualifiedName::createStatic(void* targetAddress, StringImpl* name) 140 void QualifiedName::createStatic(void* targetAddress, StringImpl* name)
139 { 141 {
140 new (targetAddress) QualifiedName(nullAtom, AtomicString(name), nullAtom, tr ue); 142 new (targetAddress) QualifiedName(nullAtom, AtomicString(name), nullAtom, tr ue);
141 } 143 }
142 144
143 } 145 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698