Title: [154685] trunk/Source/WebCore
Revision
154685
Author
[email protected]
Date
2013-08-27 08:06:04 -0700 (Tue, 27 Aug 2013)

Log Message

HTMLAppletElement: Use child iterator to walk <param> children.
<https://webkit.org/b/120361>

Reviewed by Antti Koivisto.

Take Antti's fancy new child iterator for a quick spin.

* html/HTMLAppletElement.cpp:
(WebCore::HTMLAppletElement::updateWidget):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (154684 => 154685)


--- trunk/Source/WebCore/ChangeLog	2013-08-27 15:04:22 UTC (rev 154684)
+++ trunk/Source/WebCore/ChangeLog	2013-08-27 15:06:04 UTC (rev 154685)
@@ -1,3 +1,15 @@
+2013-08-27  Andreas Kling  <[email protected]>
+
+        HTMLAppletElement: Use child iterator to walk <param> children.
+        <https://webkit.org/b/120361>
+
+        Reviewed by Antti Koivisto.
+
+        Take Antti's fancy new child iterator for a quick spin.
+
+        * html/HTMLAppletElement.cpp:
+        (WebCore::HTMLAppletElement::updateWidget):
+
 2013-08-27  Andre Moreira Magalhaes   <[email protected]>
 
         [gstreamer] Make sure gstreamer source element is thread-safe

Modified: trunk/Source/WebCore/html/HTMLAppletElement.cpp (154684 => 154685)


--- trunk/Source/WebCore/html/HTMLAppletElement.cpp	2013-08-27 15:04:22 UTC (rev 154684)
+++ trunk/Source/WebCore/html/HTMLAppletElement.cpp	2013-08-27 15:06:04 UTC (rev 154685)
@@ -24,7 +24,7 @@
 #include "config.h"
 #include "HTMLAppletElement.h"
 
-#include "Attribute.h"
+#include "ChildIterator.h"
 #include "Frame.h"
 #include "FrameLoader.h"
 #include "HTMLDocument.h"
@@ -147,11 +147,8 @@
         paramValues.append(mayScript.string());
     }
 
-    for (Node* child = firstChild(); child; child = child->nextSibling()) {
-        if (!child->hasTagName(paramTag))
-            continue;
-
-        HTMLParamElement* param = static_cast<HTMLParamElement*>(child);
+    auto paramChildren = childrenOfType<HTMLParamElement>(this);
+    for (auto param = paramChildren.begin(), end = paramChildren.end(); param != end; ++param) {
         if (param->name().isEmpty())
             continue;
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to