Closed Bug 521673 Opened 10 years ago Closed 3 years ago

Remove remnant OJI references

Categories

(Core :: Plug-ins, defect, trivial)

defect
Not set
trivial

Tracking

()

RESOLVED INCOMPLETE
mozilla1.9.3a3

People

(Reporter: sgautherie, Assigned: sgautherie)

References

()

Details

(Keywords: helpwanted)

Attachments

(5 files, 1 obsolete file)

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: 10 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-
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.
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: 10 years ago3 years ago
Resolution: --- → INCOMPLETE
You need to log in before you can comment on or make changes to this bug.