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

[UA-CH] Add userAgentData worker support #22248

Merged
merged 1 commit into from
Mar 18, 2020

Conversation

chromium-wpt-export-bot
Copy link
Collaborator

@chromium-wpt-export-bot chromium-wpt-export-bot commented Mar 13, 2020

This implements PR #80 [1] and exposes userAgentData in workers.

[1] WICG/ua-client-hints#80

Bug: 1062192
Change-Id: I1de6067ab6c00855d7bf820e9646eb4e5a8c5fdb
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2093220
Commit-Queue: Yoav Weiss <yoavweiss@chromium.org>
Reviewed-by: Hiroki Nakagawa <nhiroki@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Matt Falkenhagen <falken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#751295}

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-2093220 branch 12 times, most recently from bad451c to c0db6e6 Compare March 18, 2020 10:57
This implements PR #80 [1] and exposes userAgentData in workers.

[1] WICG/ua-client-hints#80

Bug: 1062192
Change-Id: I1de6067ab6c00855d7bf820e9646eb4e5a8c5fdb
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2093220
Commit-Queue: Yoav Weiss <yoavweiss@chromium.org>
Reviewed-by: Hiroki Nakagawa <nhiroki@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Matt Falkenhagen <falken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#751295}
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.

2 participants