Skip to content
Permalink
Browse files
2010-02-26 James Hawkins <jhawkins@chromium.org>
        Reviewed by Darin Fisher.

        [Chromium] Verify that the Nodes being enumerated are HTML elements.

        https://bugs.webkit.org/show_bug.cgi?id=35463

        * src/WebFrameImpl.cpp:
        (WebKit::WebFrameImpl::forms):

Canonical link: https://commits.webkit.org/46647@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@55346 268f45cc-cd09-0410-ab3c-d52691b4dbfc
  • Loading branch information
eseidel committed Feb 27, 2010
1 parent 14ff3bd commit 7772571367d0c3728512af838a0a37b5a2e6d724
Showing 2 changed files with 12 additions and 1 deletion.
@@ -1,3 +1,14 @@
2010-02-26 James Hawkins <jhawkins@chromium.org>

Reviewed by Darin Fisher.

[Chromium] Verify that the Nodes being enumerated are HTML elements.

https://bugs.webkit.org/show_bug.cgi?id=35463

* src/WebFrameImpl.cpp:
(WebKit::WebFrameImpl::forms):

2010-02-26 Adam Barth <abarth@webkit.org>

Reviewed by Darin Fisher.
@@ -559,7 +559,7 @@ void WebFrameImpl::forms(WebVector<WebFormElement>& results) const
for (size_t i = 0; i < formCount; ++i) {
Node* node = forms->item(i);
// Strange but true, sometimes item can be 0.
if (node)
if (node && node->isHTMLElement())
temp[i] = static_cast<HTMLFormElement*>(node);
}
results.swap(temp);

0 comments on commit 7772571

Please sign in to comment.