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

ServiceWorker: Add new tests for updating the registration with different script type. #13692

Merged
merged 1 commit into from Oct 25, 2018

Conversation

chromium-wpt-export-bot
Copy link
Collaborator

@chromium-wpt-export-bot chromium-wpt-export-bot commented Oct 24, 2018

These tests check that a registration is updated correctly with
different script type. At first Service Worker is register as classic
script type, then it is re-registered as module script type, and
vice versa. A main script is also updated at the same time.

Bug: 824647
Change-Id: I3ab9391791508662de8dc73ac932a6105c4d7fe6
Reviewed-on: https://chromium-review.googlesource.com/c/1295763
Commit-Queue: Asami Doi <asamidoi@google.com>
Reviewed-by: Matt Falkenhagen <falken@chromium.org>
Reviewed-by: Hiroki Nakagawa <nhiroki@chromium.org>
Cr-Commit-Position: refs/heads/master@{#602650}

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.

Already reviewed downstream.

@chromium-wpt-export-bot chromium-wpt-export-bot changed the title ServiceWorker: Add a new test for updating the registration with different script type. ServiceWorker: Add new tests for updating the registration with different script type. Oct 24, 2018
@chromium-wpt-export-bot chromium-wpt-export-bot force-pushed the chromium-export-cl-1295763 branch 6 times, most recently from b9ea12b to ec6574d Compare October 25, 2018 07:55
different script type.

These tests check that a registration is updated correctly with
different script type. At first Service Worker is register as classic
script type, then it is re-registered as module script type, and
vice versa. A main script is also updated at the same time.

Bug: 824647
Change-Id: I3ab9391791508662de8dc73ac932a6105c4d7fe6
Reviewed-on: https://chromium-review.googlesource.com/c/1295763
Commit-Queue: Asami Doi <asamidoi@google.com>
Reviewed-by: Matt Falkenhagen <falken@chromium.org>
Reviewed-by: Hiroki Nakagawa <nhiroki@chromium.org>
Cr-Commit-Position: refs/heads/master@{#602650}
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