[Vertical Tabs] Allow OOB Mouse Drag Scroll [chromium/src : main]

0 views
Skip to first unread message

Gazal Agarwal (Gerrit)

unread,
2:16 PM (3 hours ago) 2:16 PM
to Jack Thiesen, Calder Kitagawa, Chromium LUCI CQ, chromium...@chromium.org, davidj...@chromium.org, gogeral...@chromium.org, hanxi...@chromium.org, mattsimm...@chromium.org, meilian...@chromium.org, wychen...@chromium.org, yuezhang...@chromium.org, yusufo...@chromium.org
Attention needed from Calder Kitagawa and Jack Thiesen

Gazal Agarwal voted Code-Review+1

Code-Review+1
Open in Gerrit

Related details

Attention is currently required from:
  • Calder Kitagawa
  • Jack Thiesen
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: If50d7fff0541b378745661d147edfecd91f346e8
Gerrit-Change-Number: 8070119
Gerrit-PatchSet: 1
Gerrit-Owner: Jack Thiesen <jthi...@chromium.org>
Gerrit-Reviewer: Calder Kitagawa <ckit...@chromium.org>
Gerrit-Reviewer: Gazal Agarwal <aga...@google.com>
Gerrit-Reviewer: Jack Thiesen <jthi...@chromium.org>
Gerrit-Attention: Jack Thiesen <jthi...@chromium.org>
Gerrit-Attention: Calder Kitagawa <ckit...@chromium.org>
Gerrit-Comment-Date: Thu, 09 Jul 2026 18:16:07 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
satisfied_requirement
unsatisfied_requirement
open
diffy

Calder Kitagawa (Gerrit)

unread,
2:34 PM (3 hours ago) 2:34 PM
to Jack Thiesen, Gazal Agarwal, Chromium LUCI CQ, chromium...@chromium.org, davidj...@chromium.org, gogeral...@chromium.org, hanxi...@chromium.org, mattsimm...@chromium.org, meilian...@chromium.org, wychen...@chromium.org, yuezhang...@chromium.org, yusufo...@chromium.org
Attention needed from Jack Thiesen

Calder Kitagawa voted and added 1 comment

Votes added by Calder Kitagawa

Code-Review+1

1 comment

Patchset-level comments
File-level comment, Patchset 1 (Latest):
Calder Kitagawa . resolved

LGTM

Open in Gerrit

Related details

Attention is currently required from:
  • Jack Thiesen
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: If50d7fff0541b378745661d147edfecd91f346e8
Gerrit-Change-Number: 8070119
Gerrit-PatchSet: 1
Gerrit-Owner: Jack Thiesen <jthi...@chromium.org>
Gerrit-Reviewer: Calder Kitagawa <ckit...@chromium.org>
Gerrit-Reviewer: Gazal Agarwal <aga...@google.com>
Gerrit-Reviewer: Jack Thiesen <jthi...@chromium.org>
Gerrit-Attention: Jack Thiesen <jthi...@chromium.org>
Gerrit-Comment-Date: Thu, 09 Jul 2026 18:34:50 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
satisfied_requirement
open
diffy

Gazal Agarwal (Gerrit)

unread,
2:41 PM (3 hours ago) 2:41 PM
to Jack Thiesen, Calder Kitagawa, Chromium LUCI CQ, chromium...@chromium.org, davidj...@chromium.org, gogeral...@chromium.org, hanxi...@chromium.org, mattsimm...@chromium.org, meilian...@chromium.org, wychen...@chromium.org, yuezhang...@chromium.org, yusufo...@chromium.org
Attention needed from Jack Thiesen

Gazal Agarwal voted Commit-Queue+2

Commit-Queue+2
Open in Gerrit

Related details

Attention is currently required from:
  • Jack Thiesen
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: If50d7fff0541b378745661d147edfecd91f346e8
Gerrit-Change-Number: 8070119
Gerrit-PatchSet: 1
Gerrit-Owner: Jack Thiesen <jthi...@chromium.org>
Gerrit-Reviewer: Calder Kitagawa <ckit...@chromium.org>
Gerrit-Reviewer: Gazal Agarwal <aga...@google.com>
Gerrit-Reviewer: Jack Thiesen <jthi...@chromium.org>
Gerrit-Attention: Jack Thiesen <jthi...@chromium.org>
Gerrit-Comment-Date: Thu, 09 Jul 2026 18:41:38 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
satisfied_requirement
open
diffy

Chromium LUCI CQ (Gerrit)

unread,
2:46 PM (3 hours ago) 2:46 PM
to Jack Thiesen, Gazal Agarwal, Calder Kitagawa, chromium...@chromium.org, davidj...@chromium.org, gogeral...@chromium.org, hanxi...@chromium.org, mattsimm...@chromium.org, meilian...@chromium.org, wychen...@chromium.org, yuezhang...@chromium.org, yusufo...@chromium.org

Chromium LUCI CQ submitted the change

Change information

Commit message:
[Vertical Tabs] Allow OOB Mouse Drag Scroll

Allow cursor drags to trigger an out-of-bounds scroll. Additionally
disable the OnMoved() block for mouse as a followup bugfix.
Bug: 518307037
Change-Id: If50d7fff0541b378745661d147edfecd91f346e8
Commit-Queue: Gazal Agarwal <aga...@google.com>
Reviewed-by: Gazal Agarwal <aga...@google.com>
Commit-Queue: Jack Thiesen <jthi...@chromium.org>
Reviewed-by: Calder Kitagawa <ckit...@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1659731}
Files:
  • M chrome/android/features/tab_ui/java/src/org/chromium/chrome/browser/tasks/tab_management/TabListItemTouchHelperCallback.java
  • M chrome/android/features/tab_ui/java/src/org/chromium/chrome/browser/tasks/tab_management/vertical_tabs/VerticalTabListItemTouchHelperCallback.java
Change size: S
Delta: 2 files changed, 37 insertions(+), 7 deletions(-)
Branch: refs/heads/main
Submit Requirements:
  • requirement satisfiedCode-Review: +1 by Gazal Agarwal, +1 by Calder Kitagawa
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: If50d7fff0541b378745661d147edfecd91f346e8
Gerrit-Change-Number: 8070119
Gerrit-PatchSet: 2
Gerrit-Owner: Jack Thiesen <jthi...@chromium.org>
Gerrit-Reviewer: Calder Kitagawa <ckit...@chromium.org>
Gerrit-Reviewer: Chromium LUCI CQ <chromiu...@luci-project-accounts.iam.gserviceaccount.com>
Gerrit-Reviewer: Gazal Agarwal <aga...@google.com>
Gerrit-Reviewer: Jack Thiesen <jthi...@chromium.org>
open
diffy
satisfied_requirement
Reply all
Reply to author
Forward
0 new messages