Closed
Bug 535594
Opened 15 years ago
Closed 13 years ago
Fix/Clean-up 'security.enable_java' preference references
Categories
(Core Graveyard :: Plug-ins, defect)
Core Graveyard
Plug-ins
Tracking
(Not tracked)
VERIFIED
FIXED
mozilla14
People
(Reporter: sgautherie, Assigned: pranavrc)
References
Details
Attachments
(1 file)
2.16 KB,
patch
|
benjamin
:
review+
|
Details | Diff | Splinter Review |
http://mxr.mozilla.org/mozilla-central/search?string=security.enable_java&case=on
"Found 2 matching lines in 2 files"
It looks like further cleanup should be done.
http://mxr.mozilla.org/mozilla1.9.2/search?string=security.enable_java&case=on
"Found 6 matching lines in 4 files"
Not sure if the pref should be restored or the code cleaned up.
Reporter | ||
Comment 1•13 years ago
|
||
(In reply to comment #0)
> http://mxr.mozilla.org/mozilla-central/search?string=security.
> enable_java&case=on
> "Found 2 matching lines in 2 files"
{
/modules/libpref/src/init/all.js
* line 1853 -- // Note: Does not scan if security.enable_java is not true
/js/src/tests/user.js
* line 18 -- user_pref("security.enable_java", false);
}
Whiteboard: [good first bug]
Assignee | ||
Comment 2•13 years ago
|
||
Comment on attachment 584045 [details] [diff] [review]
Cleaned up security.enable_java references
Forgot to request review?
Attachment #584045 -
Flags: review?(benjamin)
Updated•13 years ago
|
Attachment #584045 -
Flags: review?(benjamin) → review+
Keywords: checkin-needed
Whiteboard: [good first bug]
Comment 4•13 years ago
|
||
Assignee: nobody → prp.1111
Flags: in-testsuite-
Keywords: checkin-needed
Target Milestone: --- → mozilla14
Comment 5•13 years ago
|
||
Status: NEW → RESOLVED
Closed: 13 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
•