Add structs which mirror the new Corp IqStanza messages [chromium/src : main]

0 views
Skip to first unread message

Joe Downing (Gerrit)

unread,
Mar 13, 2026, 8:18:50 PM (19 hours ago) Mar 13
to Yuwei Huang, Chromium LUCI CQ, chromium...@chromium.org, chromotin...@chromium.org
Attention needed from Yuwei Huang

Joe Downing added 1 comment

Patchset-level comments
File-level comment, Patchset 7 (Latest):
Joe Downing . resolved

PTAL!

Open in Gerrit

Related details

Attention is currently required from:
  • Yuwei Huang
Submit Requirements:
  • requirement satisfiedCode-Coverage
  • requirement 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: I3a3d51811eed5dac4cc2061a0b031a50a0bd8340
Gerrit-Change-Number: 7666646
Gerrit-PatchSet: 7
Gerrit-Owner: Joe Downing <joe...@chromium.org>
Gerrit-Reviewer: Joe Downing <joe...@chromium.org>
Gerrit-Reviewer: Yuwei Huang <yuw...@chromium.org>
Gerrit-Attention: Yuwei Huang <yuw...@chromium.org>
Gerrit-Comment-Date: Sat, 14 Mar 2026 00:18:40 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
satisfied_requirement
unsatisfied_requirement
open
diffy

Yuwei Huang (Gerrit)

unread,
Mar 13, 2026, 8:29:46 PM (18 hours ago) Mar 13
to Joe Downing, Chromium LUCI CQ, chromium...@chromium.org, chromotin...@chromium.org
Attention needed from Joe Downing

Yuwei Huang voted Code-Review+1

Code-Review+1
Open in Gerrit

Related details

Attention is currently required from:
  • Joe Downing
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: I3a3d51811eed5dac4cc2061a0b031a50a0bd8340
    Gerrit-Change-Number: 7666646
    Gerrit-PatchSet: 7
    Gerrit-Owner: Joe Downing <joe...@chromium.org>
    Gerrit-Reviewer: Joe Downing <joe...@chromium.org>
    Gerrit-Reviewer: Yuwei Huang <yuw...@chromium.org>
    Gerrit-Attention: Joe Downing <joe...@chromium.org>
    Gerrit-Comment-Date: Sat, 14 Mar 2026 00:29:33 +0000
    Gerrit-HasComments: No
    Gerrit-Has-Labels: Yes
    satisfied_requirement
    open
    diffy

    Joe Downing (Gerrit)

    unread,
    Mar 13, 2026, 9:26:58 PM (17 hours ago) Mar 13
    to Yuwei Huang, Chromium LUCI CQ, chromium...@chromium.org, chromotin...@chromium.org

    Joe Downing voted and added 1 comment

    Votes added by Joe Downing

    Commit-Queue+2

    1 comment

    Patchset-level comments
    Joe Downing . resolved

    Thanks!

    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: I3a3d51811eed5dac4cc2061a0b031a50a0bd8340
    Gerrit-Change-Number: 7666646
    Gerrit-PatchSet: 7
    Gerrit-Owner: Joe Downing <joe...@chromium.org>
    Gerrit-Reviewer: Joe Downing <joe...@chromium.org>
    Gerrit-Reviewer: Yuwei Huang <yuw...@chromium.org>
    Gerrit-Comment-Date: Sat, 14 Mar 2026 01:26:42 +0000
    Gerrit-HasComments: Yes
    Gerrit-Has-Labels: Yes
    satisfied_requirement
    open
    diffy

    Chromium LUCI CQ (Gerrit)

    unread,
    Mar 13, 2026, 10:14:19 PM (17 hours ago) Mar 13
    to Joe Downing, Yuwei Huang, chromium...@chromium.org, chromotin...@chromium.org

    Chromium LUCI CQ submitted the change

    Change information

    Commit message:
    Add structs which mirror the new Corp IqStanza messages
    Bug: 359620500
    Change-Id: I3a3d51811eed5dac4cc2061a0b031a50a0bd8340
    Reviewed-by: Yuwei Huang <yuw...@chromium.org>
    Commit-Queue: Joe Downing <joe...@chromium.org>
    Cr-Commit-Position: refs/heads/main@{#1599441}
    Files:
    • M remoting/base/BUILD.gn
    • M remoting/internal
    • M remoting/proto/BUILD.gn
    • M remoting/proto/messaging_service.cc
    • M remoting/proto/messaging_service.h
    Change size: L
    Delta: 5 files changed, 310 insertions(+), 3 deletions(-)
    Branch: refs/heads/main
    Submit Requirements:
    • requirement satisfiedCode-Review: +1 by Yuwei Huang
    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: I3a3d51811eed5dac4cc2061a0b031a50a0bd8340
    Gerrit-Change-Number: 7666646
    Gerrit-PatchSet: 8
    Gerrit-Owner: Joe Downing <joe...@chromium.org>
    Gerrit-Reviewer: Chromium LUCI CQ <chromiu...@luci-project-accounts.iam.gserviceaccount.com>
    Gerrit-Reviewer: Joe Downing <joe...@chromium.org>
    Gerrit-Reviewer: Yuwei Huang <yuw...@chromium.org>
    open
    diffy
    satisfied_requirement

    Mitsuru Oshima (Gerrit)

    unread,
    5:39 AM (9 hours ago) 5:39 AM
    to Joe Downing, Chromium LUCI CQ, Yuwei Huang, chromium...@chromium.org, chromotin...@chromium.org

    Mitsuru Oshima added 1 comment

    Patchset-level comments
    File-level comment, Patchset 8 (Latest):
    Mitsuru Oshima . resolved

    Is this intentional? My local repo complains that the submoule commit has difference after this CL.

    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: I3a3d51811eed5dac4cc2061a0b031a50a0bd8340
    Gerrit-Change-Number: 7666646
    Gerrit-PatchSet: 8
    Gerrit-Owner: Joe Downing <joe...@chromium.org>
    Gerrit-Reviewer: Chromium LUCI CQ <chromiu...@luci-project-accounts.iam.gserviceaccount.com>
    Gerrit-Reviewer: Joe Downing <joe...@chromium.org>
    Gerrit-Reviewer: Yuwei Huang <yuw...@chromium.org>
    Gerrit-CC: Mitsuru Oshima <osh...@chromium.org>
    Gerrit-Comment-Date: Sat, 14 Mar 2026 09:39:19 +0000
    Gerrit-HasComments: Yes
    Gerrit-Has-Labels: No
    satisfied_requirement
    open
    diffy

    Sky Malice (Gerrit)

    unread,
    8:57 AM (6 hours ago) 8:57 AM
    to Joe Downing, Chromium LUCI CQ, Mitsuru Oshima, Yuwei Huang, chromium...@chromium.org, chromotin...@chromium.org
    Attention needed from Joe Downing

    Sky Malice added 1 comment

    Patchset-level comments
    Mitsuru Oshima . unresolved

    Is this intentional? My local repo complains that the submoule commit has difference after this CL.

    Sky Malice

    I'm having the same problem. It looks like this should typically move in lock step with the associated DEPS entry, like in https://crrev.com/c/7667242 or the previous https://crrev.com/c/7595031

    I think we should revert this CL.

    Open in Gerrit

    Related details

    Attention is currently required from:
    • Joe Downing
    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: I3a3d51811eed5dac4cc2061a0b031a50a0bd8340
    Gerrit-Change-Number: 7666646
    Gerrit-PatchSet: 8
    Gerrit-Owner: Joe Downing <joe...@chromium.org>
    Gerrit-Reviewer: Chromium LUCI CQ <chromiu...@luci-project-accounts.iam.gserviceaccount.com>
    Gerrit-Reviewer: Joe Downing <joe...@chromium.org>
    Gerrit-Reviewer: Yuwei Huang <yuw...@chromium.org>
    Gerrit-CC: Mitsuru Oshima <osh...@chromium.org>
    Gerrit-CC: Sky Malice <sk...@chromium.org>
    Gerrit-Attention: Joe Downing <joe...@chromium.org>
    Gerrit-Comment-Date: Sat, 14 Mar 2026 12:57:31 +0000
    Gerrit-HasComments: Yes
    Gerrit-Has-Labels: No
    Comment-In-Reply-To: Mitsuru Oshima <osh...@chromium.org>
    satisfied_requirement
    open
    diffy

    Sky Malice (Gerrit)

    unread,
    8:59 AM (6 hours ago) 8:59 AM
    to Joe Downing, Chromium LUCI CQ, Mitsuru Oshima, Yuwei Huang, chromium...@chromium.org, chromotin...@chromium.org
    Attention needed from Joe Downing

    Sky Malice has created a revert of this change

    Related details

    Attention is currently required from:
    • Joe Downing
    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: revert
    satisfied_requirement
    open
    diffy
    Reply all
    Reply to author
    Forward
    0 new messages