Closed
Bug 26297
Opened 25 years ago
Closed 25 years ago
Use of NS_OpenURI(nsIInputStream*,...) creates blocking stream.
Categories
(Core :: CSS Parsing and Computation, defect, P3)
Tracking
()
CLOSED
WONTFIX
People
(Reporter: jud, Assigned: pierre)
Details
http://lxr.mozilla.org/seamonkey/source/layout/html/style/src/nsCSSLoader.cpp#10
77 shows a line using the netUtil function that wraps an OpenInputStream() call.
The result is that the user is handed an input stream to read data from. The
inputStream->Read() call is a blocking call and can/will block the thread it's
called on. If this steam is guaranteed to be read from a thread other than the
UI thread, evaluate whether or not the blocking behavior is ok, and if so, close
this bug. Otherwise, your code need to do an asynchronous load of the URI. If
you don't want to implement nsIStreamListener, please look at
nsIStreamLoader.idl which will call a callback function you implement when all
the data has arrived (it does all the asyncronous work for you).
Reporter | ||
Comment 1•25 years ago
|
||
Comment 2•25 years ago
|
||
peterl no longer works at netscape. assigning to waterson for triage.
Comment 5•25 years ago
|
||
Potentially a problem in the way the CSS loader works. pierre, could you take a
look at jud's commentary and determine whether or not this is a bug? It looks
like some style sheet loading runs the risk of deadlocking...
Assignee: waterson → pierre
Component: Browser-General → Style System
Comment 6•25 years ago
|
||
Taking over 1/3 of Pierre's NEW bugs to help reduce his doomage factor
Assignee: pierre → attinasi
Assignee | ||
Comment 7•25 years ago
|
||
Reassigned back to me these bugs that shouldn't have left my list.
Assignee: attinasi → pierre
Reporter | ||
Comment 8•25 years ago
|
||
the first call is off the hook (it loads a file from disk only). The one in
loadagentsheet is still a problem.
Reporter | ||
Comment 9•25 years ago
|
||
vidur just confirmed that the second call is a local load only also. I'm marking
this wontfix as the current code is *safe* and changing it would be a headache.
Status: NEW → RESOLVED
Closed: 25 years ago
Resolution: --- → WONTFIX
Comment 11•24 years ago
|
||
*** Bug 44461 has been marked as a duplicate of this bug. ***
You need to log in
before you can comment on or make changes to this bug.
Description
•