Closed
Bug 1183372
Opened 9 years ago
Closed 9 years ago
add tests for plugin.allowed_types pref
Categories
(Core Graveyard :: Plug-ins, defect)
Core Graveyard
Plug-ins
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: myk, Assigned: myk)
Details
Attachments
(1 file)
|
(deleted),
patch
|
jaas
:
review+
|
Details | Diff | Splinter Review |
Here are tests for the plugin.allowed_types pref, which didn't land with tests in bug 755551.
Attachment #8633133 -
Flags: review?(jaas)
|
Assignee | |
Comment 1•9 years ago
|
||
(The patch also makes the IsTypeInList comparison be case-insensitive, since MIME types are case-insensitive. I noticed this while writing tests with the second test plugin, whose type is sometimes referenced in our codebase as the mixed-case "application/x-Second-Test".)
Attachment #8633133 -
Flags: review?(jaas) → review+
|
|
Assignee | |
Comment 2•9 years ago
|
||
|
|
Assignee | |
Comment 3•9 years ago
|
||
|
|
Assignee | |
Comment 4•9 years ago
|
||
|
|
Assignee | |
Comment 5•9 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
|
||
Updated•3 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•