Issue metadata
Sign in to add a comment
|
[Stable Feedback] Loading progress bar freeze even if webpage is finished loading |
||||||||||||||||||||||
Issue descriptionChrome Version: 62.0.3202.66, 62.0.3202.73 OS: Android What steps will reproduce the problem? We saw 3 users report this issue in these devices: SM-J700F Lenovo A2010-a Samsung Grand Prime Plus In-app Feedback https://listnr.corp.google.com/report/77121974041 https://listnr.corp.google.com/report/79849794751 Play review https://listnr.corp.google.com/report/80492006050
,
Nov 3 2017
,
Nov 3 2017
Tagging as RBS for M62 stable. In case of no upcoming stable refresh, we may need to have a fix for M63 (RBB).
,
Nov 3 2017
pnangunoori@ - the video at http://go/chrome-androidlogs/780856 doesn't seem to be showing the issue. Can you please attach the right video? I was able to repro this issue on Samsung Galaxy Grand Prime/5.0.2 on few sites like cnet.com and xda-developers.com. Didn't repro on sites like, Google search page, yahoo.com, amazon.com, nytimes.com.
,
Nov 3 2017
+cc japhet for insight about progress bar completion signals from the blink side. I have seen this occur on sites that appear to be done loading but have not (usually due to ads on the page). @askatte does this happen 100% of the time and are you able to repro on earlier versions of Chrome? I'm unable to repro on any of my devices. If we if progress from the web page does not reach 100% or we don't get a 'stop' event from the web page, the progress bar will continue showing. The patch in question only launches a feature that we had been experimenting with large groups for several mile stones. It is unclear to me why this is considered such high priority given the few cases and the fact that 62 is already stable.
,
Nov 3 2017
I was not able to repro it on other devices that i tried, Samsung J7 (SM-J710MN) / 6.0, Lenovo A6000/ 5.0 and Samsung S7 / 7.0.1. I have been able to repro the issue consistently only for xda-developers.com. Didn't see the issue for cnet.com the next time. It also reproed with engadget.com a couple times but not 100% of times. I was not able to repro the issue in M61 - 61.0.3163.98. Also, note that the progress bar then stuck progress bar disappears once we try to drag the omnibox to either side. Since its a regression and not too old, adding back M63 RBS.
,
Nov 4 2017
Issue can be reproducible on the mentioned device with below steps: Application Version :62.0.3202.73 Android Build Number : LMY48B Device: Samsung Galaxy Grand Prime 4G(SM-G531F) Steps to reproduce:. 1.launch chrome 2.Open new tab 3.launch https://www.xda-developers.com/ and wait for the page to load fully Observed results: Progress bar freeze after loading Expected results: Progress bar should disappear after loading Number of times you were able to reproduce: 5/5 Note: Good build: 62.0.3202.44 Bad build : 62.0.3202.45 Bisect Range: https://chromium.googlesource.com/chromium/src/+log/62.0.3202.44..62.0.3202.45?pretty=fuller&n=10000 Not able to bisect since it shows all commit positions are same. Also there are no enough CL's went for change here. Suspecting this CL: https://chromium.googlesource.com/chromium/src/+/62acd607eeb44ca7b4ff49e89bb3123947c6c45d Please find logs and Video @ http://go/chrome-androidlogs1/7/780856
,
Nov 6 2017
The suspected CL only adds an option back to the context menu and has no relation to status bars.
,
Nov 6 2017
Since it seems we now have a repro, I'll grab a device and take a look.
,
Nov 8 2017
Any update here mdjones@?
,
Nov 9 2017
In-flight patch here: https://chromium-review.googlesource.com/c/chromium/src/+/756233
,
Nov 13 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/0234f4bde0871f66ca3c0964eb996daf8e5aea00 commit 0234f4bde0871f66ca3c0964eb996daf8e5aea00 Author: Matthew Jones <mdjones@chromium.org> Date: Mon Nov 13 21:20:36 2017 Prevent progress bar from freezing In cases where the indeterminate animation was running and stopped, if progress jumped to 100%, the progress bar would remain on-screen. In most cases, if the progress bar reaches 100%, the progress bar is hidden immediately. If the animation is running, the user should be able to see progress reach 100% before it disappears. The discrepancy was in this logic. This change ensures that the progress bar restarts animation on update and blocks finish until the animation is complete. BUG= 780856 Change-Id: Ife33e5d69824cb48b03a38312f817a31963b848a Reviewed-on: https://chromium-review.googlesource.com/766848 Reviewed-by: Ted Choc <tedchoc@chromium.org> Commit-Queue: Matthew Jones <mdjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#516052} [modify] https://crrev.com/0234f4bde0871f66ca3c0964eb996daf8e5aea00/chrome/android/java/src/org/chromium/chrome/browser/widget/ToolbarProgressBar.java
,
Nov 16 2017
Test: 1.launch chrome 2.Open new tab 3.launch https://www.xda-developers.com/ and wait for the page to load fully Expected Behavior: Progress bar disappears after loading Works as per expected behavior, verified on 64.0.3269.0
,
Nov 16 2017
I honestly don't think this is critical enough to merge back to M-63 this late in the game (especially if it is already seen in M-62). If anyone feels differently, re-add the label; otherwise I'm marking this as fixed.
,
Nov 16 2017
[Auto-generated comment by a script] We noticed that this issue is targeted for M-62; it appears the fix may have landed after branch point, meaning a merge might be required. Please confirm if a merge is required here - if so add Merge-Request-62 label, otherwise remove Merge-TBD label. Thanks.
,
Nov 16 2017
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by pnangunoori@chromium.org
, Nov 3 2017Labels: -Type-Bug -Pri-3 -Needs-Bisect hasbisect-per-revision M-64 Needs-triage-Mobile Triaged-Mobile Pri-1 Type-Bug-Regression
Owner: mdjones@chromium.org
Status: Assigned (was: Untriaged)