Description of "Mozilla Default Plug-in" should not include suffixes

VERIFIED WONTFIX

Status

()

Core
Plug-ins
--
minor
VERIFIED WONTFIX
17 years ago
16 years ago

People

(Reporter: Henrik Gemal, Assigned: av (gone))

Tracking

Trunk
x86
Windows 2000
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

17 years ago
The description on the npnul32.dll file is:
"Mozilla Default Plug-in (*.*)"
it should be:
"Mozilla Default Plug-in"
since the suffixes can be read by looking at the Suffixes attribute.
(Reporter)

Comment 1

17 years ago
Created attachment 57360 [details] [diff] [review]
patch to fix reduntant info
(Reporter)

Updated

17 years ago
Keywords: patch, review
(Assignee)

Comment 2

17 years ago
Comment on attachment 57360 [details] [diff] [review]
patch to fix reduntant info

r=av
Attachment #57360 - Flags: review+
(Reporter)

Comment 3

17 years ago
who should be giving sr here? Will you cc people?

Comment 4

17 years ago
I think Patrick Beard recently got ordained a super reviewer.

btw, this plugin is going away in bug 83754 but some of our embedding vendors
have choosen to stick with this C++ version.

Updated

17 years ago
Summary: Desecription of "Mozilla Default Plug-in" should not include suffix → Description of "Mozilla Default Plug-in" should not include suffixes
(Reporter)

Comment 5

17 years ago
should this get checked in or wontfix?

Updated

16 years ago
Status: NEW → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → WONTFIX

Comment 6

16 years ago
The plug-ins triage team (av, beppe, peterl, serge and shrir) have reviewed this
issue and have made the following determination:

This will be resolved with bug 106413. In the current state, the *.* is
hardcoded in the plug-in.

Note, that the current default plug-in will not be fixed, the resolution for bug
106413 is to replace the current default plug-in and information can be altered
at that time. Consequently, we are marking this bug as won't fix in lieu of bug
106413

Comment 7

16 years ago
marking verif, pls reopen stating a reason, Thx!
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.