758291 - Remove PDF Viewer addon on update

Status: VERIFIED FIXED

(Firefox :: PDF Viewer, defect)

Description

[JK]

In Bug 740795, PDF Viewer was changed to not be shipped as an add-on. However, the update process doesn't remove the addon from the \extensions folder.

Comment 1

[JK]

Reproduction:
Install an old Nightly build, which has PDF Viewer as an add-on.
Update it to latest version (in which it is no longer an add-on).

Result:
PDF Viewer is still listed as an add-on.

Expected result:
PDF Viewer should not be listed as an add-on.

Comment 2

[:Gavin Sharp [email: [e-mail]]]

We need to add it to removed-files.in, I guess?

Comment 3

[Yury Delendik (:yury)]

To replicate:

1. Install Nightly from http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2012-05-20-03-05-30-mozilla-central/
2. Open "About Nightly" to update the browser

Comment 4

[Yury Delendik (:yury)]

Attachment: Add [e-mail] to removed-files

Looking at the history (blame/bugs) of the removed-files.in, adding the entry in this file shall solve the issue.

Comment 5

[Yury Delendik (:yury)]

Attachment: patch for checkin

Comment 6

[:Gavin Sharp [email: [e-mail]]]

AIUI, this is only applicable to channels that have previously shipped the add-on version, which doesn't include Aurora. So this isn't actually a concern for 15, and we don't actually need to backport the patch.

Comment 7

[Jared Wein [:jaws] (please needinfo? me)]

(In reply to :Gavin Sharp (use [e-mail] for email) from comment #6)
> AIUI, this is only applicable to channels that have previously shipped the
> add-on version, which doesn't include Aurora. So this isn't actually a
> concern for 15, and we don't actually need to backport the patch.

So this patch should only affect Nightly users? Hence meaning that we could land it on 16 and be done with it? If that's the case, how necessary is it to fix this bug? When this change propagates through to the other release channels, it will just add extra code paths that aren't needed.

Comment 8

[Dave Townsend [:mossop]]

(In reply to Jared Wein [:jaws] from comment #7)
> (In reply to :Gavin Sharp (use [e-mail] for email) from comment
> #6)
> > AIUI, this is only applicable to channels that have previously shipped the
> > add-on version, which doesn't include Aurora. So this isn't actually a
> > concern for 15, and we don't actually need to backport the patch.
> 
> So this patch should only affect Nightly users? Hence meaning that we could
> land it on 16 and be done with it? If that's the case, how necessary is it
> to fix this bug? When this change propagates through to the other release
> channels, it will just add extra code paths that aren't needed.

It would also affect users who switched from Nightly to other channels, but really the patch is so simple it isn't worth thinking about. We should just take it and move along.

Comment 9

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/7985ee3a7734

Comment 10

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/7985ee3a7734

Comment 11

[:Gavin Sharp [email: [e-mail]]]

Comment on attachment 630145 [details] [diff] [review]
patch for checkin

[Triage Comment]
a=me

Comment 12

[:Gavin Sharp [email: [e-mail]]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/399d0a248065

Comment 13

[Paul Silaghi, QA [:pauly]]

(In reply to JK from comment #1)
> Reproduction:
> Install an old Nightly build, which has PDF Viewer as an add-on.
> Update it to latest version (in which it is no longer an add-on).
> 
> Result:
> PDF Viewer is still listed as an add-on.
> 
> Expected result:
> PDF Viewer should not be listed as an add-on.

So in order to test this on FF 15, I installed FF 14b12, then the PDF Viewer addon and then updated to FF 15b3.
PDF Viewer is still listed as an add-on.
Any thoughts?

Comment 14

[Yury Delendik (:yury)]

> So in order to test this on FF 15, I installed FF 14b12, then the PDF Viewer
> addon and then updated to FF 15b3.
> PDF Viewer is still listed as an add-on.
> Any thoughts?

Don't install PDF Viewer addon -- it shall be a part of the FF (also see comment 3)

Comment 15

[Paul Silaghi, QA [:pauly]]

So:
1. Installed Nightly 2012-05-20 (comment 3) --> PDF Viewer listed as an add-on
2. Pave over installation of FF 15b3 (comment 8) --> PDF Viewer not in add-ons list

Verified fixed on FF 15b3 on Win 7.