[wpt-sync] Sync PR 33494 - Convert <embed hidden> into a proper boolean attribute
Categories
(Core :: DOM: Core & HTML, task, P4)
Tracking
()
Tracking | Status | |
---|---|---|
firefox101 | --- | fixed |
People
(Reporter: mozilla.org, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream])
Sync web-platform-tests PR 33494 into mozilla-central (this bug is closed when the sync is complete).
PR: https://github.com/web-platform-tests/wpt/pull/33494
Details from upstream follow.
Mason Freed <masonf@chromium.org> wrote:
Convert <embed hidden> into a proper boolean attribute
Prior to this CL, \<embed hidden> would still be visible. Only the
special values of \<embed hidden=yes> or \<embed hidden=true> would
"hide" the \<embed>. (And it would be "hidden" by making it zero-
size, presumably to not change loading behavior.)After this CL, the 'hidden' attribute behaves as a normal
boolean attribute - presence of the attribute is interpreted as
a "true" value.This appears to be per-spec:
https://html.spec.whatwg.org/#the-embed-element
https://html.spec.whatwg.org/#the-hidden-attribute
https://html.spec.whatwg.org/#boolean-attributeGecko does not respect the 'hidden' attribute at all on the \<embed>
element. WebKit behavior matches the new behavior here - it
treats 'hidden' as a regular boolean attribute, honoring it if present.
Because the spec matches this new behavior, and because WebKit already
agrees with the behavior, I'm treating this like a bug fix, and not
something that needs an I2S. It seems like quite a corner case for a
site to be using \<embed hidden=dont> or something, and expecting the
embed not to be hidden.Fixed: 1063756
Change-Id: I1fc11d0de6ff9e3cc2f7c0d3e438ab9fc929e36bReviewed-on: https://chromium-review.googlesource.com/3565492
WPT-Export-Revision: d41b82205761d5209c82b45d85375fb984abc9fd
Assignee | ||
Updated•2 years ago
|
Assignee | ||
Comment 1•2 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=27d04935f200829c126ce93e8698e6c7db7a4bf3
Assignee | ||
Comment 2•2 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=3878e5aac5984b0b4a90a6df3a848272f27aec72
Assignee | ||
Comment 3•2 years ago
|
||
CI Results
Ran 0 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI
Total 1 tests
Status Summary
Firefox
FAIL: 1
Chrome
FAIL: 1
Safari
FAIL: 1
Links
Details
New Tests That Don't Pass
/html/semantics/embedded-content/the-embed-element/embed-hidden-attribute.html: FAIL (Chrome: FAIL, Safari: FAIL)
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/514c6ce06e8f [wpt PR 33494] - Convert <embed hidden> into a proper boolean attribute, a=testonly https://hg.mozilla.org/integration/autoland/rev/01142f5edb61 [wpt PR 33494] - Update wpt metadata, a=testonly
Comment 5•2 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/514c6ce06e8f
https://hg.mozilla.org/mozilla-central/rev/01142f5edb61
Description
•