Closed Bug 362391 Opened 14 years ago Closed 14 years ago

DOMAttrModified doesn't handle namespaced attributes properly

Categories

(Core :: DOM: Events, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: smaug, Assigned: smaug)

Details

(Keywords: testcase)

Attachments

(3 files)

a testcase and patch coming.
Attached file testcase
Comment on attachment 247095 [details]
testcase

All tests should be true
Attached patch proposed patchSplinter Review
Keeping things simple for now.
If dispatching DOMAttrModified shows in profiles, we could add a method to get attr node using PRInt32 and nsIAtom*.
Attachment #247096 - Flags: superreview?(bugmail)
Attachment #247096 - Flags: review?(bugmail)
Attachment #247096 - Flags: superreview?(bugmail)
Attachment #247096 - Flags: superreview+
Attachment #247096 - Flags: review?(bugmail)
Attachment #247096 - Flags: review+
We should "mochify" those tests.
Flags: in-testsuite?
Keywords: testcase
Patch checked in.
Testcase needs mochifying.
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Attached patch mochitestSplinter Review
Comment on attachment 247162 [details] [diff] [review]
mochitest

Not sure if mochitests need reviews
Attachment #247162 - Flags: review?(bugmail)
Flags: in-testsuite? → in-testsuite+
You need to log in before you can comment on or make changes to this bug.