Closed Bug 476409 Opened 16 years ago Closed 16 years ago

clean up nsPluginHostImpl

Categories

(Core Graveyard :: Plug-ins, defect)

x86
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: jaas, Assigned: jaas)

Details

Attachments

(1 file)

There is a lot of cleanup that we could do in nsPluginHostImpl.
Attached patch patch v1.0Splinter Review
This does at least the following: - removes an old workaround for mac flash 6 - stop using the phrase "active plugin" to reference plugin instances, that is confusing because an "active plugin" sounds like a plugin that is not blocklisted, loaded into memory, or something else like that - remove some dead code - fixes some spelling
Attachment #360039 - Flags: superreview?(jst)
Comment on attachment 360039 [details] [diff] [review] patch v1.0 r+sr=jst
Attachment #360039 - Flags: superreview?(jst)
Attachment #360039 - Flags: superreview+
Attachment #360039 - Flags: review+
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: