New issue
Advanced search Search tips

Issue 871602 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: Aug 9
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug

Blocking:
issue 841020



Sign in to add a comment

Scroll with touch input does not work on Keyboard Shortcut Helper app

Project Member Reported by sdantul...@chromium.org, Aug 7

Issue description

Google Chrome	69.0.3497.29 (Official Build) dev (64-bit)
Revision	e825743804c33f2218a678227f0aa838d07ecc28-refs/branch-heads/3497@{#407}
Platform	10895.16.0 (Official Build) dev-channel eve

What steps will reproduce the problem?
1. Open Keyboard Shortcut Helper app from launcher
2. Try to scroll using touch input

What is the expected result?
Able to scroll through the page without any issue.

What happens instead?
Unable to scroll through the page.

 
Owner: wutao@chromium.org
Status: Assigned (was: Untriaged)
Cc: msw@chromium.org
Issue not reproduced on turning off the flag: "Keyboard Shortcut Viewer mojo app" in chrome://flags.
Cc: -msw@chromium.org jamescook@chromium.org wutao@chromium.org
Owner: msw@chromium.org
+jamescook@

Hi msw@, could you please take a look.
Labels: ReleaseBlock-Stable
I think this is a known issue in M69 and something msw has either fixed or is fixing in M70. We're hoping to let the app version get to beta channel, get some data, then backport a fix to turn off the app in M69.

RB-Stable so we don't forget

Cc: dxie@chromium.org
Components: Internals>Services>Ash
+dxie FYI

Yeah, this will be fixed by https://chromium-review.googlesource.com/c/chromium/src/+/1152102.
That change will land for M-70 and we'll revert to the non-app KSV for M-70 in the next couple days.
Blocking: 841020
Status: Fixed (was: Assigned)
This should be fixed by 5613aca3bca8598bc814598bb8c90ddc5cd001b1.
Please verify on the next dev after #581760, thanks!
Labels: Merge-TBD
[Auto-generated comment by a script] We noticed that this issue is targeted for M-69; it appears the fix may have landed after branch point, meaning a merge might be required. Please confirm if a merge is required here - if so add Merge-Request-69 label, otherwise remove Merge-TBD label. Thanks.
Labels: -Merge-TBD -ReleaseBlock-Stable -M-69 M-70
We're disabling the app for M-69, so this is no longer blocking, see:
https://chromium-review.googlesource.com/c/chromium/src/+/1169433

Sign in to add a comment