Closed Bug 614825 Opened 14 years ago Closed 11 years ago

<embed> should be display:none; when hidden attribute is set

Categories

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

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla22

People

(Reporter: mounir, Assigned: mounir)

References

(Depends on 1 open bug)

Details

Attachments

(1 file)

Because of bug 90268, <embed> with no frame do nothing so, with @hidden making the element display:none;, <embed> was disabled when hidden, see bug 613722.
@hidden now does nothing for <embed> and this should be reverted as soon as bug 90268 is fixed.
Attached patch PatchSplinter Review
This was bitrotting in my patch queue.
Assignee: nobody → mounir
Status: NEW → ASSIGNED
Attachment #720473 - Flags: review?(bzbarsky)
Comment on attachment 720473 [details] [diff] [review]
Patch

Remove that TODO comment in the test, and r=me
Attachment #720473 - Flags: review?(bzbarsky) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/f73c5ece0869
Flags: in-testsuite+
Target Milestone: --- → mozilla22
https://hg.mozilla.org/mozilla-central/rev/f73c5ece0869
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Depends on: 878673
This was found, by bisection, to have regressed 3D silverlight video: see bug 878673.
Depends on: 890516
Depends on: 893946
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: