Closed Bug 488042 Opened 16 years ago Closed 15 years ago

disable XPCOM plugin loading

Categories

(Core Graveyard :: Plug-ins, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: jaas, Assigned: jaas)

References

Details

Attachments

(2 files, 3 obsolete files)

We need to remove XPCOM plugin support piece-by-piece and first we need to disable XPCOM plugin loading so we can start removing parts.
Attached patch fix v1.0 (obsolete) (deleted) — Splinter Review
Comment on attachment 372309 [details] [diff] [review] fix v1.0 - In nsPluginDirServiceProvider::GetFile(): + nsresult rv; This stray nsresult rv declaration (which you didn't add, just got moved) shadows the one that's returned from this method, and thus leaves the one being removed set incorrectly. This is causing the Java plugin loading problem on windows that you mentioned to me. This didn't use to be a problem because of the locality of the stray rv declaration prior to it being moved...
Attached patch fix v1.1 (obsolete) (deleted) — Splinter Review
Attachment #372309 - Attachment is obsolete: true
Attachment #372546 - Flags: superreview?(jst)
Attachment #372546 - Flags: superreview?(jst)
Attachment #372546 - Flags: superreview+
Attachment #372546 - Flags: review+
Comment on attachment 372546 [details] [diff] [review] fix v1.1 r+sr=jst
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Depends on: 491834
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Attached file crash trace from Windows tbox (deleted) —
Attached patch fix v1.2 (obsolete) (deleted) — Splinter Review
Don't load the OJI-based Java plugin on Windows. We should also blocklist it for > Gecko 1.9.1.
Attachment #372546 - Attachment is obsolete: true
Attachment #377703 - Flags: superreview?(jst)
Attachment #377703 - Flags: superreview?(jst)
Attachment #377703 - Flags: superreview+
Attachment #377703 - Flags: review+
Attached patch fix v1.2 (updated to tip) (deleted) — Splinter Review
Attachment #377703 - Attachment is obsolete: true
Status: REOPENED → RESOLVED
Closed: 15 years ago15 years ago
Resolution: --- → FIXED
backed out, strange unit test failure that I don't think is this but being safe (tests passed on the try server), I don't have time to watch the tree any more tonight http://hg.mozilla.org/mozilla-central/rev/e720085bf9ba
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Pushed to mozilla-central after Windows tboxes upgraded to Java SE 6 Update 14. http://hg.mozilla.org/mozilla-central/rev/b15b92cb7b91
Status: REOPENED → RESOLVED
Closed: 15 years ago15 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: