Closed Bug 711446 Opened 12 years ago Closed 12 years ago

Test defaultMuted with reflect.js

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla12

People

(Reporter: Ms2ger, Assigned: plop2plopplop)

References

Details

(Whiteboard: [good first bug][mentor=Ms2ger][lang=js])

Attachments

(1 file, 1 obsolete file)

<http://mxr.mozilla.org/mozilla-central/source/content/html/content/test/reflect.js?force=1> contains a function to test boolean reflection rather thoroughly, so it would be nice to use it for the defaultMuted test: <http://mxr.mozilla.org/mozilla-central/source/content/media/test/test_defaultMuted.html?force=1>.
Whiteboard: [good first bug][mentor=Ms2ger][lang=js]
Attached patch Patch version 1 (obsolete) — Splinter Review
Attachment #583635 - Flags: review?(Ms2ger)
Comment on attachment 583635 [details] [diff] [review]
Patch version 1

r=me, thanks for the patch.

It would be really nice if you could provide a patch as described at <https://developer.mozilla.org/en/Mercurial_FAQ#How_can_I_generate_a_patch_for_somebody_else_to_check-in_for_me.3f>.
Attachment #583635 - Flags: review?(Ms2ger) → review+
Assignee: nobody → plop2plopplop
Attached patch Patch version 2Splinter Review
Attachment #583635 - Attachment is obsolete: true
Attachment #583942 - Flags: checkin?
Keywords: checkin-needed
Thanks again for the patch.

https://hg.mozilla.org/mozilla-central/rev/fbef0361b27a
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Attachment #583942 - Flags: checkin? → checkin+
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.