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)

Product:
Core
Component:
CSS Parsing and Computation
Platform:
x86
Windows NT
Type:
defect

Tracking

()

Status:
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).
peterl no longer works at netscape. assigning to waterson for triage.
this one sounds like it's for you, tom.
QA Contact: cbegle → tever
is this your's chris?
Assignee: peterl → waterson
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
Taking over 1/3 of Pierre's NEW bugs to help reduce his doomage factor
Assignee: pierre → attinasi
Reassigned back to me these bugs that shouldn't have left my list.
Assignee: attinasi → pierre
the first call is off the hook (it loads a file from disk only). The one in 
loadagentsheet is still a problem.
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
development issues, closing
Status: RESOLVED → CLOSED
*** 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.