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 client hints] Align implementation with PR#46, #48 & #70 #21428

Merged
merged 1 commit into from
Mar 6, 2020

Conversation

chromium-wpt-export-bot
Copy link
Collaborator

@chromium-wpt-export-bot chromium-wpt-export-bot commented Jan 26, 2020

This CL aligns the UA-CH implementation with PR#46[1], #48[2] and
#70[3].

[1] WICG/ua-client-hints#46
[2] WICG/ua-client-hints#48
[3] WICG/ua-client-hints#70

Change-Id: I8221d8a967213180a1aa1d9ef23f17e6f95718b7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2019369
Reviewed-by: Mike West <mkwst@chromium.org>
Reviewed-by: Daniel Vogelheim <vogelheim@chromium.org>
Reviewed-by: Aaron Tagliaboschi <aarontag@chromium.org>
Reviewed-by: Avi Drissman <avi@chromium.org>
Commit-Queue: Yoav Weiss <yoavweiss@chromium.org>
Cr-Commit-Position: refs/heads/master@{#747644}

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-2019369 branch 2 times, most recently from 620375a to 048a545 Compare January 26, 2020 23:11
@chromium-wpt-export-bot chromium-wpt-export-bot changed the title [UA client hints] Align implementation with PR#46 [UA client hints] Align implementation with PR#46 and PR#48 Jan 27, 2020
@chromium-wpt-export-bot chromium-wpt-export-bot force-pushed the chromium-export-cl-2019369 branch 2 times, most recently from 0e1f939 to ba16c7b Compare January 27, 2020 17:23
@chromium-wpt-export-bot chromium-wpt-export-bot changed the title [UA client hints] Align implementation with PR#46 and PR#48 [UA client hints] Align implementation with PR#46, #48 & #70 Mar 3, 2020
@chromium-wpt-export-bot chromium-wpt-export-bot force-pushed the chromium-export-cl-2019369 branch 7 times, most recently from 2be175d to 547d6aa Compare March 5, 2020 14:59
This CL aligns the UA-CH implementation with PR#46[1], #48[2] and
#70[3].

[1] WICG/ua-client-hints#46
[2] WICG/ua-client-hints#48
[3] WICG/ua-client-hints#70

Change-Id: I8221d8a967213180a1aa1d9ef23f17e6f95718b7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2019369
Reviewed-by: Mike West <mkwst@chromium.org>
Reviewed-by: Daniel Vogelheim <vogelheim@chromium.org>
Reviewed-by: Aaron Tagliaboschi <aarontag@chromium.org>
Reviewed-by: Avi Drissman <avi@chromium.org>
Commit-Queue: Yoav Weiss <yoavweiss@chromium.org>
Cr-Commit-Position: refs/heads/master@{#747644}
@Hexcles
Copy link
Member

Hexcles commented Mar 6, 2020

There seems to be a flake in /client-hints/sec-ch-ua.http.html, bu it's not likely caused by this change, just a false positive of "affected tests" because of the change in the resource file.

@Hexcles Hexcles merged commit e0be414 into master Mar 6, 2020
@Hexcles Hexcles deleted the chromium-export-cl-2019369 branch March 6, 2020 21:48
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.

3 participants