Stylish 1.1b1 breaks the extension manager which in turn breaks application update

RESOLVED FIXED in Firefox 4.0b6

Status

()

Firefox
Extension Compatibility
--
minor
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: Terrell Kelley, Assigned: christian)

Tracking

Trunk
Firefox 4.0b6
x86
Windows XP
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

6 years ago
User-Agent:       Mozilla/5.0 (Windows NT 5.1; rv:2.0b7) Gecko/20100101 Firefox/4.0b7
Build Identifier: Mozilla/5.0 (Windows NT 5.1; rv:2.0b7) Gecko/20100101 Firefox/4.0b7

While I was able to successfully upgrade to this version from Beta 6, I cannot get it to work for Beta 8. When ever I use the About Firefox dialog, I get the spinner, and it never determines whether there is a new version or not.

I did not see this on the list of known issues, and, for all I know, you fixed it Beta 8.

Reproducible: Always

Steps to Reproduce:
1. Click Help on either the Firefox button or the menu bar.
2. Click About Firefox.
Actual Results:  
Dialog box continually says "Checking for updates."

Expected Results:  
Dialog box should say "Update Found,"  "Click here to update," and/or similar.

I'm using a copy of a profile I've been using since Firefox 3.0.

And do remember that this worked properly back on Beta 6.

And please do not ignore because it's from Beta 7, as your feedback button seems to claim you will. It would be improper to say the bug was in Beta 8.

The "easy workaround is to install Beta 8 manually."

Updated

6 years ago
Component: General → Application Update
Product: Firefox → Toolkit
QA Contact: general → application.update
Version: unspecified → Trunk
Please set app.update.log (create it beforehand) in about:config to "true".
Try to Update again.
Then post your Error Console Output.
(Reporter)

Comment 2

6 years ago
Created attachment 500142 [details]
Requested error console output from Comment 1.
(Reporter)

Comment 3

6 years ago
It's hanging on the following:

Warning: WARN addons.manager: Exception calling callback: TypeError: aAddon.isCompatibleWith is not a function

Just in case, I put the rest of the output in an attachment. Sorry that I didn't realize that I could have included this message above.
Please post a lit of the extensions you have installed.
(Reporter)

Comment 5

6 years ago
I can do you one better. Once I saw the error, I went through all my extensions. I know exactly which one was causing the problem.

Stylish 1.1b1, available here: https://addons.mozilla.org/en-US/firefox/addon/2108/#install-beta

I'm not closing, just in case the bug is with Firefox, not the extension.
Thanks, moving over to add-on compatibility in the hope of getting the extension fixed. 

Note: there is bug 595078 to try to prevent extensions from doing this on the app update side and bug 595081 should also help.
Component: Application Update → Extension Compatibility
Product: Toolkit → Firefox
QA Contact: application.update → extension.compatibility
Target Milestone: --- → Firefox 4.0b6
Summary: Automatic updates not working → Stylish 1.1b1 breaks the extension manager which in turn breaks application update
(Reporter)

Comment 7

6 years ago
I do want to add that GreaseMonkey 2010.12.24.nightly also modifies the extension manager, but does not break updates.
(Reporter)

Comment 8

6 years ago
Oops, I forgot the URL:

https://arantius.com/misc/gm-nightly/
(Assignee)

Comment 9

6 years ago
This seems like a good candidate for blocklisting, yes? Just for FF4 that is.
(Reporter)

Comment 10

6 years ago
I don't know. The extension manager works, other than breaking application updates. And it only breaks them when an update exists. The user can easily install the update manually. (Something like the fix for Bug 595455 would be nice, though. If the update manager has hung for a certain amount of time, display a failure notice and add an option to download the update manually.)

Perhaps a block only on older versions of the 4.0beta would make sense. Is that possible?

Also, is anyone going to confirm?
(Reporter)

Comment 11

6 years ago
Okay, this is fixed in Stylish 1.1b3 on my end. I vote close RESOLVED, but I'll let you guys decide. 

If you want, blacklist anyone from downloading 1.1b1.
I verified that with Stylish 1.1b1 still breaks the add-ons manager even with the code that somewhat locks down the add-ons manager. I think we should blocklist Stylish 1.1b1.

btw: the patch in bug 595078 fixes this for me.
Status: UNCONFIRMED → NEW
Ever confirmed: true
(Assignee)

Comment 13

6 years ago
Created attachment 503997 [details]
Add to the blocklist DB, which will then get slurped into the tree
Attachment #503997 - Flags: review?(morgamic)
(Assignee)

Updated

6 years ago
Assignee: nobody → clegnitto
Comment on attachment 503997 [details]
Add to the blocklist DB, which will then get slurped into the tree

Looks good to me.
Attachment #503997 - Flags: review?(morgamic) → review+
(Assignee)

Updated

6 years ago
Depends on: 627406
(Assignee)

Comment 15

6 years ago
Filed bug 627406 for IT to update the production DB.
(Assignee)

Comment 16

6 years ago
This has been run on production and should now be fixed.
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.