Closed Bug 1550437 Opened 6 years ago Closed 5 years ago

Update spinner in the About dialog is not animating anymore during an update download

Categories

(Core :: Graphics, defect)

defect
Not set
normal

Tracking

()

VERIFIED FIXED
mozilla69
Tracking Status
firefox-esr60 --- unaffected
firefox66 --- unaffected
firefox67 --- unaffected
firefox68 --- verified
firefox69 --- verified

People

(Reporter: itiel_yn8, Assigned: whawkins)

References

(Regression)

Details

(Keywords: regression)

Attachments

(1 file, 1 obsolete file)

This started about a month ago.

STR:

  1. Run a not-so-up-to-date Nightly build
  2. Open menu > Help > About Nightly

During the downloading of the update, you'll see the spinner "stuck" and not animating (note that at times it won't appear at all).

Not sure how to bisect using Mozregression, because it sets a policy to disable updates.
Is there a way to disable this policy pre (or during) bisection?

I can reproduce the issue on Nightly68 Windows10.
I tried with manually downloading from tools.taskcluster.net/index/gecko.v2 and spoofing update channel.

Regression window:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=402c24bed3464222e62965b67142f9ea3a92e48e&tochange=a0dcfd3d82f2f8c2407afbb45de34d455156a048

Suspect: f5dd833c3b60b3365d2c49d930748e33a22651b6 Will Hawkins — Bug 1354913: Fix nsDeckFrame, nsImageBoxFrame and nsImageFrame so that nsDeckFrame does not tick the refresh driver when its child images are animated and hidden. r=aosmond

Regressed by: 1354913

Will Hawkins,
Your patch seems to cause the regression. Can you please look into this?

Flags: needinfo?(whawkins)
Assignee: nobody → whawkins

Will do you think you'll get to this in time for 68?

Sorry for not responding until now, what is the deadline for 68?

Flags: needinfo?(jcristau)

68 is beta now, and about 5 weeks away from RC week.

Flags: needinfo?(jcristau)
Component: General → Graphics
Product: Firefox → Core

I am working on this now.

Flags: needinfo?(whawkins)

There is a patch ready for this and I have tested. Alice0775 White do you build from source and, if so, do you want to test this in your setup?

Flags: needinfo?(alice0775)

(In reply to Will Hawkins from comment #8)

There is a patch ready for this and I have tested. Alice0775 White do you
build from source and, if so, do you want to test this in your setup?

How can I simulate update process(download) using the local build?

Flags: needinfo?(alice0775)

(In reply to Alice0775 White from comment #10)

(In reply to Will Hawkins from comment #8)

There is a patch ready for this and I have tested. Alice0775 White do you
build from source and, if so, do you want to test this in your setup?

How can I simulate update process(download) using the local build?

Here's how I did it? Not sure if this is the /best/ way, but it seemed to work for me.

diff --git a/toolkit/mozapps/update/UpdateService.jsm b/toolkit/mozapps/update/UpdateService.jsm
--- a/toolkit/mozapps/update/UpdateService.jsm
+++ b/toolkit/mozapps/update/UpdateService.jsm
@@ -3476,6 +3476,8 @@ Checker.prototype = {
       if (!url) {
         return;
       }
+      url =
+      "https://aus5.mozilla.org/update/6/Firefox/69.0a1/20190604212047/Linux_x86_64-gcc3/en-US/nightly/Linux%204.15.0-50-generic%20(GTK%203.22.30%2Clibpulse%2011.1.0)/ISET:SSE4_2,MEM:31808/default/default/update.xml?force=1"
 
       this._request = new XMLHttpRequest();
       this._request.open("GET", url, true);
Flags: needinfo?(alice0775)

Umm, Update process(downloading) would not start.
Sorry, I cannot.
I think it would be better to make a try build or an ordinary Nightly build.

Flags: needinfo?(alice0775)

Okay, I'm not really sure what you mean ...

You are not able to build locally?

or

You are not able to apply that patch for some reason?

If you can't build locally, no problem! I will get the code into shape and let you know when you can test nightly to make sure that everything is fiex.

Thanks for your help!
Will

Attachment #9070030 - Flags: feedback?(zbraniecki)
Attachment #9070030 - Flags: feedback?(jaws)
Attachment #9070030 - Flags: feedback?(francesco.lodolo)

Comment on attachment 9070030 [details]
Bug 1550437: Animate throbber when download is updating.

Please don't flag multiple people in multiple places ;-)

Attachment #9070030 - Flags: feedback?(zbraniecki)
Attachment #9070030 - Flags: feedback?(jaws)
Attachment #9070030 - Flags: feedback?(francesco.lodolo)

(In reply to Francesco Lodolo [:flod] from comment #14)

Comment on attachment 9070030 [details]
Bug 1550437: Animate throbber when download is updating.

Please don't flag multiple people in multiple places ;-)
Very sorry! I have no idea how these Bugzilla notifications interact with Phabricator. I will make sure not to do it again!

Attachment #9070030 - Attachment is obsolete: true
Keywords: checkin-needed

Pushed by btara@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/640fcc4d6787
Make sure that About dialog animates the throbber. r=aosmond

Keywords: checkin-needed
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla69
Flags: needinfo?(jcristau)

Itiel,

I have confirmed that this update fixes the problem. Can you verify that it looks okay to you? The fix should be in Nightly after June 12th.

Thank you for reporting the problem! I hope that this fixes it!

Will

Flags: needinfo?(jcristau) → needinfo?(itiel_yn8)

(In reply to Will Hawkins from comment #19)

Itiel,

I have confirmed that this update fixes the problem. Can you verify that it looks okay to you? The fix should be in Nightly after June 12th.

Thank you for reporting the problem! I hope that this fixes it!

Will

Working great now, thanks!
Marking as VERIFIED FIXED.

Status: RESOLVED → VERIFIED
Flags: needinfo?(itiel_yn8)

Comment on attachment 9071111 [details]
Bug 1550437: Make sure that About dialog animates the throbber. r=aosmond,jfkthame

Beta/Release Uplift Approval Request

  • User impact if declined: The throbber does not show/animate in the About window when Firefox is downloading/processing an update.
  • Is this code covered by automated tests?: No
  • Has the fix been verified in Nightly?: Yes
  • Needs manual test from QE?: No
  • If yes, steps to reproduce:
  • List of other uplifts needed: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): This fix has been verified by the original reporter to have worked. Most users will not see this animation change. It is in the About window and only visible when that screen is open and there is an update available.
  • String changes made/needed: None
Attachment #9071111 - Flags: approval-mozilla-beta?

Comment on attachment 9071111 [details]
Bug 1550437: Make sure that About dialog animates the throbber. r=aosmond,jfkthame

new regression in 68, verified in nightly, approved for 68.0b11

Attachment #9071111 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Flags: qe-verify+
QA Whiteboard: [qa-triaged]

Waiting for the official update to Beta 68.0b12 to verify this one in the About Nightly window. Leaving the NI? for me to remember

Flags: needinfo?(tbabos)

Verified fixed on latest Beta 68.0b12 on Windows 10, Mac OS 10.14 and Ubuntu 18.04.
Updating from beta11 to beta12 will display the spinner correctly.

QA Whiteboard: [qa-triaged]
Flags: qe-verify+
Flags: needinfo?(tbabos)
Has Regression Range: --- → yes
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: