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

BFCache: Make a WPT test work with BFCache #26408

Merged
merged 1 commit into from Nov 5, 2020

Conversation

chromium-wpt-export-bot
Copy link
Collaborator

@chromium-wpt-export-bot chromium-wpt-export-bot commented Nov 5, 2020

If we enable same-site backward/forward cache, backward navigation
does not trigger onload event.

This CL updates a WPT that depends on the onload event to handle
pageshow event and makes it work with backward/forward cache.

Bug: 1132180
Change-Id: I663f9c78e8b9835f081c5262b446bd56dba07cf9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2520558
Reviewed-by: Fergal Daly <fergal@chromium.org>
Reviewed-by: Rakina Zata Amni <rakina@chromium.org>
Commit-Queue: Hitoshi Yoshida <peria@chromium.org>
Cr-Commit-Position: refs/heads/master@{#824321}

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.

If we enable same-site backward/forward cache, backward navigation
does not trigger onload event.

This CL updates a WPT that depends on the onload event to handle
pageshow event and makes it work with backward/forward cache.

Bug: 1132180
Change-Id: I663f9c78e8b9835f081c5262b446bd56dba07cf9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2520558
Reviewed-by: Fergal Daly <fergal@chromium.org>
Reviewed-by: Rakina Zata Amni <rakina@chromium.org>
Commit-Queue: Hitoshi Yoshida <peria@chromium.org>
Cr-Commit-Position: refs/heads/master@{#824321}
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.

None yet

3 participants