New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 648126 link

Starred by 3 users
Status: Fixed
Owner:
dgozman@chromium.org
Closed: Mar 2017
Cc:
lfg@chromium.org
wjmaclean@chromium.org
nyerramilli@chromium.org
█ ranjitkan@chromium.org
█ nasko@chromium.org
wfh@chromium.org
creis@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 2
Type: Bug

Blocking:
issue 652783



Sign in to add a comment

DevTools: touch emulation doesn't work for OOPIF.

Reported by jshan...@etouch.net, Sep 19 2016 
Chrome Version: 55.0.2864.0 Revision 533dcff13b0b002494a3040d2f90de937a93e76d-refs/heads/master@{#419385} 
OS: Windows(7,8,10), Mac(10.10.5,10.11.4),Linux(14.04 LTS)

URL: http://www.orimi.com/pdf-test.pdf

Steps:
1. Launch Chrome and navigate to above URL.
2. Press 'F12' key to open dev-tool, click on 'Toggle device toolbar' icon and hover mouse on pdf page 
3. Observe the mouse pointer.

Actual: Device mode pointer (grey round mouse pointer) is not seen after emulating the pdf page i.e glimpse of device mode pointer is seen on emulating page, it does not stays. 

Expected: Device mode pointer (grey round mouse pointer) should be seen after emulating pdf page. 

This is a regression issue broken in 'M-55', will soon update bisect info:

Good Build: 55.0.2841.0
Bad Build: 55.0.2842.0
Actual_video.mp4
753 KB View Download
Expected_video.mp4
351 KB View Download

Comment 1 by msrchandra@chromium.org, Sep 19 2016

Labels: ReleaseBlock-Stable
Owner: nasko@chromium.org
Status: Assigned (was: Unconfirmed)
Using the per-revision bisect providing the bisect results,
Good build: 55.0.28641.0 (Revision: 414854).
Bad build: 55.0.2842.0 (Revision: 414942).

You are probably looking for a change made after 414878 (known good), but no later than 414879 (first known bad).
CHANGELOG URL:
  https://chromium.googlesource.com/chromium/src/+log/5d6411a1a10652b1a3d9ced8d744f7e427918805..d8f0aefde00132b06bd97cb17555f2ec89a0c203

From the CL above, assigning the issue to the concern owner --
@nasko -- Could you please look into the issue, pardon me if it has nothing to do with your changes and if possible please assign it to concern owner.

Note: Adding RB Label as this is a recent Regression. Please remove if not required.
Thank You.

Comment 2 by msrchandra@chromium.org, Sep 19 2016

Labels: hasbisect-per-revison

Comment 3 by nasko@chromium.org, Sep 19 2016

Components: Internals>Sandbox>SiteIsolation

Comment 4 by lfg@chromium.org, Sep 19 2016

Cc: nasko@chromium.org
Owner: lfg@chromium.org

Comment 5 by lfg@chromium.org, Sep 19 2016

Cc: lfg@chromium.org
Labels: -Pri-1 -ReleaseBlock-Stable Pri-2
Owner: dgozman@chromium.org
This is broken because devtools doesn't support out-of-process iframes.

The mouse move events usually get filtered here: https://cs.chromium.org/chromium/src/content/browser/renderer_host/input/touch_emulator.cc?l=148 , but RenderWidgetHosts for out-of-process iframes don't have a TouchEmulator, so the move events go through.

I don't think this should be a release blocker, re-add the label if you disagree.

Comment 6 by dgozman@chromium.org, Sep 19 2016

Labels: -M-55 -Type-Bug-Regression Type-Bug
Summary: DevTools: touch emulation doesn't work for OOPIF. (was: Regression: Device mode mouse pointer is not seen after emulating the pdf page in devtool. )
This is not a blocker. We are going to address this in Q4.

Comment 7 by lfg@chromium.org, Sep 20 2016

Cc: wjmaclean@chromium.org
+wjmaclean@ who is also working around TouchEmulator.

Comment 8 by ligim...@chromium.org, Sep 27 2016

Labels: -hasbisect-per-revison hasbisect-per-revision
Fixing the spelling error in label.

Comment 9 by dgozman@chromium.org, Oct 4 2016

Blocking: 652783

Comment 10 by jmukthavaram@chromium.org, Oct 6 2016 

 Issue 653036  has been merged into this issue.

Comment 13 by dgozman@chromium.org, Mar 8 2017

Status: Fixed (was: Assigned)
This should be fixed for regular iframes. Note that pdf will not work since we disable plugins in mobile device emulation to be closer to real devices.
►

Sign in to add a comment