Closed Bug 637336 Opened 13 years ago Closed 13 years ago

HttpChannelChild::OnRedirectVerifyCallback should return nsresult but tries to return boolean value

Categories

(Core :: Networking: HTTP, defect)

Product:
Core
Component:
Networking: HTTP
Platform:
x86
Linux
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED DUPLICATE of bug 637339

People

(Reporter: jdm, Assigned: u408661)

Details

Specifically:

>  return SendRedirect2Verify(result, *headerTuples);

It's a good thing this value is never used.
Assignee: nobody → hurley
Nick, see also bug 637339 comment 1 before you start writing a patch for this.
So I think we want to simply call SendRedirect2Verify() (with a guard for mIPCOpen), and then return NS_OK.  There's really no failure case here to report to the caller (i.e the redirect observer).
Agree with comment 2.
This got folded into bug 637339
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.