Closed Bug 1286332 Opened 4 years ago Closed 4 years ago

Port Bug 1284517 to TB [The installer needs to update the lastWriteTime of the Firefox startmenu shortcut]

Categories

(Thunderbird :: Installer, defect)

All
Windows 8
defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 50.0

People

(Reporter: Paenglab, Assigned: Paenglab)

References

Details

Attachments

(1 file)

The start menu icon on Windows 8 and up needs to update the lastWriteTime clear the icon cache and then show the new icon we introduced in bug 1284080.
Attached patch iconUpdate.patchSplinter Review
Jörg, I know you're not familiar with this files, me neither. But this is a straight port from FX to TB. Most of this files are similar to the FX files except the stub installer parts we're not use.
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #8770246 - Flags: review?(mozilla)
Comment on attachment 8770246 [details] [diff] [review]
iconUpdate.patch

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

Looks like a straight port to me.
rs=jorgk, if you don't mind, please put "rs" (rubber-stamp) and "jorgk" (all lower). Thanks.
Attachment #8770246 - Flags: review?(mozilla) → review+
Changed the commit message.

https://hg.mozilla.org/comm-central/rev/389b5b23a5f0
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 50.0
Great, thanks for the push, I was waiting for one to see whether the Xpcshell failures are gone now that bug 1285735 is fixed.
You need to log in before you can comment on or make changes to this bug.