Skip to content

Conversation

@chromium-wpt-export-bot
Copy link
Collaborator

@chromium-wpt-export-bot chromium-wpt-export-bot commented Jul 17, 2024

Add preferInitialWindowPlacement to DocumentPictureInPictureOptions
to provide a hint to the UA that the bounds of any previously opened
pip window probably shouldn't be re-used.

See WICG/document-picture-in-picture#119 and
https://chromestatus.com/feature/5183881532932096 for more details.

Change-Id: I3c3cf468046f4e08cbedbde93532e4b34e12a29a
Bug: 312495380
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5608743
Reviewed-by: Philip Rogers <[email protected]>
Commit-Queue: Frank Liberato <[email protected]>
Reviewed-by: Tommy Steimel <[email protected]>
Auto-Submit: Frank Liberato <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1328934}

Add `preferInitialWindowPlacement` to DocumentPictureInPictureOptions
to provide a hint to the UA that the bounds of any previously opened
pip window probably shouldn't be re-used.

See WICG/document-picture-in-picture#119 and
https://chromestatus.com/feature/5183881532932096 for more details.

Change-Id: I3c3cf468046f4e08cbedbde93532e4b34e12a29a
Bug: 312495380
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5608743
Reviewed-by: Philip Rogers <[email protected]>
Commit-Queue: Frank Liberato <[email protected]>
Reviewed-by: Tommy Steimel <[email protected]>
Auto-Submit: Frank Liberato <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1328934}
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.

@chromium-wpt-export-bot chromium-wpt-export-bot merged commit e8d2870 into master Jul 17, 2024
@chromium-wpt-export-bot chromium-wpt-export-bot deleted the chromium-export-cl-5608743 branch July 17, 2024 18:14
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