Closed
Bug 567184
Opened 15 years ago
Closed 15 years ago
A bootstrap add-on with a target app version less than the current app version is enabled when installed
Categories
(Toolkit :: Add-ons Manager, defect)
Toolkit
Add-ons Manager
Tracking
()
VERIFIED
FIXED
mozilla1.9.3a5
Tracking | Status | |
---|---|---|
blocking2.0 | --- | beta1+ |
People
(Reporter: robert.strong.bugs, Assigned: robert.strong.bugs)
References
Details
(Whiteboard: [AddonsRewrite])
Attachments
(1 file)
4.25 KB,
patch
|
mossop
:
review+
|
Details | Diff | Splinter Review |
Found while writing the tests for bug 546595.
I was able to install a bootstrap add-on and it was enabled with an install.rdf as follows
<?xml version="1.0"?>
<RDF xmlns="http://www.w3.org/1999/02/22-rdf-syntax-ns#"
xmlns:em="http://www.mozilla.org/2004/em-rdf#">
<Description about="urn:mozilla:install-manifest">
<em:id>test1@updatetest.mozilla.org</em:id>
<em:version>2.0</em:version>
<em:bootstrap>true</em:bootstrap>
<em:name>test1</em:name>
<em:description>Test Description</em:description>
<em:targetApplication>
<Description>
<em:id>{ec8030f7-c20a-464f-9b0e-13a3a9e97384}</em:id>
<em:minVersion>undefined</em:minVersion>
<em:maxVersion>undefined</em:maxVersion>
</Description>
</em:targetApplication>
</Description>
</RDF>
The bootstrap.js just stubbed the required functions
There was nothing in the error console
Updated•15 years ago
|
blocking2.0: --- → beta1+
Assignee | ||
Comment 1•15 years ago
|
||
I think bug 567175 might fix this
Assignee | ||
Comment 2•15 years ago
|
||
Simple fix with a simple test
Assignee: nobody → robert.bugzilla
Status: NEW → ASSIGNED
Attachment #447944 -
Flags: review?(dtownsend)
Assignee | ||
Updated•15 years ago
|
Summary: A bootstrap add-on with invalid target app minVersion / maxVersion is enabled when installed → A bootstrap add-on with a target app version less than the current app version is enabled when installed
Updated•15 years ago
|
Flags: in-testsuite?
Flags: in-litmus?
Updated•15 years ago
|
Whiteboard: [rewrite] → [AddonsRewrite]
Assignee | ||
Comment 3•15 years ago
|
||
No need for a litmus test for this specific issue... the xpcshell test suffices for this specific bug.
Flags: in-litmus? → in-litmus-
Comment 4•15 years ago
|
||
Comment on attachment 447944 [details] [diff] [review]
patch with test
Good catch, thanks
Attachment #447944 -
Flags: review?(dtownsend) → review+
Assignee | ||
Comment 5•15 years ago
|
||
Pushed to mozilla-central
http://hg.mozilla.org/mozilla-central/rev/7e51fa360a01
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Flags: in-testsuite? → in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.3a5
Comment 6•14 years ago
|
||
Verified fixed with Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.3a5pre) Gecko/20100530 Minefield/3.7a5pre ID:20100530040319
The one remaining issue I have filed as bug 569149.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•