299047 - About page can be brought up for disabled extensions

Status: VERIFIED INVALID

(Toolkit :: Add-ons Manager, defect)

Description

[Inactive]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8b2) Gecko/20050628 Firefox/1.0+
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8b2) Gecko/20050628 Firefox/1.0+

Basic functionality QA test indicates that the "About [extension]" dialog should
not work for disabled extensions. 


Reproducible: Always

Steps to Reproduce:
1.Disable an extension
2.Restart Firefox
3.Open the about dialog on the disabled extension

Actual Results:  
About dialog opens

Expected Results:  
About dialog option in context menu should be disabled if the extension is disabled.

Comment 1

[Robert Strong (they/them - no direct email)]

I believe this should be invalid... there is no harm in displaying the about
dialog info of a disabled extension due to a recent patch. Previously, if an
extension was disabled and about was displayed it could cause a hang if the
about dialog was specified by the extension vs. using the generic about dialog.
Now, if the extension is disabled the generic about dialog is displayed, etc.

Comment 2

[Benjamin Smedberg]

Who did these "basic functionality QA tests"? There are important reasons that
users might want to see details about the extension even when it is disabled
(most importantly minversion/maxversion and such).

Comment 3

[Inactive]

(In reply to comment #2)
> Who did these "basic functionality QA tests"? There are important reasons that
> users might want to see details about the extension even when it is disabled
> (most importantly minversion/maxversion and such).

It's on Testrunner...
Here's the public example: http://testrunner.mozilla.org/test.cgi?run_id=1319#898

Comment 4

[Robert Strong (they/them - no direct email)]

I don't believe there is a product for filing bugs on testrunner but that is IMO
where this needs to be fixed due to the reasons previously mentioned.

Comment 5

[Peter van der Woude [:Peter6]]

this is so unwanted for Firefox
->INVA

Comment 6

[Tracy Walker [:tracy]]

verified invalid.  I'll make note to update the testcase.