[WPT] Add Client Hints tests for speculation rules prefetch + redirect [chromium/src : main]

0 views
Skip to first unread message

Hiroki Nakagawa (Gerrit)

unread,
Dec 22, 2025, 9:41:57 PM (10 hours ago) Dec 22
to Hiroshige Hayashizaki, Taiyo Mizuhashi, Ken Okada, Chromium LUCI CQ, chromium...@chromium.org, AyeAye, blink-revie...@chromium.org, blink-...@chromium.org
Attention needed from Hiroshige Hayashizaki and Taiyo Mizuhashi

Hiroki Nakagawa voted and added 2 comments

Votes added by Hiroki Nakagawa

Code-Review+1

2 comments

Patchset-level comments
File-level comment, Patchset 4 (Latest):
Hiroki Nakagawa . resolved

LGTM, thanks!

File third_party/blink/web_tests/external/wpt/speculation-rules/prefetch/resources/executor.sub.html
Line 37, Patchset 4 (Latest): "sec-ch-dpr": "{{header_or_default(Sec-CH-DPR, NONEXISTENT)}}",
Hiroki Nakagawa . unresolved

Is `sec-ch-dpr` used somewhere? If not, can we remove this?

Open in Gerrit

Related details

Attention is currently required from:
  • Hiroshige Hayashizaki
  • Taiyo Mizuhashi
Submit Requirements:
  • requirement satisfiedCode-Coverage
  • requirement satisfiedCode-Owners
  • requirement satisfiedCode-Review
  • requirement is not satisfiedNo-Unresolved-Comments
  • requirement satisfiedReview-Enforcement
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: chromium/src
Gerrit-Branch: main
Gerrit-Change-Id: I33a239310c87edf2fc69c738de9029980a339ad1
Gerrit-Change-Number: 7284405
Gerrit-PatchSet: 4
Gerrit-Owner: Hiroshige Hayashizaki <hiro...@chromium.org>
Gerrit-Reviewer: Hiroki Nakagawa <nhi...@chromium.org>
Gerrit-Reviewer: Hiroshige Hayashizaki <hiro...@chromium.org>
Gerrit-Reviewer: Taiyo Mizuhashi <ta...@chromium.org>
Gerrit-CC: Ken Okada <ken...@chromium.org>
Gerrit-Attention: Taiyo Mizuhashi <ta...@chromium.org>
Gerrit-Attention: Hiroshige Hayashizaki <hiro...@chromium.org>
Gerrit-Comment-Date: Tue, 23 Dec 2025 02:41:22 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
satisfied_requirement
unsatisfied_requirement
open
diffy
Reply all
Reply to author
Forward
0 new messages