Minor cleans up in custom elements' code per Darin's comments
authorrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 6 Mar 2016 04:25:32 +0000 (04:25 +0000)
committerrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 6 Mar 2016 04:25:32 +0000 (04:25 +0000)
https://bugs.webkit.org/show_bug.cgi?id=155081

Reviewed by Darin Adler.

Cleanup per Darin's comments.

* dom/CustomElementDefinitions.cpp:
(WebCore::CustomElementDefinitions::findInterface): Use get instead of explicitly checking against end().
* dom/LifecycleCallbackQueue.h: Remove unecessary header includes.

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@197630 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/dom/CustomElementDefinitions.cpp
Source/WebCore/dom/LifecycleCallbackQueue.h

index f27f955..05cf89b 100644 (file)
@@ -1,3 +1,16 @@
+2016-03-05  Ryosuke Niwa  <rniwa@webkit.org>
+
+        Minor cleans up in custom elements' code per Darin's comments
+        https://bugs.webkit.org/show_bug.cgi?id=155081
+
+        Reviewed by Darin Adler.
+
+        Cleanup per Darin's comments.
+
+        * dom/CustomElementDefinitions.cpp:
+        (WebCore::CustomElementDefinitions::findInterface): Use get instead of explicitly checking against end().
+        * dom/LifecycleCallbackQueue.h: Remove unecessary header includes.
+
 2016-03-05  Sam Weinig  <sam@webkit.org>
 
         Fix two minor typos from http://trac.webkit.org/changeset/197626 that were causing some test failures.
index 774cbfb..06d1920 100644 (file)
@@ -85,14 +85,12 @@ JSCustomElementInterface* CustomElementDefinitions::findInterface(const Qualifie
 
 JSCustomElementInterface* CustomElementDefinitions::findInterface(const AtomicString& name) const
 {
-    auto it = m_nameMap.find(name);
-    return it == m_nameMap.end() ? nullptr : it->value.get();
+    return m_nameMap.get(name);
 }
 
 JSCustomElementInterface* CustomElementDefinitions::findInterface(const JSC::JSObject* constructor) const
 {
-    auto it = m_constructorMap.find(constructor);
-    return it == m_constructorMap.end() ? nullptr : it->value;
+    return m_constructorMap.get(constructor);
 }
 
 bool CustomElementDefinitions::containsConstructor(const JSC::JSObject* constructor) const
index 0e9ca18..de3b18e 100644 (file)
 
 #if ENABLE(CUSTOM_ELEMENTS)
 
+#include <wtf/Forward.h>
 #include <wtf/Noncopyable.h>
-#include <wtf/RefCounted.h>
 #include <wtf/Vector.h>
-#include <wtf/text/AtomicString.h>
 
 namespace WebCore {
 
 class JSCustomElementInterface;
 class Document;
 class Element;
-class QualifiedName;
 class LifecycleQueueItem;
+class QualifiedName;
 
 class LifecycleCallbackQueue {
     WTF_MAKE_NONCOPYABLE(LifecycleCallbackQueue);