Closed Bug 558801 Opened 12 years ago Closed 12 years ago

e10s HTTP: support XMLHttpRequests under e10s

Categories

(Core Graveyard :: Tracking, defect)

Other Branch
x86
Linux
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: jduell.mcbugs, Unassigned)

References

Details

Attachments

(1 obsolete file)

Right now XHR requests fail in the testXUL demo because XHR calls nsIHttpChannel::SetOwner(). That's covered by bug 547051.  There may be other work needed.

Anyone want to own this?
Attached patch mOwner patch, 0 (obsolete) — Splinter Review
Reading the comment6 of bug 547051, this seems sufficient.
Assignee: nobody → lusian
Status: NEW → ASSIGNED
Attachment #440136 - Flags: review?(jduell.mcbugs)
Comment on attachment 440136 [details] [diff] [review]
mOwner patch, 0

>diff --git a/netwerk/protocol/http/src/HttpBaseChannel.cpp b/netwerk/protocol/http/src/HttpBaseChannel.cpp
>--- a/netwerk/protocol/http/src/HttpBaseChannel.cpp
>+++ b/netwerk/protocol/http/src/HttpBaseChannel.cpp
>@@ -237,16 +237,32 @@ HttpBaseChannel::GetURI(nsIURI **aURI)
> {
>   NS_ENSURE_ARG_POINTER(aURI);
>   *aURI = mURI;
>   NS_ADDREF(*aURI);
>   return NS_OK;
> }
> 
> NS_IMETHODIMP
>+HttpBaseChannel::GetOwner(nsISupports **aOwner)
>+{
>+    NS_ENSURE_ARG_POINTER(aOwner);
>+    *aOwner = mOwner;
>+    NS_IF_ADDREF(*aOwner);
>+    return NS_OK;
>+}
>+
>+NS_IMETHODIMP
>+HttpBaseChannel::SetOwner(nsISupports *aOwner)
>+{
>+    mOwner = aOwner;
>+    return NS_OK;
>+}
>+

Please use 2-space indents in the future if that's what the rest of the file uses!  I'll fix before I check this in.  


>diff --git a/netwerk/test/unit/test_bug558801.js b/netwerk/test/unit/test_bug558801.js

Nice test!  I renamed it "test_xmlhttprequest.js", just to be more informative.

BTW--this patch would have been better submitted to bug 547051--that's the bug that's going to be fixed by this.  As it turns out we may have other issues with XHR once we're in the browser (see bug 559200).
Attachment #440136 - Flags: review?(jduell.mcbugs) → review+
At this point XHR is working in xpcshell, but we suspect bug 559200 may mess it up in fennec.
Depends on: 559200
test_xmlhttprequest_wrap.js aborts (test_xmlhttprequest.js works fine).  I think Bug 557605 is related to the assertion failure.

###!!! ASSERTION: wrong context on XPCJSContextStack!: 'cx == topJSContext', file c:/mozilla-build/e10s/js/src/xpconnect/src/xpcquickstubs.cpp, line 1144

(http://mxr.mozilla.org/projects-central/source/electrolysis/js/src/xpconnect/src/xpcquickstubs.cpp#1144)

nsIXMLHttpRequest_SetOnreadystatechange in dom_quickstubs.cpp called xpc_qsAssertContextOK.
Depends on: 564351
Let's keep this bug as a tracking bug only.  I've opened bug 564351 for the XHR stubs issue.   Thanks for noticing, Jae-Seong.
Assignee: lusian → nobody
Component: Networking: HTTP → Tracking
QA Contact: networking.http → chofmann
Attachment #440136 - Attachment is obsolete: true
This is a mass change. Every comment has "assigned-to-new" in it.

I didn't look through the bugs, so I'm sorry if I change a bug which shouldn't be changed. But I guess these bugs are just bugs that were once assigned and people forgot to change the Status back when unassigning.
Status: ASSIGNED → NEW
Depends on: 581279, 572979
No longer depends on: 564351
Depends on: 584604
All dependencies fixed.  AFAIK XHR works fine under e10s now.  Reopen if evidence to the contrary emerges.

Yay.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.