###!!! 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

RESOLVED FIXED in mozilla0.9.1

Status

()

--
critical
RESOLVED FIXED
18 years ago
18 years ago

People

(Reporter: vikasb, Assigned: gagan)

Tracking

Trunk
mozilla0.9.1
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

18 years ago
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
(Reporter)

Comment 1

18 years ago
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.
Assignee: neeti → darin

Updated

18 years ago
Status: NEW → ASSIGNED
Keywords: nsbeta1
Target Milestone: --- → mozilla0.9.1

Comment 2

18 years ago
dougt: can you help with this one?  bounce back to me if not.
Assignee: darin → dougt
Status: ASSIGNED → NEW

Comment 3

18 years ago
nice try, but currently as doomed as you... 
Assignee: dougt → darin

Comment 4

18 years ago
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

18 years ago
mass move, v2.
qa to me.
QA Contact: tever → benc

Updated

18 years ago
Whiteboard: gagan is working on a patch

Comment 6

18 years ago
-> gagan
Assignee: darin → gagan
(Assignee)

Comment 7

18 years ago
Created attachment 35893 [details] [diff] [review]
patch that fixes this and a bunch of other PAC related bugs :)

Comment 8

18 years ago
sr=darin

Updated

18 years ago
Keywords: patch
Whiteboard: gagan is working on a patch → r=? sr=darin a=?

Comment 9

18 years ago
r=pavlov
(Assignee)

Comment 10

18 years ago
got r=Pavlov
Whiteboard: r=? sr=darin a=? → r=Pavlov sr=darin a=?

Comment 11

18 years ago
a= asa@mozilla.org on behalf of drivers
(Assignee)

Comment 12

18 years ago
Fix checked in.
Status: NEW → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED
Whiteboard: r=Pavlov sr=darin a=?
You need to log in before you can comment on or make changes to this bug.