Touch events do not respect display rotation. |
|||||||||||||||
Issue descriptionChrome Version: 69.0.3473.0 OS: ChromeOS What steps will reproduce the problem? (1) Open GMail in a tab. (2) Flip Pixelbook into "tablet" mode. (3) Turn it "upside down" i.e. hinge at top. (4) Tap on e.g. Archive button of an email. (5) Turn device "right way up", i.e. hinge at bottom. (6) Tap on Archive button. What is the expected result? Expect that tapping Archive button at #4 archives the email. What happens instead? There is no effect to tapping the Archive button. When the device is turned right-way-up the button works as expected. If you tap when it is upside down then it is clear that taps are mis-interpreted as being 180-degrees rotated from where you tapped. Unsure if this also affects other platforms, or only ChromeOS.
,
Jul 10
Happens to me too. I am on the latest beta 68.0.3440.40 and Google pixelbook. When in portrait mode every touch at the top of the screen activates something that is in lower portion of the screen. Note that Chrome tabs and tools menus touching works, only what is the body the tab does not.
,
Jul 10
Chromebook X2 user and I have this problem as well
,
Jul 10
Tagging M68 as per #2.
,
Jul 13
Touch issues on Chromebook Tab 10 dev channel, Gmail seems to be particularly susceptible (in its full fat version - basic html view is better). Touches either not registered, or assigned to wrong place. Rotating the tablet occasionally, but not always, helps.
,
Jul 15
Samsung Chromebook Pro user here, happens to me on the dev channel with CROS 69.0.3486.0 PS Only commenting to confirm the issue happens on the CBPro / Caroline as well
,
Jul 16
This bug is marked as a 68 stable release blocker, and we are coming up on stable in a couple weeks, so we need to get it fixed or reclassify it. Oshima, is this in your jurisdiction?
,
Jul 16
This may be related to viz hit testing finch experiement. Can you try disabing "enable-viz-hit-test-draw-quad"?
,
Jul 17
Disabling the "enable-viz-hit-test-draw-quad" flag seems to fix things on a Chromebook Tab 10 in dev channel.
,
Jul 17
Disabling "enable-viz-hit-test-draw-quad" seems to fix the issue on the latest Beta channel / Pixelbook.
,
Jul 17
thank you for confirming. riajiang@, can you look into this?
,
Jul 17
,
Jul 17
,
Jul 17
I can repro this on ChromeOS with VizHitTestDrawQuad turned on and display rotated 90 degrees and 180 degrees; can't repro on Android landscape mode. Right now the upmost level of transform we are using in VizHitTestDrawQuad is the transform_to_root_target in RenderPass [1], which I thought included transform to the display/ root window? Is there another level of transform for the display? If so could someone please point me to that? Thanks! [1] https://cs.chromium.org/chromium/src/components/viz/common/quads/render_pass.h?q=transform_to_root_targ&sq=package:chromium&g=0&l=123
,
Jul 17
Which channels does this experiment apply to? (If this is only affecting Dev and Canary channel users then we can remove ReleaseBlock-Stable)
,
Jul 17
It applies to canary, dev and beta 50%. Since this is only affecting ChromeOS, I'll turn on stable 1% for other platforms.
,
Jul 17
#14, it looks to me that the transform is not applied to the content's frames. (or wrong transform is applied).
,
Jul 19
May be related to internal bug for Android Apps: http://b/111611913
,
Jul 19
,
Jul 19
Here's a user report on M69 Dev that disabling #enable_viz_hit_test_draw_quad had no impact. https://listnr.corp.google.com/report/85554819615
,
Jul 19
User reports from the Android App bug (with logs) - all on M69 Dev: veyron-minnie - https://listnr.corp.google.com/report/85549720280 eve - https://listnr.corp.google.com/report/85530606071 eve - https://listnr.corp.google.com/report/85546619336 scarlet - https://listnr.corp.google.com/report/85531954707
,
Jul 19
For this specific bug on ChromeOS (root transform not applied to event location), I have a CL in review https://chromium-review.googlesource.com/c/chromium/src/+/1140926. I'll take a look to see if the one for Android Apps is related.
,
Jul 19
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/4c1d89115ee1b07d9c5fe7cea0ba721ebdc130ba commit 4c1d89115ee1b07d9c5fe7cea0ba721ebdc130ba Author: Ria Jiang <riajiang@chromium.org> Date: Thu Jul 19 22:24:34 2018 Apply root window transform to event location in VizHitTestDrawQuad. On ChromeOS, we can apply transform to the root window, e.g. rotation. Before sending an event location to HitTestQuery for targeting, we need to account for this transform so that the location is correctly in the coordinate space of the root surface. This CL updates GetOffsetFromRootSurface to be TransformPointToRootSurface - instead of just getting the offset from RenderWidgetHostView to root surface, we pass the point to different RenderWidgetHostView implementations so that RenderWidgetHostViewAura can apply root window's transform to the point, other implementations remain the same. Bug: 860356 Test: site_per_process_hit_test_browsertest Change-Id: Ieed02489c9a225fbb7a4f7a42a9788878079acb1 Reviewed-on: https://chromium-review.googlesource.com/1140926 Reviewed-by: Sadrul Chowdhury <sadrul@chromium.org> Reviewed-by: Avi Drissman <avi@chromium.org> Commit-Queue: Ria Jiang <riajiang@chromium.org> Cr-Commit-Position: refs/heads/master@{#576662} [modify] https://crrev.com/4c1d89115ee1b07d9c5fe7cea0ba721ebdc130ba/content/browser/renderer_host/render_widget_host_input_event_router.cc [modify] https://crrev.com/4c1d89115ee1b07d9c5fe7cea0ba721ebdc130ba/content/browser/renderer_host/render_widget_host_view_android.cc [modify] https://crrev.com/4c1d89115ee1b07d9c5fe7cea0ba721ebdc130ba/content/browser/renderer_host/render_widget_host_view_android.h [modify] https://crrev.com/4c1d89115ee1b07d9c5fe7cea0ba721ebdc130ba/content/browser/renderer_host/render_widget_host_view_aura.cc [modify] https://crrev.com/4c1d89115ee1b07d9c5fe7cea0ba721ebdc130ba/content/browser/renderer_host/render_widget_host_view_aura.h [modify] https://crrev.com/4c1d89115ee1b07d9c5fe7cea0ba721ebdc130ba/content/browser/renderer_host/render_widget_host_view_base.cc [modify] https://crrev.com/4c1d89115ee1b07d9c5fe7cea0ba721ebdc130ba/content/browser/renderer_host/render_widget_host_view_base.h [modify] https://crrev.com/4c1d89115ee1b07d9c5fe7cea0ba721ebdc130ba/content/browser/renderer_host/render_widget_host_view_child_frame.cc [modify] https://crrev.com/4c1d89115ee1b07d9c5fe7cea0ba721ebdc130ba/content/browser/renderer_host/render_widget_host_view_child_frame.h [modify] https://crrev.com/4c1d89115ee1b07d9c5fe7cea0ba721ebdc130ba/content/browser/renderer_host/render_widget_host_view_mac.h [modify] https://crrev.com/4c1d89115ee1b07d9c5fe7cea0ba721ebdc130ba/content/browser/renderer_host/render_widget_host_view_mac.mm [modify] https://crrev.com/4c1d89115ee1b07d9c5fe7cea0ba721ebdc130ba/content/browser/site_per_process_hit_test_browsertest.cc [modify] https://crrev.com/4c1d89115ee1b07d9c5fe7cea0ba721ebdc130ba/content/shell/browser/shell_aura.cc [modify] https://crrev.com/4c1d89115ee1b07d9c5fe7cea0ba721ebdc130ba/content/test/test_render_view_host.cc [modify] https://crrev.com/4c1d89115ee1b07d9c5fe7cea0ba721ebdc130ba/content/test/test_render_view_host.h
,
Jul 20
VizHitTestDrawQuad finch experiment has been disabled on ChromeOS. I'll turn it back on after the CL in c#23 gets into Canary/Dev/Beta/Stable channel. The Android Apps bug mentioned in c#18 shouldn't be related to VizHitTestDrawQuad because VizHitTestDrawQuad does hit-testing at a different level that shouldn't be used by Android Apps hit-testing.
,
Jul 23
I've tested this CL on my ChromeOS local build - I believe ChromeOS hasn't released a version with this CL yet https://cros-omahaproxy.appspot.com. The real change to this CL is this line https://chromium-review.googlesource.com/c/chromium/src/+/1140926/8/content/browser/renderer_host/render_widget_host_view_aura.cc to apply root window transform in ChromeOS, other changes are just overrides in subclasses and tests. This should only affect ChromeOS (ChromeOS M68 stable cut is July 24) and is only used in VizHitTestDrawQuad finch experiment which has already been turned off on ChromeOS. Thanks!
,
Jul 23
This bug requires manual review: We are only 0 days from stable. Please contact the milestone owner if you have questions. Owners: cmasso@(Android), kariahda@(iOS), bhthompson@(ChromeOS), abdulsyed@(Desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jul 24
,
Jul 24
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/2677cac0528497947958c59aff044e43c579294b commit 2677cac0528497947958c59aff044e43c579294b Author: Ria Jiang <riajiang@chromium.org> Date: Tue Jul 24 23:44:30 2018 Apply root window transform to event location in VizHitTestDrawQuad. On ChromeOS, we can apply transform to the root window, e.g. rotation. Before sending an event location to HitTestQuery for targeting, we need to account for this transform so that the location is correctly in the coordinate space of the root surface. This CL updates GetOffsetFromRootSurface to be TransformPointToRootSurface - instead of just getting the offset from RenderWidgetHostView to root surface, we pass the point to different RenderWidgetHostView implementations so that RenderWidgetHostViewAura can apply root window's transform to the point, other implementations remain the same. Bug: 860356 Test: site_per_process_hit_test_browsertest Change-Id: Ieed02489c9a225fbb7a4f7a42a9788878079acb1 Reviewed-on: https://chromium-review.googlesource.com/1140926 Reviewed-by: Sadrul Chowdhury <sadrul@chromium.org> Reviewed-by: Avi Drissman <avi@chromium.org> Commit-Queue: Ria Jiang <riajiang@chromium.org> Cr-Original-Commit-Position: refs/heads/master@{#576662} Reviewed-on: https://chromium-review.googlesource.com/1149200 Reviewed-by: Ria Jiang <riajiang@chromium.org> Cr-Commit-Position: refs/branch-heads/3440@{#747} Cr-Branched-From: 010ddcfda246975d194964ccf20038ebbdec6084-refs/heads/master@{#561733} [modify] https://crrev.com/2677cac0528497947958c59aff044e43c579294b/content/browser/renderer_host/render_widget_host_input_event_router.cc [modify] https://crrev.com/2677cac0528497947958c59aff044e43c579294b/content/browser/renderer_host/render_widget_host_view_android.cc [modify] https://crrev.com/2677cac0528497947958c59aff044e43c579294b/content/browser/renderer_host/render_widget_host_view_android.h [modify] https://crrev.com/2677cac0528497947958c59aff044e43c579294b/content/browser/renderer_host/render_widget_host_view_aura.cc [modify] https://crrev.com/2677cac0528497947958c59aff044e43c579294b/content/browser/renderer_host/render_widget_host_view_aura.h [modify] https://crrev.com/2677cac0528497947958c59aff044e43c579294b/content/browser/renderer_host/render_widget_host_view_base.cc [modify] https://crrev.com/2677cac0528497947958c59aff044e43c579294b/content/browser/renderer_host/render_widget_host_view_base.h [modify] https://crrev.com/2677cac0528497947958c59aff044e43c579294b/content/browser/renderer_host/render_widget_host_view_child_frame.cc [modify] https://crrev.com/2677cac0528497947958c59aff044e43c579294b/content/browser/renderer_host/render_widget_host_view_child_frame.h [modify] https://crrev.com/2677cac0528497947958c59aff044e43c579294b/content/browser/renderer_host/render_widget_host_view_mac.h [modify] https://crrev.com/2677cac0528497947958c59aff044e43c579294b/content/browser/renderer_host/render_widget_host_view_mac.mm [modify] https://crrev.com/2677cac0528497947958c59aff044e43c579294b/content/browser/site_per_process_hit_test_browsertest.cc [modify] https://crrev.com/2677cac0528497947958c59aff044e43c579294b/content/shell/browser/shell_aura.cc [modify] https://crrev.com/2677cac0528497947958c59aff044e43c579294b/content/test/test_render_view_host.cc [modify] https://crrev.com/2677cac0528497947958c59aff044e43c579294b/content/test/test_render_view_host.h
,
Jul 25
|
|||||||||||||||
►
Sign in to add a comment |
|||||||||||||||
Comment 1 by kcto...@gmail.com
, Jul 9