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

blink: Fire pageconceal for non-ViewTransition navigations. #44714

Merged
merged 1 commit into from
Feb 26, 2024

Conversation

chromium-wpt-export-bot
Copy link
Collaborator

@chromium-wpt-export-bot chromium-wpt-export-bot commented Feb 21, 2024

Fire the pageconceal event for cross-document navigations when there is
no ViewTransition. This is dispatched right before sending the
CommitNavigation IPC. For ViewTransition, the event is dispatched
earlier when running the commit deferring conditions.

Bug: 1522213
Change-Id: Ie3b6df0a1c5692c6d8c13330e023e1fac2280073
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5315496
Reviewed-by: Joe Mason <[email protected]>
Auto-Submit: Khushal Sagar <[email protected]>
Commit-Queue: Vladimir Levin <[email protected]>
Reviewed-by: Rakina Zata Amni <[email protected]>
Reviewed-by: Vladimir Levin <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1265230}

Fire the pageconceal event for cross-document navigations when there is
no ViewTransition. This is dispatched right before sending the
CommitNavigation IPC. For ViewTransition, the event is dispatched
earlier when running the commit deferring conditions.

Bug: 1522213
Change-Id: Ie3b6df0a1c5692c6d8c13330e023e1fac2280073
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5315496
Reviewed-by: Joe Mason <[email protected]>
Auto-Submit: Khushal Sagar <[email protected]>
Commit-Queue: Vladimir Levin <[email protected]>
Reviewed-by: Rakina Zata Amni <[email protected]>
Reviewed-by: Vladimir Levin <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1265230}
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 49d1948 into master Feb 26, 2024
22 checks passed
@chromium-wpt-export-bot chromium-wpt-export-bot deleted the chromium-export-cl-5315496 branch February 26, 2024 17:06
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