[AtMemory] Control PCService request timeout with a feature param [chromium/src : main]

0 views
Skip to first unread message

Karol Sygiet (Gerrit)

unread,
7:47 AM (6 hours ago) 7:47 AM
to Jan Keitel, chromium...@chromium.org, browser-comp...@chromium.org
Attention needed from Jan Keitel

Karol Sygiet added 1 comment

Patchset-level comments
File-level comment, Patchset 3 (Latest):
Karol Sygiet . resolved

Hi Jan, could you review this CL?

Open in Gerrit

Related details

Attention is currently required from:
  • Jan Keitel
Submit Requirements:
  • requirement satisfiedCode-Coverage
  • requirement is not satisfiedCode-Owners
  • requirement is not satisfiedCode-Review
  • requirement is not 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: I3f1c6b334273166043e59dec7a581cfbb500a260
Gerrit-Change-Number: 8064267
Gerrit-PatchSet: 3
Gerrit-Owner: Karol Sygiet <syg...@google.com>
Gerrit-Reviewer: Jan Keitel <jke...@google.com>
Gerrit-Attention: Jan Keitel <jke...@google.com>
Gerrit-Comment-Date: Wed, 08 Jul 2026 11:46:53 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
satisfied_requirement
unsatisfied_requirement
open
diffy

Jan Keitel (Gerrit)

unread,
8:49 AM (5 hours ago) 8:49 AM
to Karol Sygiet, chromium...@chromium.org, browser-comp...@chromium.org
Attention needed from Karol Sygiet

Jan Keitel voted and added 1 comment

Votes added by Jan Keitel

Code-Review+1

1 comment

Patchset-level comments
Jan Keitel . resolved

Thanks!

Open in Gerrit

Related details

Attention is currently required from:
  • Karol Sygiet
Submit Requirements:
  • requirement satisfiedCode-Coverage
  • requirement satisfiedCode-Owners
  • requirement satisfiedCode-Review
  • 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: I3f1c6b334273166043e59dec7a581cfbb500a260
Gerrit-Change-Number: 8064267
Gerrit-PatchSet: 3
Gerrit-Owner: Karol Sygiet <syg...@google.com>
Gerrit-Reviewer: Jan Keitel <jke...@google.com>
Gerrit-Attention: Karol Sygiet <syg...@google.com>
Gerrit-Comment-Date: Wed, 08 Jul 2026 12:48:50 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
satisfied_requirement
open
diffy

Karol Sygiet (Gerrit)

unread,
8:49 AM (5 hours ago) 8:49 AM
to Jan Keitel, chromium...@chromium.org, browser-comp...@chromium.org

Karol Sygiet voted Commit-Queue+2

Commit-Queue+2
Open in Gerrit

Related details

Attention set is empty
Submit Requirements:
  • requirement satisfiedCode-Coverage
  • requirement satisfiedCode-Owners
  • requirement satisfiedCode-Review
  • 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: I3f1c6b334273166043e59dec7a581cfbb500a260
Gerrit-Change-Number: 8064267
Gerrit-PatchSet: 3
Gerrit-Owner: Karol Sygiet <syg...@google.com>
Gerrit-Reviewer: Jan Keitel <jke...@google.com>
Gerrit-Reviewer: Karol Sygiet <syg...@google.com>
Gerrit-Comment-Date: Wed, 08 Jul 2026 12:49:28 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
satisfied_requirement
open
diffy

Chromium LUCI CQ (Gerrit)

unread,
9:43 AM (4 hours ago) 9:43 AM
to Karol Sygiet, Jan Keitel, chromium...@chromium.org, browser-comp...@chromium.org

Chromium LUCI CQ submitted the change

Change information

Commit message:
[AtMemory] Control PCService request timeout with a feature param

This CL adds an kAutofillAtMemory feature param that controls the
timeout used for requests made to PersonalContextService. Since the
default timeout value is non changing this CL should be a noop.
Bug: 525668259
Change-Id: I3f1c6b334273166043e59dec7a581cfbb500a260
Reviewed-by: Jan Keitel <jke...@google.com>
Commit-Queue: Karol Sygiet <syg...@google.com>
Cr-Commit-Position: refs/heads/main@{#1658697}
Files:
  • M components/autofill/core/browser/integrators/at_memory/at_memory_query_service.cc
  • M components/autofill/core/browser/integrators/at_memory/at_memory_query_service_unittest.cc
  • M components/autofill/core/common/autofill_features.cc
  • M components/autofill/core/common/autofill_features.h
Change size: S
Delta: 4 files changed, 34 insertions(+), 2 deletions(-)
Branch: refs/heads/main
Submit Requirements:
  • requirement satisfiedCode-Review: +1 by Jan Keitel
Open in Gerrit
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: merged
Gerrit-Project: chromium/src
Gerrit-Branch: main
Gerrit-Change-Id: I3f1c6b334273166043e59dec7a581cfbb500a260
Gerrit-Change-Number: 8064267
Gerrit-PatchSet: 4
Gerrit-Owner: Karol Sygiet <syg...@google.com>
Gerrit-Reviewer: Chromium LUCI CQ <chromiu...@luci-project-accounts.iam.gserviceaccount.com>
open
diffy
satisfied_requirement
Reply all
Reply to author
Forward
0 new messages