Closed
Bug 78116
Opened 23 years ago
Closed 22 years ago
nsIFileStream.cpp is deprecated. cookies should be using necko file io.
Categories
(Core :: Networking: Cookies, enhancement)
Core
Networking: Cookies
Tracking
()
VERIFIED
FIXED
Future
People
(Reporter: timeless, Assigned: dwitte)
References
()
Details
(Keywords: arch, helpwanted)
SunOS 5.7 Generic_106541-15 sun4u sparc SUNW,Ultra-2
xlib cvs build from this morning.
http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/extensions/cookie/nsPermiss
ions.cpp&rev=1.7&mark=432-434#320
It's nice to know that something failed, and that it's not an error, but could
we please tell the developer where the file lives? userChrome.css and
userContent.css do this. It would also be nice if for debugging purposes we
found out _why_ the open failed (most likely case is file not found, but others
do exist -- I presume)
Comment 1•23 years ago
|
||
nsIFileStream.cpp is deprecated. cookies should be using necko file io.
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → WONTFIX
ok, not a networking bug, sorry. i don't see any cookie bugs for this so i'm
morphing it.
Status: RESOLVED → REOPENED
Component: Networking: File → Cookies
Resolution: WONTFIX → ---
poof
Assignee: dougt → morse
Status: REOPENED → NEW
Summary: Opening file cookperm.txt failed → nsIFileStream.cpp is deprecated. cookies should be using necko file io.
Updated•23 years ago
|
Status: NEW → ASSIGNED
Target Milestone: --- → Future
Keywords: arch,
helpwanted
Assignee | ||
Updated•22 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Updated•22 years ago
|
Assignee | ||
Comment 5•22 years ago
|
||
fixed per dougt's recent minimo checkins
and also, simultaneously, by the rewrite ;)
Status: ASSIGNED → RESOLVED
Closed: 23 years ago → 22 years ago
Resolution: --- → FIXED
VERIFIED:
http://lxr.mozilla.org/mozilla1.0/ident?i=nsInputFileStream
shows several uses of nsInputFileStream in cookie code.
# extensions/cookie/nsCookies.cpp:
* View change log or Blame annotations line 1741
# extensions/cookie/nsPermissions.cpp:
* View change log or Blame annotations line 457
# extensions/cookie/nsUtils.cpp:
* View change log or Blame annotations line 54
* line 82
# extensions/cookie/nsUtils.h:
* View change log or Blame annotations line 45
http://lxr.mozilla.org/mozilla/ident?i=nsInputFileStream
shows no current uses of of nsInputFileStream in cookie code.
Status: RESOLVED → VERIFIED
OS: Solaris → All
QA Contact: tever → cookieqa
Hardware: Sun → All
resize image thank for post
You need to log in
before you can comment on or make changes to this bug.
Description
•