1470299 - In Nexus 4 API 23, m.yelp.de progress stuck at 25%

Status: RESOLVED FIXED

(GeckoView :: General, defect, P1)

Description

[No-Jun Park [:njpark]]

On latest klar geckoview, going to m.yelp.com on Nexus 4 5.1 device causes the site to continuously reload.  This is different from previous behavior, where the progressbar was not completing/disappearing.

Comment 1

[Chris Peterson [:cpeterson]]

No-Jun says the site loads correctly on his Pixel 2. The site loads correctly for me on my Moto G5 Plus.

It sounds like this is a recent regression.

Comment 2

[No-Jun Park [:njpark]]

On today's Klar Geckoview, instead of reloading, it exhibits https://bugzilla.mozilla.org/show_bug.cgi?id=1454477#c3 (progressbar stuck at 25%)

Comment 3

[Mike Taylor [:miketaylr]]

Chris, are we shipping Klar/Geckoview in 62 or 63?

Comment 4

[Liz Henry (:lizzard) (relman/hg->git project)]

Looks like the plan as of June 20 is to ship in 62.

Comment 5

[Chris Peterson [:cpeterson]]

(In reply to Liz Henry (:lizzard) (needinfo? me) from comment #4)
> Looks like the plan as of June 20 is to ship in 62.

Yes. We're aiming to ship GeckoView 62 in Focus (7.0 in September). We'll find an owner for this P1 bug in GeckoView triage tomorrow.

Comment 6

[James Willcox (:snorp) ([e-mail]) (he/him)]

I was able to reproduce this on a Nexus 4 on cnn.com. I tried m.yelp.de and it seemed fine. Looks like Gecko is just not sending us a message via the progress listener. Don't know why yet. Investigating further...

Comment 7

[Chris Peterson [:cpeterson]]

[geckoview:klar:p1] because this is a Focus+GV blocker.

Comment 8

[Eugen Sawin [:esawin]]

With bug 1437988 landed, can we move away from using onPageStop as a signal for the progress bar on Focus, Emily?

The progress bar is a visual indicator for the visual page load progression, it should probably be based entirely on the value emitted by onProgressChanged.

On the GV side, we will continue investigating why on some page-device combinations onPageStop is not fired, but generally this issue not directly affecting the visual page load progress.

Comment 9

[Eugen Sawin [:esawin]]

I think Emily might be on PTO, Sebastian, can you look at comment 8?

Comment 10

[Emily Kager [:ekager]]

Merged a fix into Focus master that uses 100% progress to stop the progress bar.

Comment 11

[Eugen Sawin [:esawin]]

No-Jun, can you please verify that this issue is fixed on Focus' nightly channel?

Comment 12

[No-Jun Park [:njpark]]

Just verified on nexus 4 that with Geckoview the progressbar progresses normally, and disappears.  Can't reproduce the original issue anymore after multiple tries.