Closed
Bug 536848
Opened 15 years ago
Closed 15 years ago
Plugin finder service resorts to manual install for Flash
Categories
(addons.mozilla.org Graveyard :: Plugins, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: cilias, Assigned: morgamic)
References
Details
Attachments
(1 file)
2.74 KB,
patch
|
clouserw
:
review+
|
Details | Diff | Splinter Review |
Here's a screencast: <http://screencast.com/t/NWI0ZDQ4Z>. Using Build identifier: Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.2b5) Gecko/20091204 Firefox/3.6b5 and Windows 7, the plugin finder service does not find Flash and resorts to a manual install. Seems to work fine on 3.5.x. I haven't tested on trunk. I also tried installing 3.6b5 in C:\Program Files\Mozilla Firefox\ and go prompted for a manual install.
Assignee | ||
Comment 1•15 years ago
|
||
There's a regex for version since the installLocation support didn't start until 3.0+ and it's not recognizing 3.6. Will post a patch.
Assignee | ||
Updated•15 years ago
|
Component: Plugin Finder Service → Plugins
Product: Toolkit → addons.mozilla.org
QA Contact: plugin.finder → plugin-listings
Assignee | ||
Updated•15 years ago
|
Assignee: nobody → morgamic
Assignee | ||
Comment 2•15 years ago
|
||
Attachment #420389 -
Flags: review?(clouserw)
Updated•15 years ago
|
Attachment #420389 -
Flags: review?(clouserw) → review+
Comment 3•15 years ago
|
||
Comment on attachment 420389 [details] [diff] [review] v1, removes bad regex and obsolete fx2 tests sure. If that regex grows we should convert to something like 3\.[05]
Assignee | ||
Comment 4•15 years ago
|
||
Thanks Wil - thought about that, but I'd rather have no regex than a different one and fx2 support is moot. r59148
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 5•15 years ago
|
||
This went to production. If someone could verify that'd be awesome.
Reporter | ||
Updated•15 years ago
|
Status: RESOLVED → VERIFIED
Updated•8 years ago
|
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•