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

RESOLVED FIXED in mozilla22

Status

()

defect
RESOLVED FIXED
9 years ago
6 years ago

People

(Reporter: mounir, Assigned: mounir)

Tracking

(Depends on 1 bug)

Trunk
mozilla22
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

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.
Posted 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: 7 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.