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)

enhancement
Not set
normal

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)
Severity: normal → enhancement
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.
Status: NEW → ASSIGNED
Target Milestone: --- → Future
Blocks: 79119
Keywords: arch, helpwanted
taking
Assignee: morse → dwitte
Status: ASSIGNED → NEW
Status: NEW → ASSIGNED
Blocks: 187304
No longer depends on: 187304
fixed per dougt's recent minimo checkins and also, simultaneously, by the rewrite ;)
Status: ASSIGNED → RESOLVED
Closed: 23 years ago22 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.