[omnibox] Replace `prefer_keyword` variables with `in_keyword_mode` [chromium/src : main]

0 views
Skip to first unread message

manuk hovanesian (Gerrit)

unread,
Jun 23, 2026, 7:01:42 PM (2 days ago) Jun 23
to Tomasz Wiszkowski, Chromium LUCI CQ, chromium...@chromium.org, christia...@chromium.org, chromium-a...@chromium.org, extension...@chromium.org, jdonnel...@chromium.org, omnibox-...@chromium.org
Attention needed from Tomasz Wiszkowski

manuk hovanesian voted Commit-Queue+1

Commit-Queue+1
Open in Gerrit

Related details

Attention is currently required from:
  • Tomasz Wiszkowski
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: I1e7a1b14afc37145bbfcf5d14ed26434377e854a
Gerrit-Change-Number: 7988397
Gerrit-PatchSet: 2
Gerrit-Owner: manuk hovanesian <man...@chromium.org>
Gerrit-Reviewer: Tomasz Wiszkowski <en...@google.com>
Gerrit-Reviewer: manuk hovanesian <man...@chromium.org>
Gerrit-Attention: Tomasz Wiszkowski <en...@google.com>
Gerrit-Comment-Date: Tue, 23 Jun 2026 23:01:20 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
satisfied_requirement
unsatisfied_requirement
open
diffy

manuk hovanesian (Gerrit)

unread,
1:12 PM (8 hours ago) 1:12 PM
to Chromium IPC Reviews, Christian Xu, android-bu...@system.gserviceaccount.com, Tomasz Wiszkowski, Chromium LUCI CQ, chromium...@chromium.org, ios-revie...@chromium.org, ios-r...@chromium.org, marq+...@chromium.org, christia...@chromium.org, chromium-a...@chromium.org, extension...@chromium.org, jdonnel...@chromium.org, omnibox-...@chromium.org
Attention needed from Christian Xu, Chromium IPC Reviews and Tomasz Wiszkowski

manuk hovanesian added 1 comment

Patchset-level comments
File-level comment, Patchset 5 (Latest):
manuk hovanesian . resolved

chrome-ipc-reviews, pls review .mojom file

christianxu, pls review ios/

Ender, pls review everything

Open in Gerrit

Related details

Attention is currently required from:
  • Christian Xu
  • Chromium IPC Reviews
  • Tomasz Wiszkowski
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: I1e7a1b14afc37145bbfcf5d14ed26434377e854a
Gerrit-Change-Number: 7988397
Gerrit-PatchSet: 5
Gerrit-Owner: manuk hovanesian <man...@chromium.org>
Gerrit-Reviewer: Christian Xu <chris...@chromium.org>
Gerrit-Reviewer: Chromium IPC Reviews <chrome-ip...@google.com>
Gerrit-Reviewer: Tomasz Wiszkowski <en...@google.com>
Gerrit-Reviewer: manuk hovanesian <man...@chromium.org>
Gerrit-Attention: Tomasz Wiszkowski <en...@google.com>
Gerrit-Attention: Christian Xu <chris...@chromium.org>
Gerrit-Attention: Chromium IPC Reviews <chrome-ip...@google.com>
Gerrit-Comment-Date: Thu, 25 Jun 2026 17:11:57 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
satisfied_requirement
unsatisfied_requirement
open
diffy

gwsq (Gerrit)

unread,
1:18 PM (8 hours ago) 1:18 PM
to manuk hovanesian, Chromium IPC Reviews, Fred Shih, Christian Xu, android-bu...@system.gserviceaccount.com, Tomasz Wiszkowski, Chromium LUCI CQ, chromium...@chromium.org, ios-revie...@chromium.org, ios-r...@chromium.org, marq+...@chromium.org, christia...@chromium.org, chromium-a...@chromium.org, extension...@chromium.org, jdonnel...@chromium.org, omnibox-...@chromium.org
Attention needed from Christian Xu, Fred Shih and Tomasz Wiszkowski

Message from gwsq

From googleclient/chrome/chromium_gwsq/ipc/config.gwsq:
IPC: ff...@chromium.org

📎 It looks like you’re making a possibly security-sensitive change! 📎 IPC security review isn’t a rubberstamp, so your friendly security reviewer will need a fair amount of context to review your CL effectively. Please review your CL description and code comments to make sure they provide context for someone unfamiliar with your project/area. Pay special attention to where data comes from and which processes it flows between (and their privilege levels). Feel free to point your security reviewer at design docs, bugs, or other links if you can’t reasonably make a self-contained CL description. (Also see https://cbea.ms/git-commit/).

IPC reviewer(s): ff...@chromium.org


Reviewer source(s):
ff...@chromium.org is from context(googleclient/chrome/chromium_gwsq/ipc/config.gwsq)

Open in Gerrit

Related details

Attention is currently required from:
  • Christian Xu
  • Fred Shih
  • Tomasz Wiszkowski
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: I1e7a1b14afc37145bbfcf5d14ed26434377e854a
Gerrit-Change-Number: 7988397
Gerrit-PatchSet: 5
Gerrit-Owner: manuk hovanesian <man...@chromium.org>
Gerrit-Reviewer: Christian Xu <chris...@chromium.org>
Gerrit-Reviewer: Fred Shih <ff...@chromium.org>
Gerrit-Reviewer: Tomasz Wiszkowski <en...@google.com>
Gerrit-Reviewer: manuk hovanesian <man...@chromium.org>
Gerrit-CC: Chromium IPC Reviews <chrome-ip...@google.com>
Gerrit-CC: gwsq
Gerrit-Attention: Tomasz Wiszkowski <en...@google.com>
Gerrit-Attention: Christian Xu <chris...@chromium.org>
Gerrit-Attention: Fred Shih <ff...@chromium.org>
Gerrit-Comment-Date: Thu, 25 Jun 2026 17:17:55 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
satisfied_requirement
unsatisfied_requirement
open
diffy

manuk hovanesian (Gerrit)

unread,
3:00 PM (6 hours ago) 3:00 PM
to Ameur Hosni, Christian Xu, Chromium IPC Reviews, Fred Shih, android-bu...@system.gserviceaccount.com, Tomasz Wiszkowski, Chromium LUCI CQ, chromium...@chromium.org, ios-revie...@chromium.org, ios-r...@chromium.org, marq+...@chromium.org, christia...@chromium.org, chromium-a...@chromium.org, extension...@chromium.org, jdonnel...@chromium.org, omnibox-...@chromium.org
Attention needed from Ameur Hosni, Christian Xu, Fred Shih and Tomasz Wiszkowski

manuk hovanesian added 1 comment

Patchset-level comments
manuk hovanesian . resolved

Ameur for ios file since Crhistian's ooo

Open in Gerrit

Related details

Attention is currently required from:
  • Ameur Hosni
  • Christian Xu
  • Fred Shih
  • Tomasz Wiszkowski
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: I1e7a1b14afc37145bbfcf5d14ed26434377e854a
Gerrit-Change-Number: 7988397
Gerrit-PatchSet: 5
Gerrit-Owner: manuk hovanesian <man...@chromium.org>
Gerrit-Reviewer: Ameur Hosni <ameur...@google.com>
Gerrit-Reviewer: Fred Shih <ff...@chromium.org>
Gerrit-Reviewer: Tomasz Wiszkowski <en...@google.com>
Gerrit-Reviewer: manuk hovanesian <man...@chromium.org>
Gerrit-CC: Christian Xu <chris...@chromium.org>
Gerrit-Attention: Ameur Hosni <ameur...@google.com>
Gerrit-Attention: Tomasz Wiszkowski <en...@google.com>
Gerrit-Attention: Christian Xu <chris...@chromium.org>
Gerrit-Attention: Fred Shih <ff...@chromium.org>
Gerrit-Comment-Date: Thu, 25 Jun 2026 19:00:25 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
satisfied_requirement
unsatisfied_requirement
open
diffy

Tomasz Wiszkowski (Gerrit)

unread,
4:26 PM (5 hours ago) 4:26 PM
to manuk hovanesian, Ameur Hosni, Christian Xu, Chromium IPC Reviews, Fred Shih, android-bu...@system.gserviceaccount.com, Chromium LUCI CQ, chromium...@chromium.org, ios-revie...@chromium.org, ios-r...@chromium.org, marq+...@chromium.org, christia...@chromium.org, chromium-a...@chromium.org, extension...@chromium.org, jdonnel...@chromium.org, omnibox-...@chromium.org
Attention needed from Ameur Hosni, Christian Xu, Fred Shih and manuk hovanesian

Tomasz Wiszkowski voted Code-Review+1

Code-Review+1
Open in Gerrit

Related details

Attention is currently required from:
  • Ameur Hosni
  • Christian Xu
  • Fred Shih
  • manuk hovanesian
Submit Requirements:
    • requirement satisfiedCode-Coverage
    • requirement is not 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: I1e7a1b14afc37145bbfcf5d14ed26434377e854a
    Gerrit-Change-Number: 7988397
    Gerrit-PatchSet: 5
    Gerrit-Owner: manuk hovanesian <man...@chromium.org>
    Gerrit-Reviewer: Ameur Hosni <ameur...@google.com>
    Gerrit-Reviewer: Fred Shih <ff...@chromium.org>
    Gerrit-Reviewer: Tomasz Wiszkowski <en...@google.com>
    Gerrit-Reviewer: manuk hovanesian <man...@chromium.org>
    Gerrit-CC: Christian Xu <chris...@chromium.org>
    Gerrit-CC: Chromium IPC Reviews <chrome-ip...@google.com>
    Gerrit-CC: gwsq
    Gerrit-Attention: Ameur Hosni <ameur...@google.com>
    Gerrit-Attention: manuk hovanesian <man...@chromium.org>
    Gerrit-Attention: Christian Xu <chris...@chromium.org>
    Gerrit-Attention: Fred Shih <ff...@chromium.org>
    Gerrit-Comment-Date: Thu, 25 Jun 2026 20:26:14 +0000
    Gerrit-HasComments: No
    Gerrit-Has-Labels: Yes
    satisfied_requirement
    unsatisfied_requirement
    open
    diffy

    Fred Shih (Gerrit)

    unread,
    6:29 PM (3 hours ago) 6:29 PM
    to manuk hovanesian, Tomasz Wiszkowski, Ameur Hosni, Christian Xu, Chromium IPC Reviews, android-bu...@system.gserviceaccount.com, Chromium LUCI CQ, chromium...@chromium.org, ios-revie...@chromium.org, ios-r...@chromium.org, marq+...@chromium.org, christia...@chromium.org, chromium-a...@chromium.org, extension...@chromium.org, jdonnel...@chromium.org, omnibox-...@chromium.org
    Attention needed from Ameur Hosni, Christian Xu and manuk hovanesian

    Fred Shih voted Code-Review+1

    Code-Review+1
    Open in Gerrit

    Related details

    Attention is currently required from:
    • Ameur Hosni
    • Christian Xu
    • manuk hovanesian
    Gerrit-Comment-Date: Thu, 25 Jun 2026 22:29:30 +0000
    Gerrit-HasComments: No
    Gerrit-Has-Labels: Yes
    satisfied_requirement
    unsatisfied_requirement
    open
    diffy
    Reply all
    Reply to author
    Forward
    0 new messages