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: Implement to return registration data after evaluate module scripts, and add WPT tests for ServiceWorker module scripts #12959

Merged
merged 1 commit into from Oct 1, 2018

Conversation

chromium-wpt-export-bot
Copy link
Collaborator

@chromium-wpt-export-bot chromium-wpt-export-bot commented Sep 12, 2018

Implement to return registration data after evaluate module scripts
and commonize WillEvaluate/DidEvaluate functions of classic and module.
In addition, import module tests for ServiceWorker scripts reuse tests
for DedicatedWorker. The data path in both type is "Main scirpt ->
Worker script -> Main scirpt" because ServiceWorker can reply to its
source only in MessageEvent.

Change-Id: I2bcc1670bd43109710af30c7df3aba5ff039201a
Bug: 824647
Reviewed-on: https://chromium-review.googlesource.com/1205916
Commit-Queue: Asami Doi <asamidoi@google.com>
Reviewed-by: Ilya Sherman <isherman@chromium.org>
Reviewed-by: Matt Falkenhagen <falken@chromium.org>
Reviewed-by: Hiroki Nakagawa <nhiroki@chromium.org>
Reviewed-by: Kouhei Ueno <kouhei@chromium.org>
Reviewed-by: Makoto Shimazu <shimazu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595369}

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 Worker: Add WPT tests for ServiceWorker module scripts. ServiceWorker: Implement to return registration data after evaluate module scripts, and add WPT tests for ServiceWorker module scripts Sep 12, 2018
@chromium-wpt-export-bot chromium-wpt-export-bot force-pushed the chromium-export-cl-1205916 branch 9 times, most recently from 461fff9 to 66d421b Compare September 14, 2018 08:15
@chromium-wpt-export-bot chromium-wpt-export-bot changed the title ServiceWorker: Implement to return registration data after evaluate module scripts, and add WPT tests for ServiceWorker module scripts ServiceWorker: Create script_type field to local storage to store "classic" or "module". Sep 14, 2018
@chromium-wpt-export-bot chromium-wpt-export-bot changed the title ServiceWorker: Create script_type field to local storage to store "classic" or "module". ServiceWorker: Implement to return registration data after evaluate module scripts, and add WPT tests for ServiceWorker module scripts Sep 14, 2018
@chromium-wpt-export-bot chromium-wpt-export-bot force-pushed the chromium-export-cl-1205916 branch 4 times, most recently from aa7cf7b to 11edabd Compare September 18, 2018 08:31
@chromium-wpt-export-bot chromium-wpt-export-bot force-pushed the chromium-export-cl-1205916 branch 6 times, most recently from 8625a05 to 73c3564 Compare September 26, 2018 09:01
@chromium-wpt-export-bot chromium-wpt-export-bot force-pushed the chromium-export-cl-1205916 branch 4 times, most recently from df84d7f to 3ad9d6f Compare October 1, 2018 01:18
module scripts, and add WPT tests for ServiceWorker module scripts

Implement to return registration data after evaluate module scripts
and commonize WillEvaluate/DidEvaluate functions of classic and module.
In addition, import module tests for ServiceWorker scripts reuse tests
for DedicatedWorker. The data path in both type is "Main scirpt ->
Worker script -> Main scirpt" because ServiceWorker can reply to its
source only in MessageEvent.

Change-Id: I2bcc1670bd43109710af30c7df3aba5ff039201a
Bug: 824647
Reviewed-on: https://chromium-review.googlesource.com/1205916
Commit-Queue: Asami Doi <asamidoi@google.com>
Reviewed-by: Ilya Sherman <isherman@chromium.org>
Reviewed-by: Matt Falkenhagen <falken@chromium.org>
Reviewed-by: Hiroki Nakagawa <nhiroki@chromium.org>
Reviewed-by: Kouhei Ueno <kouhei@chromium.org>
Reviewed-by: Makoto Shimazu <shimazu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595369}
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