Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add a WPT to check for standards mode plugin document #29264

Closed
wants to merge 1 commit into from

Conversation

chromium-wpt-export-bot
Copy link
Collaborator

@chromium-wpt-export-bot chromium-wpt-export-bot commented Jun 7, 2021

The [1] PR will change the standard so that plugin documents
are in standard (no quirks) mode. This CL adds a test, as
much as this is testable. Some UAs display PDF documents
using "content that uses plugins" [2] while others use
"inline content that doesn't have a DOM [3]. This test will
pass in either case.

Chromium already made this change in code in Sept 2020
(see CLs on crbug.com/1131185) and found no compat
issues.

[1] whatwg/html#6745
[2] https://html.spec.whatwg.org/#read-plugin
[3] https://html.spec.whatwg.org/#read-ua-inline

Bug: 1131185
Change-Id: Iccd1668199ac8b3b1c90a69b97a0adbaf8d02655
Reviewed-on: https://chromium-review.googlesource.com/2944721
WPT-Export-Revision: bd363be8c360879cf3e4680fa8085fde26e2f656

Copy link
Collaborator

@wpt-pr-bot wpt-pr-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The review process for this patch is being conducted in the Chromium project.

The [1] PR will change the standard so that plugin documents
are in standard (no quirks) mode. This CL adds a test, as
much as this is testable. Some UAs display PDF documents
using "content that uses plugins" [2] while others use
"inline content that doesn't have a DOM [3]. This test will
pass in either case.

Chromium already made this change in code in Sept 2020
(see CLs on crbug.com/1131185) and found no compat
issues.

[1] whatwg/html#6745
[2] https://html.spec.whatwg.org/#read-plugin
[3] https://html.spec.whatwg.org/#read-ua-inline

Bug: 1131185
Change-Id: Iccd1668199ac8b3b1c90a69b97a0adbaf8d02655
@KyleJu
Copy link
Contributor

KyleJu commented Oct 13, 2021

Close this PR because the Chromium CL has been abandoned.

@KyleJu KyleJu closed this Oct 13, 2021
@KyleJu KyleJu deleted the chromium-export-cl-2944721 branch October 13, 2021 23:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants