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

Issue 701715 link

Starred by 2 users

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug-Regression

Blocked on:
issue 672921



Sign in to add a comment

[Pull to refresh] Refresh animation getting displayed twice in "plus.google.com" page

Project Member Reported by dknandiraju@chromium.org, Mar 15 2017

Issue description

Application Version (from "Chrome Settings > About Chrome"): 59.0.3042.0
Android Build Number (from "Android Settings > About Phone/Tablet"): MRA58K
Device: Lenovo K50a40

Steps to reproduce: 
1) Launch chrome
2) Visit any posts in "plus.google.com" page (Eg: https://plus.google.com/collection/su3EW)
3) Pull To refresh Page

Observed behavior: Refresh animation appears twice 

Expected behavior: only one refresh animation should be visible

Frequency: 100%
<number of times you were able to reproduce> 

Additional comments: 

 

Comment 1 Deleted

Note: 
1) This issue exists on all devices.
2) This issue also exists on al builds of M56, m57 and M58 milestones
3) Please find logs and video @http://go/chrome-androidlogs1/6/701715
Cc: tedc...@chromium.org
Owner: jdduke@chromium.org
Status: Assigned (was: Untriaged)
tedchoc@, 

Could you please assign this issue to other developers if jdduke@ is unavailable at the moment, thanks!
Owner: aelias@chromium.org
Status: WontFix (was: Assigned)
This is a problem with google plus.  They are drawing their own refresh spinner (if you load the page on desktop chrome with mobile mode emulation via dev tools), you'll see they show the spinner on pull down.

There might be signals they can give to chrome that would prevent our spinner from showing, but aelias@ would definitely know that better than me.

Comment 5 by jdduke@chromium.org, Mar 15 2017

Cc: rbyers@chromium.org

Comment 6 by jdduke@chromium.org, Mar 15 2017

Sounds like we should punish the developer who added pull-to-refresh to Chrome on Android in the first place.

There were a number of workarounds at the time it was added, though I imagine there might be some cleaner APIs for addressing the issue on the Plus side.

Comment 7 by aelias@chromium.org, Mar 15 2017

Cc: aelias@chromium.org dtapu...@chromium.org
Components: Internals>Input>Touch>Screen
Labels: -Restrict-View-Google Type-Bug-Regression
Owner: ----
Status: Available (was: WontFix)
No repro on 55.0.2883.105, so I think this is likely caused by the passive document-level listener intervention which launched in M56: https://www.chromestatus.com/feature/5093566007214080
Even with the option off (you can disable it via about://flags) I'm able to get into situations where there are multiple pull to refresh images. You need to do it on first load because it appears the pull to refresh item gets added then.

But it does seem like it is more reproducible with the flags on. The page though is much more responsive in scrolling with the flags on.

Comment 9 by aelias@chromium.org, Mar 15 2017

Blockedon: 656801
Owner: sunyunjia@chromium.org
Status: Assigned (was: Available)
Sounds good.  Particularly as there is a preexisting bug and the intervention improves performance, I don't think we should necessarily recommend to G+ team that they mark the listener nonpassive.  Let's tell them to use "overscroll-action" when it ships.
Blockedon: -656801 672921

Sign in to add a comment