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

Issue 770438 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Jan 14
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug

Blocked on:
issue 890390



Sign in to add a comment

Gesture navigation paints content area black, does not show image of page, with PlzNavigate

Project Member Reported by jamescook@chromium.org, Sep 30 2017

Issue description

Google Chrome	61.0.3163.108 (Official Build) beta (64-bit)
Platform	9765.72.0 (Official Build) beta-channel samus

* Close all tabs and sign out (to ensure you won't have a bunch of tabs on sign in)
* Sign in
* Navigate to a lightweight page like http://xkcd.com/1234
* Navigate to the next comic http://xkcd.com/1235
* Two-finger side scroll back
* Two-finger side scroll forward

Expected: Screenshot of previous page slides in, or failing that a white content area.

Actual: The content area is black as the current page image slides off, then the new page flashes in at the end.

I cannot take a screenshot, as any keypress causes gesture nav to abort. However, it's obvious to see and 100% reproducible.

This is a regression. It does not happen on my wife's samus, running 59.0.3071.134 / OS 9460.73.0. Interestingly it doesn't happen on 62.0.3193.0 (dev) on my caroline. Perhaps there is a fix that needs to be backported? Or it's samus-only?

Calling this ReleaseBlock-Stable because it's a user-visible regression in a commonly used feature.

mohsen, are you the right owner for this?

abodenha, are there known graphics/compositing issues in M61 right now? Other than gesturenav I haven't noticed problems.

 
Photo attached.

IMG_20170929_214903.jpg
448 KB View Download

Comment 2 by sadrul@chromium.org, Sep 30 2017

Does taking screenshots work correctly? (because I believe the same compositor code is involved in both ash0screenshot and gesture-nav screenshot)
Cc: clamy@chromium.org
Regular screenshots work fine.

The problem goes away if I disable browser-side-navigation (PlzNavigate) in about:flags.

+clamy, I'm not sure what component to tag for PlzNavigate.

Summary: Gesture navigation paints content area black, does not show image of page, with PlzNavigate (was: Gesture navigation paints content area black, does not show image of page)
Cc: carlosk@chromium.org creis@chromium.org
Components: -UI>Browser>Navigation>GestureNav UI>Browser>Navigation
+creis, carlosk, looks like clamy is away.

The PSA mentions PlzNavigate performance issues with cached back/forward navigations, I wonder if it could be related, like a race.

https://groups.google.com/a/chromium.org/d/msg/chromium-dev/U0VowGH6ilw/K4Zkrvj5BwAJ

Possibly a duplicate of  issue 767394 ?
Cc: nasko@chromium.org

Comment 8 by nasko@chromium.org, Oct 2 2017

Does it repro if you use the regular buttons for doing back/forward navigations?

Comment 9 by nasko@chromium.org, Oct 2 2017

Labels: Proj-PlzNavigate
Re question in description there is  bug 767680  and friends affecting samus and lulu

I'm unable to repro this on 61.0.3163.108 on my samus.

Is PlzNavigate on by default? Or is it only affecting a portion of the population? If it's finch controlled we should disable it on beta and stable until the issue is fixed (and remove the releaseblock label)
PlzNavigate is indeed on 100% for M61 and later, except Android WebView. It looks like this is affecting specific devices on ChromeOS, so I would rather not disable all of PlzNavigate.
Can this be reproduced on a linux build of Chrome for ChromeOS? If yes, then debugging might be easier to allow us to narrow down a root cause.
Labels: -ReleaseBlock-Stable
I cannot reproduce this on the same version, 61.0.3163.108 on link, with the same account. Also, my repro on samus is now flaky -- sometimes it works OK.

Given that I'm the only one who has seen this, and only on one machine, dropping ReleaseBlock label.

It does not repro with the normal back/forward buttons, just gesture nav.

I don't know how to trigger gesture nav on a linux Chrome for Chrome OS build, as it requires two-finger swipes.

mohsen/sadrul, is there a way to simulate gesture nav on desktop?

Cc: mfomitchev@chromium.org
I'm also unable to repro this on my Minnie. Adding mfomitchev@ who has worked on screenshot gesture nav and might know better what's happening here.

Re #12: You can use touch emulation on desktop and do the gesture nav using touch ('--touch-events=enabled --touch-devices=<id>' where <id> is id of the mouse device you want to use to emulate touch which can be retrieved using 'xinput list' command).


How do you simulate two-finger swipe with touch-emulation?

You can try gesture nav with touchscreen (or touch-emulation). If you specifically want to try gesture nav with touchpad two-finger swipe, I have no idea!
The white background is used when the screenshot is empty or doesn't match the size of the web contents (see https://cs.chromium.org/chromium/src/ui/aura_extra/image_window_delegate.cc?sq=package:chromium&dr=CSs&l=73).

If we see black, this means that the screenshot was "taken", but for some reason it is all black. Considering screenshots work fine otherwise, this sounds like a race condition between the destruction of the renderer and the readback logic. 

Note that we are in the process of replacing the existing gesture nav UI with the "simplified" MD UI that doesn't use screenshots, but that won't happen until M64.
Cc: ahemery@chromium.org
Cc: arthurso...@chromium.org
I was able to test this on linux (on a ChromeOS build).
I haven't seen any black screen.

I tested the swipe back/forward navigation with:
./out/Release/chrome --touch-devices=<mouse_id_from_xinput>
output.mp4
8.3 MB View Download
Cc: -mfomitchev@chromium.org
Blockedon: 890390
Status: WontFix (was: Assigned)
Obsolete as there is no gesture-nav with screenshot anymore.

Sign in to add a comment