Add hmac-secret(-mc) support to DevTools WebAuthn panel [devtools/devtools-frontend : main]

0 views
Skip to first unread message

Byungwoo Lee (Gerrit)

unread,
Mar 19, 2026, 12:43:10 AMĀ (5 days ago)Ā Mar 19
to Eric Leese, Devtools-frontend LUCI CQ, devtools-rev...@chromium.org
Attention needed from Eric Leese

Byungwoo Lee added 1 comment

Patchset-level comments
File-level comment, Patchset 1 (Latest):
Byungwoo Lee . resolved

Hello Eric,

This CL adds 'Supports hmac-secret' and 'Supports hmac-secret-mc' check boxes to the WebAuthn devtools panel.

Could you please review this?

Thank you!

Open in Gerrit

Related details

Attention is currently required from:
  • Eric Leese
Submit Requirements:
  • 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: devtools/devtools-frontend
Gerrit-Branch: main
Gerrit-Change-Id: I29ca266ec744b4339c90551b312a144a29d8a2f0
Gerrit-Change-Number: 7683510
Gerrit-PatchSet: 1
Gerrit-Owner: Byungwoo Lee <bl...@igalia.com>
Gerrit-Reviewer: Byungwoo Lee <bl...@igalia.com>
Gerrit-Reviewer: Eric Leese <le...@chromium.org>
Gerrit-CC: Devtools-frontend LUCI CQ <devtools-fro...@luci-project-accounts.iam.gserviceaccount.com>
Gerrit-Attention: Eric Leese <le...@chromium.org>
Gerrit-Comment-Date: Thu, 19 Mar 2026 04:43:05 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
unsatisfied_requirement
open
diffy

Eric Leese (Gerrit)

unread,
Mar 19, 2026, 4:03:32 AMĀ (5 days ago)Ā Mar 19
to Byungwoo Lee, Devtools-frontend LUCI CQ, devtools-rev...@chromium.org
Attention needed from Byungwoo Lee

Eric Leese added 1 comment

Patchset-level comments
Eric Leese . unresolved

Looks good, but let's wait for these files to be copied as part of the normal browser-protocol roll (hopefully less than a day) and then rebase this change.

Open in Gerrit

Related details

Attention is currently required from:
  • Byungwoo Lee
Submit Requirements:
    • requirement is not satisfiedCode-Owners
    • requirement is not satisfiedCode-Review
    • requirement is not satisfiedNo-Unresolved-Comments
    • 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: devtools/devtools-frontend
    Gerrit-Branch: main
    Gerrit-Change-Id: I29ca266ec744b4339c90551b312a144a29d8a2f0
    Gerrit-Change-Number: 7683510
    Gerrit-PatchSet: 1
    Gerrit-Owner: Byungwoo Lee <bl...@igalia.com>
    Gerrit-Reviewer: Byungwoo Lee <bl...@igalia.com>
    Gerrit-Reviewer: Eric Leese <le...@chromium.org>
    Gerrit-CC: Devtools-frontend LUCI CQ <devtools-fro...@luci-project-accounts.iam.gserviceaccount.com>
    Gerrit-Attention: Byungwoo Lee <bl...@igalia.com>
    Gerrit-Comment-Date: Thu, 19 Mar 2026 08:03:28 +0000
    Gerrit-HasComments: Yes
    Gerrit-Has-Labels: No
    unsatisfied_requirement
    open
    diffy

    Byungwoo Lee (Gerrit)

    unread,
    Mar 22, 2026, 7:07:32 PMĀ (2 days ago)Ā Mar 22
    to Eric Leese, Devtools-frontend LUCI CQ, devtools-rev...@chromium.org
    Attention needed from Eric Leese

    Byungwoo Lee added 2 comments

    Patchset-level comments
    File-level comment, Patchset 1:
    Eric Leese . resolved

    Looks good, but let's wait for these files to be copied as part of the normal browser-protocol roll (hopefully less than a day) and then rebase this change.

    Byungwoo Lee

    Done

    File-level comment, Patchset 2 (Latest):
    Byungwoo Lee . resolved

    Thank you for the review! I rebased after the browser-protocol roll.
    Could you please take another look?

    Open in Gerrit

    Related details

    Attention is currently required from:
    • Eric Leese
    Submit Requirements:
      • 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: devtools/devtools-frontend
      Gerrit-Branch: main
      Gerrit-Change-Id: I29ca266ec744b4339c90551b312a144a29d8a2f0
      Gerrit-Change-Number: 7683510
      Gerrit-PatchSet: 2
      Gerrit-Owner: Byungwoo Lee <bl...@igalia.com>
      Gerrit-Reviewer: Byungwoo Lee <bl...@igalia.com>
      Gerrit-Reviewer: Eric Leese <le...@chromium.org>
      Gerrit-CC: Devtools-frontend LUCI CQ <devtools-fro...@luci-project-accounts.iam.gserviceaccount.com>
      Gerrit-Attention: Eric Leese <le...@chromium.org>
      Gerrit-Comment-Date: Sun, 22 Mar 2026 23:07:29 +0000
      Gerrit-HasComments: Yes
      Gerrit-Has-Labels: No
      Comment-In-Reply-To: Eric Leese <le...@chromium.org>
      unsatisfied_requirement
      open
      diffy

      Eric Leese (Gerrit)

      unread,
      Mar 23, 2026, 9:29:08 AMĀ (20 hours ago)Ā Mar 23
      to Byungwoo Lee, Devtools-frontend LUCI CQ, devtools-rev...@chromium.org
      Attention needed from Byungwoo Lee

      Eric Leese voted Code-Review+1

      Code-Review+1
      Open in Gerrit

      Related details

      Attention is currently required from:
      • Byungwoo Lee
      Submit Requirements:
      • 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: devtools/devtools-frontend
      Gerrit-Branch: main
      Gerrit-Change-Id: I29ca266ec744b4339c90551b312a144a29d8a2f0
      Gerrit-Change-Number: 7683510
      Gerrit-PatchSet: 2
      Gerrit-Owner: Byungwoo Lee <bl...@igalia.com>
      Gerrit-Reviewer: Byungwoo Lee <bl...@igalia.com>
      Gerrit-Reviewer: Eric Leese <le...@chromium.org>
      Gerrit-CC: Devtools-frontend LUCI CQ <devtools-fro...@luci-project-accounts.iam.gserviceaccount.com>
      Gerrit-Attention: Byungwoo Lee <bl...@igalia.com>
      Gerrit-Comment-Date: Mon, 23 Mar 2026 13:29:05 +0000
      Gerrit-HasComments: No
      Gerrit-Has-Labels: Yes
      satisfied_requirement
      unsatisfied_requirement
      open
      diffy

      Danil Somsikov (Gerrit)

      unread,
      Mar 23, 2026, 9:32:16 AMĀ (20 hours ago)Ā Mar 23
      to Byungwoo Lee, Eric Leese, Devtools-frontend LUCI CQ, devtools-rev...@chromium.org
      Attention needed from Byungwoo Lee

      Danil Somsikov voted Code-Review+1

      Code-Review+1
      Open in Gerrit

      Related details

      Attention is currently required from:
      • Byungwoo Lee
      Submit Requirements:
      • 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: devtools/devtools-frontend
      Gerrit-Branch: main
      Gerrit-Change-Id: I29ca266ec744b4339c90551b312a144a29d8a2f0
      Gerrit-Change-Number: 7683510
      Gerrit-PatchSet: 2
      Gerrit-Owner: Byungwoo Lee <bl...@igalia.com>
      Gerrit-Reviewer: Byungwoo Lee <bl...@igalia.com>
      Gerrit-Reviewer: Danil Somsikov <d...@chromium.org>
      Gerrit-Reviewer: Eric Leese <le...@chromium.org>
      Gerrit-CC: Devtools-frontend LUCI CQ <devtools-fro...@luci-project-accounts.iam.gserviceaccount.com>
      Gerrit-Attention: Byungwoo Lee <bl...@igalia.com>
      Gerrit-Comment-Date: Mon, 23 Mar 2026 13:32:13 +0000
      Gerrit-HasComments: No
      Gerrit-Has-Labels: Yes
      satisfied_requirement
      open
      diffy

      Byungwoo Lee (Gerrit)

      unread,
      Mar 23, 2026, 9:20:49 PMĀ (8 hours ago)Ā Mar 23
      to Danil Somsikov, Eric Leese, Devtools-frontend LUCI CQ, devtools-rev...@chromium.org

      Byungwoo Lee voted and added 1 comment

      Votes added by Byungwoo Lee

      Commit-Queue+2

      1 comment

      Patchset-level comments
      File-level comment, Patchset 2 (Latest):
      Byungwoo Lee . resolved

      Thank you for the review!

      Open in Gerrit

      Related details

      Attention set is empty
      Submit Requirements:
      • 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: devtools/devtools-frontend
      Gerrit-Branch: main
      Gerrit-Change-Id: I29ca266ec744b4339c90551b312a144a29d8a2f0
      Gerrit-Change-Number: 7683510
      Gerrit-PatchSet: 2
      Gerrit-Owner: Byungwoo Lee <bl...@igalia.com>
      Gerrit-Reviewer: Byungwoo Lee <bl...@igalia.com>
      Gerrit-Reviewer: Danil Somsikov <d...@chromium.org>
      Gerrit-Reviewer: Eric Leese <le...@chromium.org>
      Gerrit-CC: Devtools-frontend LUCI CQ <devtools-fro...@luci-project-accounts.iam.gserviceaccount.com>
      Gerrit-Comment-Date: Tue, 24 Mar 2026 01:20:46 +0000
      Gerrit-HasComments: Yes
      Gerrit-Has-Labels: Yes
      satisfied_requirement
      open
      diffy

      Devtools-frontend LUCI CQ (Gerrit)

      unread,
      Mar 23, 2026, 10:06:07 PMĀ (8 hours ago)Ā Mar 23
      to Byungwoo Lee, Danil Somsikov, Eric Leese, devtools-rev...@chromium.org

      Devtools-frontend LUCI CQ submitted the change

      Change information

      Commit message:
      Add hmac-secret(-mc) support to DevTools WebAuthn panel

      Add "Supports hmac-secret" and "Supports hmac-secret-mc" checkboxes to
      "New Authenticator" section of the DevTools WebAuthn panel, so that
      the hmac-secret-mc extension behavior can be tested with a virtual
      authenticator in DevTools.

      Depends on Chromium CL: https://crrev.com/c/7673678
      Bug: 420689820
      Change-Id: I29ca266ec744b4339c90551b312a144a29d8a2f0
      Reviewed-by: Danil Somsikov <d...@chromium.org>
      Commit-Queue: Byungwoo Lee <bl...@igalia.com>
      Reviewed-by: Eric Leese <le...@chromium.org>
      Files:
      • M front_end/panels/webauthn/WebauthnPane.ts
      • M front_end/ui/visual_logging/KnownContextValues.ts
      Change size: M
      Delta: 2 files changed, 54 insertions(+), 0 deletions(-)
      Branch: refs/heads/main
      Submit Requirements:
      • requirement satisfiedCode-Review: +1 by Danil Somsikov, +1 by Eric Leese
      Open in Gerrit
      Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
      Gerrit-MessageType: merged
      Gerrit-Project: devtools/devtools-frontend
      Gerrit-Branch: main
      Gerrit-Change-Id: I29ca266ec744b4339c90551b312a144a29d8a2f0
      Gerrit-Change-Number: 7683510
      Gerrit-PatchSet: 3
      Gerrit-Owner: Byungwoo Lee <bl...@igalia.com>
      Gerrit-Reviewer: Byungwoo Lee <bl...@igalia.com>
      Gerrit-Reviewer: Danil Somsikov <d...@chromium.org>
      Gerrit-Reviewer: Devtools-frontend LUCI CQ <devtools-fro...@luci-project-accounts.iam.gserviceaccount.com>
      Gerrit-Reviewer: Eric Leese <le...@chromium.org>
      open
      diffy
      satisfied_requirement
      Reply all
      Reply to author
      Forward
      0 new messages