Provide progress and state feedback via the stub installer taskbar icon

RESOLVED FIXED in Firefox 41

Status

()

RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: rstrong, Assigned: rstrong)

Tracking

unspecified
Firefox 41
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(firefox41 fixed)

Details

Attachments

(2 attachments)

Patch coming up
Created attachment 8601642 [details] [diff] [review]
patch rev1
Assignee: nobody → robert.strong.bugs
Status: NEW → ASSIGNED
Attachment #8601642 - Flags: review?(spohl.mozilla.bugs)
I'm planning to use this bug to read up on NSIS and its scripting language to speed up reviews in the future. Let me know if this bug is too urgent for that.
Comment on attachment 8601642 [details] [diff] [review]
patch rev1

Review of attachment 8601642 [details] [diff] [review]:
-----------------------------------------------------------------

I'll take the review in the meantime spohl.
Very cool rstrong.
Attachment #8601642 - Flags: review?(spohl.mozilla.bugs) → review+
Flags: in-testsuite-
Target Milestone: --- → Firefox 40
https://hg.mozilla.org/mozilla-central/rev/39fdaf04c690
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
status-firefox40: --- → fixed
Resolution: --- → FIXED
And we only build the stub installer for nightlies, so the only way to find out whether a patch actually works is to land it, get it merged to m-c, and then burn a nightly? Fun!

Backed out in https://hg.mozilla.org/mozilla-central/rev/8f21d63b310a for https://treeherder.mozilla.org/logviewer.html#?job_id=1462107&repo=mozilla-central,

06:05:56 INFO - !insertmacro: macro "CallArtificialFunction" already being inserted!
06:05:56 INFO - Error in macro _WinVerAtLeast on macroline 2
06:05:56 INFO - Error in macro _If on macroline 9
06:05:56 INFO - Error in macro ITBL3SetProgressState_ on macroline 2
06:05:56 INFO - Error in macro CallArtificialFunction on macroline 11
06:05:56 INFO - Error in macro ITBL3SetProgressStateCall on macroline 2
06:05:56 INFO - Error in script "stub.nsi" on line 1300 -- aborting creation process
Status: RESOLVED → REOPENED
Flags: needinfo?(robert.strong.bugs)
Resolution: FIXED → ---
Target Milestone: Firefox 40 → ---
I built it locally and this must have ran into a problem when compiling with the old NSIS that I asked to have upgraded / deployed to the build slaves over a year ago in bug 989531
Flags: needinfo?(robert.strong.bugs)
Created attachment 8604443 [details] [diff] [review]
patch - compatible with nsis 2.46

This is in addition to the existing patch. It removes the windows version checks from the ITBL3SetProgressValue and ITBL3SetProgressState macros. These aren't needed since the ITBL3Create macro which doesn't use the CallArtificialFunction macro checks the Windows version and if it isn't compatible with IID_ITaskbarList3 it leaves the $ITaskbarList3 var set to 0.
Attachment #8604443 - Flags: review?(spohl.mozilla.bugs)
I also verified this works with NSIS 2.46 Unicode locally.
Attachment #8604443 - Flags: review?(spohl.mozilla.bugs) → review+
Missed the confvars change in the above try push... trying again
https://treeherder.mozilla.org/#/jobs?repo=try&revision=987b874dce19
Try looks good and the stub was built. I'll land later tonight
https://hg.mozilla.org/mozilla-central/rev/638f0db43a3f
https://hg.mozilla.org/mozilla-central/rev/932176dec8d1
Status: REOPENED → RESOLVED
Last Resolved: 3 years ago3 years ago
status-firefox41: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 41
status-firefox40: fixed → ---
You need to log in before you can comment on or make changes to this bug.