Title: [155967] trunk/Source/WebCore
- Revision
- 155967
- Author
- da...@apple.com
- Date
- 2013-09-17 10:07:16 -0700 (Tue, 17 Sep 2013)
Log Message
Get rid of a couple unneeded NeverDestroyed::get() calls
https://bugs.webkit.org/show_bug.cgi?id=121500
Reviewed by Andreas Kling.
* css/CSSSelector.cpp:
(WebCore::CSSSelector::parsePseudoType): No need to call get() here.
* dom/make_names.pl:
(printFactoryCppFile): Ditto.
(printWrapperFactoryCppFile): Ditto.
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (155966 => 155967)
--- trunk/Source/WebCore/ChangeLog 2013-09-17 17:04:36 UTC (rev 155966)
+++ trunk/Source/WebCore/ChangeLog 2013-09-17 17:07:16 UTC (rev 155967)
@@ -1,3 +1,16 @@
+2013-09-17 Darin Adler <da...@apple.com>
+
+ Get rid of a couple unneeded NeverDestroyed::get() calls
+ https://bugs.webkit.org/show_bug.cgi?id=121500
+
+ Reviewed by Andreas Kling.
+
+ * css/CSSSelector.cpp:
+ (WebCore::CSSSelector::parsePseudoType): No need to call get() here.
+ * dom/make_names.pl:
+ (printFactoryCppFile): Ditto.
+ (printWrapperFactoryCppFile): Ditto.
+
2013-09-17 Hans Muller <hmul...@adobe.com>
Bad ASSERT() in RasterShapeIntervals::firstIncludedIntervalY()
Modified: trunk/Source/WebCore/css/CSSSelector.cpp (155966 => 155967)
--- trunk/Source/WebCore/css/CSSSelector.cpp 2013-09-17 17:04:36 UTC (rev 155966)
+++ trunk/Source/WebCore/css/CSSSelector.cpp 2013-09-17 17:07:16 UTC (rev 155967)
@@ -366,7 +366,7 @@
static NeverDestroyed<HashMap<AtomicString, CSSSelector::PseudoType>> types;
if (types.get().isEmpty())
- populatePseudoTypeByNameMap(types.get());
+ populatePseudoTypeByNameMap(types);
if (PseudoType type = types.get().get(name))
return type;
Modified: trunk/Source/WebCore/dom/make_names.pl (155966 => 155967)
--- trunk/Source/WebCore/dom/make_names.pl 2013-09-17 17:04:36 UTC (rev 155966)
+++ trunk/Source/WebCore/dom/make_names.pl 2013-09-17 17:07:16 UTC (rev 155967)
@@ -978,7 +978,7 @@
static NeverDestroyed<HashMap<AtomicStringImpl*, $parameters{namespace}ConstructorFunction>> functions;
if (functions.get().isEmpty())
- populate$parameters{namespace}FactoryMap(functions.get());
+ populate$parameters{namespace}FactoryMap(functions);
if ($parameters{namespace}ConstructorFunction function = functions.get().get(name.localName().impl())) {
END
;
@@ -1212,7 +1212,7 @@
{
static NeverDestroyed<HashMap<AtomicStringImpl*, Create$parameters{namespace}ElementWrapperFunction>> functions;
if (functions.get().isEmpty())
- populate$parameters{namespace}WrapperMap(functions.get());
+ populate$parameters{namespace}WrapperMap(functions);
if (auto function = functions.get().get(element->localName().impl()))
return function(exec, globalObject, element);
return CREATE_DOM_WRAPPER(exec, globalObject, $parameters{fallbackJSInterfaceName}, element.get());
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes
