Closed Bug 521673 Opened 15 years ago Closed 8 years ago

Remove remnant OJI references

Categories

(Core Graveyard :: Plug-ins, defect)

Product:
Core Graveyard
Component:
Plug-ins
Type:
defect
Priority:
Not set
Severity:
trivial

Tracking

(Not tracked)

Status:
RESOLVED INCOMPLETE
Milestone:
mozilla1.9.3a3

People

(Reporter: sgautherie, Assigned: sgautherie)

References

(
URL
)

Details

(Keywords: helpwanted)

Attachments

(5 files, 1 obsolete file)

(Av1) /js/src/configure.in [Checkin: Comment 2]
(deleted), patch
ted
: review+
Details | Diff | Splinter Review
(Bv1) /toolkit/mozapps/installer/makecfgini.pl [Checkin: Comment 4]
(deleted), patch
benjamin
: review+
Details | Diff | Splinter Review
(Cv2) Delete /xpcom/reflect/xptcall/tests/eVC4/* [Checkin: Comment 9]
(deleted), patch
dougt
: review+
Details | Diff | Splinter Review
(Dv1) /dom/dom-config.mk: remove obsolete REQUIRES [Checkin: Comment 11]
(deleted), patch
jst
: review+
Details | Diff | Splinter Review
Remove unnecessary preferences for OJI [Checked in: Comment 16]
(deleted), patch
jaas
: review+
Details | Diff | Splinter Review
No description provided.
Flags: in-testsuite-
Blocks: 535594
It looks like this was missed in bug 485984.
Attachment #427915 - Flags: review?(ted.mielczarek)
Blocks: 485984
Attachment #427915 - Flags: review?(ted.mielczarek) → review+
Comment on attachment 427915 [details] [diff] [review] (Av1) /js/src/configure.in [Checkin: Comment 2] http://hg.mozilla.org/mozilla-central/rev/f792552e1a14
Attachment #427915 - Attachment description: (Av1) /js/src/configure.in → (Av1) /js/src/configure.in [Checkin: Comment 2]
Assignee: nobody → sgautherie.bz
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.3a3
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Status: REOPENED → ASSIGNED
I don't know this code: please double-check.
Attachment #429997 - Flags: review?(benjamin)
Attachment #429997 - Flags: review?(benjamin) → review+
Comment on attachment 429997 [details] [diff] [review] (Bv1) /toolkit/mozapps/installer/makecfgini.pl [Checkin: Comment 4] http://hg.mozilla.org/mozilla-central/rev/029aebe715a9
Attachment #429997 - Attachment description: (Bv1) /toolkit/mozapps/installer/makecfgini.pl → (Bv1) /toolkit/mozapps/installer/makecfgini.pl [Checkin: Comment 4]
Attachment #430372 - Flags: review?(ted.mielczarek)
Comment on attachment 430372 [details] [diff] [review] (Cv1) /xpcom/reflect/xptcall/tests/eVC4/XPTCInvoke_Testing.vc* dougt says that whole eVC4 directory is no longer useful. I'd take a patch to remove it. No point in poking at it otherwise.
Attachment #430372 - Flags: review?(ted.mielczarek) → review-
Per comment 6.
Attachment #430372 - Attachment is obsolete: true
Attachment #430830 - Flags: review?
Attachment #430830 - Flags: review? → review?(dougt)
Comment on attachment 430830 [details] [diff] [review] (Cv2) Delete /xpcom/reflect/xptcall/tests/eVC4/* [Checkin: Comment 9] rs
Attachment #430830 - Flags: review?(dougt) → review+
Comment on attachment 430830 [details] [diff] [review] (Cv2) Delete /xpcom/reflect/xptcall/tests/eVC4/* [Checkin: Comment 9] http://hg.mozilla.org/mozilla-central/rev/48c9e26f70b4
Attachment #430830 - Attachment description: (Cv2) Delete /xpcom/reflect/xptcall/tests/eVC4/* → (Cv2) Delete /xpcom/reflect/xptcall/tests/eVC4/* [Checkin: Comment 9]
Attachment #430936 - Flags: review? → review?(jst)
Attachment #430936 - Flags: review?(jst) → review+
Attachment #430936 - Attachment description: (Dv1) /dom/dom-config.mk: remove obsolete REQUIRES. → (Dv1) /dom/dom-config.mk: remove obsolete REQUIRES [Checkin: Comment 11]
Comment on attachment 430936 [details] [diff] [review] (Dv1) /dom/dom-config.mk: remove obsolete REQUIRES [Checkin: Comment 11] http://hg.mozilla.org/mozilla-central/rev/93594759cfb8
Blocks: 398573
Josh, could you check/update the remaining "10 matching lines in 5 files" comments (and related code)? Thanks.
URL: http://mxr.mozilla.org/mozilla-centra... → http://mxr.mozilla.org/mozilla-centra...
Keywords: helpwanted
Status update: "Found 9 matching lines in 4 files".
Also, we need remove OJI preference from pref.js. - java.global_java_version_file - java.private_java_version_file - java.default_java_location_solaris - java.default_java_location_others - java.java_plugin_library_name
Attachment #572763 - Flags: review?(joshmoz) → review+
Attachment #572763 - Attachment description: Remove unnecessary preferences for OJI → Remove unnecessary preferences for OJI [Checked in: Comment 16]
Resolving old bugs which are likely not relevant any more, since NPAPI plugins are deprecated.
Status: ASSIGNED → RESOLVED
Closed: 15 years ago → 8 years ago
Resolution: --- → INCOMPLETE
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: