982655 - removing all fxmetro registry entries after updating using fxdesktop

Status: VERIFIED DUPLICATE of bug 981166

(Firefox :: General, defect)

Description

[Kamil Jozwiak [:kjozwiak]]

Updating from fxdesktop still leaves behind the following fxmetro registry:
- HKEY_CURRENT_USER\Software\Mozilla\Firefox\MetroLastAHE

We should be removing all the registry entries that are associated with fxmetro. This is lower priority as it doesn't prevent fxdesktop from working.

Comment 1

[u279076]

Jim, can you please comment as to how this impacts shipping Firefox 28?

Comment 2

[Jim Mathies [:jimm]]

(In reply to Anthony Hughes, QA Mentor (:ashughes) from comment #1)
> Jim, can you please comment as to how this impacts shipping Firefox 28?

No impact. in fact I'm pretty sure that last key will go away on the next update. I will confirm that on oak and close this if that's the case. Regardless it won't cause any problems.

Comment 3

[Lukas Blakk [:lsblakk] use ?needinfo]

Thanks Jim, not tracking in that case.

Comment 4

[u279076]

I'm going to assume this is wontfix for Firefox 28 at this point. Please correct me if I am wrong.

Comment 5

[Brian R. Bondy [:bbondy]]

I think this was fixed by my push in bug 981166.
I noticed a typo in the reg key removal, it was trying to remove before.
MetroLastAEH instead of MetroLastAHE. I fixed that in the push in bug 981166.

Comment 6

[Kamil Jozwiak [:kjozwiak]]

Went through bug #981166 with the oak builds and the latest nightly build and can confirm that the registry entries are being removed. Checked the following locations:

- HKEY_CURRENT_USER\Software\Mozilla (anything relating to metrofx including MetroLastAHE from comment #0)
- HKEY_LOCAL_MACHINE\SOFTWARE\Classes\EEFEA8717BC47F65\
- HKEY_USERS\S-1-5-21-1791338240-857805526-2257772433-1002\Software\Classes\EEFEA8717BC47F65\

Marking as verified and removing the "verifyme" flag

Used the following build:
- http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2014-03-24-03-02-03-mozilla-central/ (once installed, updated via metrofx to pull the latest nightly)