Summary: | Adopt is/toHTMLOptionElement for code cleanup | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Kangil Han <kangil.han> | ||||
Component: | WebCore Misc. | Assignee: | Kangil Han <kangil.han> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | commit-queue, kling, koivisto | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Kangil Han
2013-06-28 23:08:10 PDT
Created attachment 205763 [details]
Patch
Comment on attachment 205763 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=205763&action=review r=me > Source/WebCore/html/HTMLCollection.cpp:227 > if (element->hasLocalName(optionTag)) { I wonder if we could use isHTMLOptionElement() here instead of just checking the localName. > Source/WebCore/html/HTMLSelectElement.cpp:244 > + if (items[i]->hasLocalName(optionTag)) { Ditto. > Source/WebCore/html/HTMLSelectElement.cpp:265 > if (items[i]->hasLocalName(optionTag)) { Ditto. > Source/WebCore/html/HTMLSelectElement.cpp:979 > if (!items[i]->hasLocalName(optionTag)) Ditto. (In reply to comment #2) > (From update of attachment 205763 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=205763&action=review > > r=me > > > Source/WebCore/html/HTMLCollection.cpp:227 > > if (element->hasLocalName(optionTag)) { > > I wonder if we could use isHTMLOptionElement() here instead of just checking the localName. > > > Source/WebCore/html/HTMLSelectElement.cpp:244 > > + if (items[i]->hasLocalName(optionTag)) { > > Ditto. > > > Source/WebCore/html/HTMLSelectElement.cpp:265 > > if (items[i]->hasLocalName(optionTag)) { > > Ditto. > > > Source/WebCore/html/HTMLSelectElement.cpp:979 > > if (!items[i]->hasLocalName(optionTag)) > > Ditto. Agreed. IMHO, I can replace hasLocalName(tag) stuff with isFooElement in another patch when adoption is/toFooElement is completed. :) Comment on attachment 205763 [details] Patch Clearing flags on attachment: 205763 Committed r152208: <http://trac.webkit.org/changeset/152208> All reviewed patches have been landed. Closing bug. |