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

RESOLVED FIXED in mozilla22

Status

()

RESOLVED FIXED
8 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)

(Assignee)

Description

8 years ago
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.
(Assignee)

Comment 1

6 years ago
Created attachment 720473 [details] [diff] [review]
Patch

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+
(Assignee)

Comment 3

6 years ago
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
Last Resolved: 6 years ago
Resolution: --- → FIXED

Updated

6 years ago
Depends on: 878673

Comment 5

6 years ago
This was found, by bisection, to have regressed 3D silverlight video: see bug 878673.

Updated

6 years ago
Depends on: 890516
(Assignee)

Updated

6 years ago
Depends on: 893946
You need to log in before you can comment on or make changes to this bug.