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

[scoped-registry] Implement scoped custom element upgrade #37943

Merged
merged 1 commit into from
May 2, 2023

Conversation

chromium-wpt-export-bot
Copy link
Collaborator

@chromium-wpt-export-bot chromium-wpt-export-bot commented Jan 13, 2023

This patch:

  1. Changes CustomElement::Registry() to return tree-scoped registries
    instead of the global registry. As this function is used by a lot
    of callers (including upgrade), this allows these callers to use
    the scoped registry associated with the tree scope.

  2. Custom element construction stack entries are augmented with the
    definition being used, so that later when calling super(), we
    can still know which custom element definition is being used.
    See [scoped-registries] Interaction with HTML element's overridden constructor steps WICG/webcomponents#969 for details.

Bug: 1304439
Change-Id: Id510ecea0f4c5cf6386f77a39d346918c9592e76
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4144367
Reviewed-by: Joey Arhar <[email protected]>
Reviewed-by: Yuki Shiino <[email protected]>
Commit-Queue: Xiaocheng Hu <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1137942}

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 force-pushed the chromium-export-cl-4144367 branch 4 times, most recently from b9df2cd to ccc7b33 Compare May 1, 2023 18:00
This patch:

1. Changes CustomElement::Registry() to return tree-scoped registries
   instead of the global registry. As this function is used by a lot
   of callers (including upgrade), this allows these callers to use
   the scoped registry associated with the tree scope.

2. Custom element construction stack entries are augmented with the
   definition being used, so that later when calling `super()`, we
   can still know which custom element definition is being used.
   See WICG/webcomponents#969 for details.

Bug: 1304439
Change-Id: Id510ecea0f4c5cf6386f77a39d346918c9592e76
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4144367
Reviewed-by: Joey Arhar <[email protected]>
Reviewed-by: Yuki Shiino <[email protected]>
Commit-Queue: Xiaocheng Hu <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1137942}
@jonathan-j-lee
Copy link
Contributor

Traceback (most recent call last):
  File "./wpt", line 10, in <module>
    wpt.main()
  File "/home/test/web-platform-tests/tools/wpt/wpt.py", line 227, in main
    rv = script(*args, **kwargs)
  File "/home/test/web-platform-tests/tools/wpt/run.py", line 885, in run
    wptrunner_kwargs = setup_wptrunner(venv, **kwargs)
  File "/home/test/web-platform-tests/tools/wpt/run.py", line 853, in setup_wptrunner
    setup_cls.setup(kwargs)
  File "/home/test/web-platform-tests/tools/wpt/run.py", line 190, in setup
    self.setup_kwargs(kwargs)
  File "/home/test/web-platform-tests/tools/wpt/run.py", line 259, in setup_kwargs
    prefs_root = self.browser.install_prefs(kwargs["binary"],
  File "/home/test/web-platform-tests/tools/wpt/browser.py", line 421, in install_prefs
    unzip(get(url).raw, dest=extract_dir)
  File "/home/test/web-platform-tests/tools/wpt/utils.py", line 106, in get
    resp.raise_for_status()
  File "/home/test/web-platform-tests/_venv3/lib/python3.8/site-packages/requests/models.py", line 1021, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 500 Server Error: Internal Server Error for url: https://hg.mozilla.org/mozilla-central/archive/tip.zip/testing/profiles/

Outage should be over. Retrying checks.

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.

5 participants