81214 - ###!!! ASSERTION: HTTP cannot send practical requests without this much: 'mAppName && mAppVersion && mPlatform && mSecurity && mOscpu', file c:\Projects\mozilla\netwerk\protocol\http\src\nsHttpHandler.cpp, line 703

Status: RESOLVED FIXED

(Core :: Networking, defect)

Description

[Vikas Bhargava]

Build : CVS Debug build from today morning (20010516)

1. In Edit->Preferences->Advanced->Proxies, enter a auto proxy URL.
2. Click Ok and close the browser.
3. Re-start the browser.

Expected Results: 
Browser starts up sucessfully using the pac url for all proxy information.

Actual Results:
Browser shows the following on the console:

Type Manifest File: C:\Projects\mozilla\dist\WIN32_D.OBJ\bin\components\xpti.dat
nsNativeComponentLoader: autoregistering begins.
nsNativeComponentLoader: autoregistering succeeded
nNCL: registering deferred (0)
###!!! ASSERTION: HTTP cannot send practical requests without this much: 
'mAppName && mAppVersion && mPlatform && mSecurity && mOscpu', file 
c:\Projects\mozilla\netwerk\protocol\http\src\nsHttpHandler.cpp, line 703
###!!! Break: at file 
c:\Projects\mozilla\netwerk\protocol\http\src\nsHttpHandler.cpp, line 703

Comment 1

[Vikas Bhargava]

On further debugging, the cause of the assertion is as follows:

On startup, mozilla needs to load 
"chrome://navigator/locale/navigator.properties", but it is using the 
http handler to load it instead, which fails as it cannot create the HTTP header 
as not all members of the protocol handler have not been assigned. 

This happens when it tries to create a channel for the chrome URI in 
nsIOService::NewChannelFromURI (nsIOService.cpp). Since an autoproxy has been 
defined the if statement on line 310 succeeds and it tries to use the http 
handler to load the URI.

The code was added as a part of the http landing of darin on 5/11. Reassigning 
to him.

Comment 2

[Darin Fisher]

dougt: can you help with this one?  bounce back to me if not.

Comment 3

[Doug Turner (:dougt)]

nice try, but currently as doomed as you... 

Comment 4

[chris hofmann]

if there is a bad user symtom lets keep this on the 0.9.1 radar.
if not lets move it to 0.9.2

Comment 5

[benc]

mass move, v2.
qa to me.

Comment 6

[Darin Fisher]

-> gagan

Comment 7

[Gagan]

Attachment: patch that fixes this and a bunch of other PAC related bugs :)

Comment 8

[Darin Fisher]

sr=darin

Comment 9

[Stuart Parmenter]

r=pavlov

Comment 10

[Gagan]

got r=Pavlov

Comment 11

[Asa Dotzler [:asa]]

a= [e-mail] on behalf of drivers

Comment 12

[Gagan]

Fix checked in.