Summary: | Remove unused ExceptionCode& argument from Element::setAttribute(QualifiedName) | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Adam Klein <adamk> | ||||||||
Component: | New Bugs | Assignee: | Adam Klein <adamk> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | abarth, darin, ojan, rniwa, webkit.review.bot | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Attachments: |
|
Description
Adam Klein
2011-12-16 12:31:58 PST
Created attachment 119651 [details]
Patch
Comment on attachment 119651 [details]
Patch
Did you verify that all callers of setAttribute(const QualifiedName& name, const AtomicString& value) can assert no exceptions? We used to ignore exceptions before this patch.
(In reply to comment #2) > (From update of attachment 119651 [details]) > Did you verify that all callers of setAttribute(const QualifiedName& name, const AtomicString& value) can assert no exceptions? We used to ignore exceptions before this patch. It's impossible for setAttribute(const QualifiedName&, const AtomicString&, ExceptionCode&) to throw an exception: the implementation in Element.cpp doesn't name the ExceptionCode argument and thus cannot possibly modify it. (In reply to comment #3) > It's impossible for setAttribute(const QualifiedName&, const AtomicString&, ExceptionCode&) to throw an exception: the implementation in Element.cpp doesn't name the ExceptionCode argument and thus cannot possibly modify it. Then why don't we get rid of the version that takes ExceptionCode& ? Created attachment 119939 [details]
Remove third arg
(In reply to comment #4) > (In reply to comment #3) > > It's impossible for setAttribute(const QualifiedName&, const AtomicString&, ExceptionCode&) to throw an exception: the implementation in Element.cpp doesn't name the ExceptionCode argument and thus cannot possibly modify it. > > Then why don't we get rid of the version that takes ExceptionCode& ? Ha, good point, done! Comment on attachment 119939 [details] Remove third arg Attachment 119939 [details] did not pass qt-ews (qt): Output: http://queues.webkit.org/results/10936196 Created attachment 119955 [details]
Patch for landing
Had to update a few callers in JSC SVG bindings, and one in Mac editing code. Comment on attachment 119955 [details] Patch for landing Clearing flags on attachment: 119955 Committed r103296: <http://trac.webkit.org/changeset/103296> All reviewed patches have been landed. Closing bug. |