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.


Canonical link: https://commits.webkit.org/173164@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@197630 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog

@@ -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.

Source/WebCore/dom/CustomElementDefinitions.cpp

@@ -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

Source/WebCore/dom/LifecycleCallbackQueue.h

@@ -28,18 +28,17 @@
 
 #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);