Issue metadata
Sign in to add a comment
|
Mojo UI Service: Mouse input does not work over browser window |
||||||||||||||||||||||
Issue descriptionToT on Eve. This regression appears to have been introduced in the last few days. It definitely wasn't here last week. Repro: (1) Log in. (2) Open a Chrome/Chromium browser window. (3) Wait a few seconds. Expected: Use the mouse/trackpad to click/scroll in the window. Actual: Cannot click/scroll in the window. This includes both the web contents as well as the native section at the top which includes the tab strip, address bar, etc. I can navigate through the page using the keyboard by tabbing through focus-able elements, and I can still use the mouse/trackpad to click/scroll the bottom elements (black bar at the bottom, quick settings). CC'ing sheriffs/gardener.
,
Apr 26 2018
Disabling the flag fixed the issue, so it looks like it was due to the #mus flag. We should probably disable the #mus flag until the issue is fixed. I'd say this should still be a P1 until that occurs.
,
Apr 26 2018
Assigning to jamescook@ for now; reassign if needed :)
,
Apr 26 2018
I haven't tested on a device, but this doesn't repro on linux-chromeos with ToT @ #553908 or #554177. It'd be nice to bisect and find the cause -> khorimoto@ what git position did you build Chrome from? And yeah, we should probably disable #mus on ToT dev builds -> Scott, WDYT?
,
Apr 26 2018
sky@: I haven't tested this on linux-chromeos, so it's possible that this only repros on a real device. I'm currently synced to 9e6579c49468b224cb9351799b96d3090250bd75, which can reproduce the issue.
,
Apr 26 2018
msw, do you have time to look into this? If not, pass back to me (or ping me on IM). I'm at home and don't have a dev device right now. I think we should disable mus while sorting this out, since it will impact cros UI devs working on device.
,
Apr 26 2018
I'm also working from home today, so I can try to repro and bisect tomorrow. I guess we can probably disable mus on ToT dev builds sooner, I just need to figure out how :)
,
Apr 26 2018
rkaplow@: would reverting https://chromium-review.googlesource.com/c/chromium/src/+/970913 disable #mus on ToT dev builds? I'm not familiar with the field trial code... we want the default #mus about:flag behavior to be off on ToT builds.
,
Apr 26 2018
khorimoto@, can you help us find someone else to confirm the issue? I'd rather not speculatively revert the field trial unless it's confirmed and urgent.
,
Apr 26 2018
Reverting the CL in #8 causes my ToT dev build to go from ash::Config::MUS to ash::Config::CLASSIC on linux-chromeos.
,
Apr 26 2018
I'm dry-running a revert at https://chromium-review.googlesource.com/c/chromium/src/+/1031253 just to see if the bots still pass in classic. That said, there's a workaround (disable mus in about:flags), so this can probably wait until tomorrow.
,
Apr 27 2018
+jhawkins, who also saw this issue on Eve. That being said, I think it's fine to wait for tomorrow morning to revert the patch for now if that helps you be more confident in the fix :)
,
Apr 27 2018
I just loaded #554177 onto my Eve and Link devices using go/simplechrome. Eve definitely repros this issue, but it seems like Link does not, I wonder why. James is going to disable #mus on dev builds and bisect to find the cause.
,
Apr 27 2018
,
Apr 27 2018
Discussed with msw and sky, we're going to disable mus while I investigate this issue. The revert from comment 11 is in the CQ. I'm starting a bisect.
,
Apr 27 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/2a40a234f5865e534f635713c2ed7388b8f849ae commit 2a40a234f5865e534f635713c2ed7388b8f849ae Author: James Cook <jamescook@chromium.org> Date: Fri Apr 27 15:48:11 2018 Revert "reland attempt 4: Adds WindowService-ChromeOS fieldtrial" This reverts commit 1d768042590f803064afd52f0216c050715bf570. Reason for revert: Cannot click/scroll in web contents or browser UI. See crbug.com/837359 Original change's description: > reland attempt 4: Adds WindowService-ChromeOS fieldtrial > > More bugs fixed. Landing again. > > I'm not sure on the name here. I went with WindowService-ChromeOS as > at some point we will launch the WindowService on other platforms, but > that won't be anytime soon and is likely to be very different. So, I'm > thinking later on we have the field trial WindowService-Desktop. > > BUG=814898 > TEST=none > TBR=rkaplow@chromium.org > > Change-Id: I474b97bc4e8d5904aa07b8de67e1b961adbac1f7 > Reviewed-on: https://chromium-review.googlesource.com/970913 > Reviewed-by: Scott Violet <sky@chromium.org> > Commit-Queue: Scott Violet <sky@chromium.org> > Cr-Commit-Position: refs/heads/master@{#544415} TBR=sky@chromium.org,rkaplow@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 814898 Change-Id: I619d6424676bd6ede21f5f9bed55b9adc283455f Reviewed-on: https://chromium-review.googlesource.com/1031253 Reviewed-by: James Cook <jamescook@chromium.org> Reviewed-by: Scott Violet <sky@chromium.org> Reviewed-by: Robert Kaplow <rkaplow@chromium.org> Commit-Queue: James Cook <jamescook@chromium.org> Cr-Commit-Position: refs/heads/master@{#554396} [modify] https://crrev.com/2a40a234f5865e534f635713c2ed7388b8f849ae/testing/variations/fieldtrial_testing_config.json
,
Apr 27 2018
This problem is caused by https://chromium-review.googlesource.com/c/chromium/src/+/1025633 ("exo: Fix event dispatch to frame view.") landed April 24. Talked to sky. I'm not going to revert that CL yet. We're going to leave mus disabled for now.
,
May 1 2018
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by jamescook@chromium.org
, Apr 26 2018