New issue
Advanced search Search tips

Issue 600453 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: May 2016
Cc:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

Hangouts App touch coordinates aren't properly scaled in use-zoom-for-dsf mode.

Project Member Reported by malaykeshav@chromium.org, Apr 4 2016

Issue description

Version: 51.0.2696.0
OS: 8108.0.0

What steps will reproduce the problem?
(1)Open Hangouts app
(2)Try tapping anything below the Green navigation bar.

What is the expected output?
The respective touch element should respond to tap. 

What do you see instead?
Nothing happens



 
Owner: osh...@chromium.org
Status: Assigned (was: Untriaged)

Comment 2 by osh...@chromium.org, Apr 21 2016

Labels: -M-51
This is a result of new scaling mode, which has been reverted on 51. I'll look into it when I re-enable it.

Comment 3 by osh...@chromium.org, Apr 21 2016

Summary: Hangouts App touch coordinates aren't properly scaled in use-zoom-for-dsf mode. (was: Hangouts App does not register touch)

Comment 4 by osh...@chromium.org, May 12 2016

Status: Started (was: Assigned)
Project Member

Comment 5 by bugdroid1@chromium.org, May 18 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/cbcc58292dabf419324cddbad21242ddc8be3896

commit cbcc58292dabf419324cddbad21242ddc8be3896
Author: oshima <oshima@chromium.org>
Date: Wed May 18 19:21:31 2016

Disable input scaling only when an input came from renderer

I didn't realized that touch events are directly sent to guest views. This fixes the issue by only skip scaling when events are sent back from renderer.

BUG=485650,  600453 
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation

Review-Url: https://codereview.chromium.org/1977513002
Cr-Commit-Position: refs/heads/master@{#394499}

[modify] https://crrev.com/cbcc58292dabf419324cddbad21242ddc8be3896/content/browser/frame_host/render_widget_host_view_guest.cc
[modify] https://crrev.com/cbcc58292dabf419324cddbad21242ddc8be3896/content/browser/renderer_host/render_widget_host_impl.cc
[modify] https://crrev.com/cbcc58292dabf419324cddbad21242ddc8be3896/content/browser/renderer_host/render_widget_host_impl.h

Comment 6 by osh...@chromium.org, May 18 2016

Status: Fixed (was: Started)
Labels: VerifyIn-52
Labels: VerifyIn-53
Labels: VerifyIn-54

Comment 10 by ka...@chromium.org, Aug 31 2016

Labels: Bulk-Verified
Status: Verified (was: Fixed)

Sign in to add a comment