554942 - [E10s] Sort out what object TabParent::GetGlobalJSObject should return and how

Status: RESOLVED FIXED

(Core :: IPC, defect)

Description

[Olli Pettay [:smaug][[e-mail]]]

The method could return TabChildGlobal or top level content page's
global object.

Comment 1

[Ben Newman (:bnewman) (:benjamn)]

Attachment: Patch that sorts things out.

Back to having only one ContextWrapperChild per TabChild.

Comment 2

[Olli Pettay [:smaug][[e-mail]]]

Comment on attachment 435275 [details] [diff] [review]
Patch that sorts things out.

>Bug 554942 - [E10s] Sort out what object TabParent::GetGlobalJSObject should return and how.
>
>diff --git a/dom/ipc/TabChild.cpp b/dom/ipc/TabChild.cpp
>--- a/dom/ipc/TabChild.cpp
>+++ b/dom/ipc/TabChild.cpp
>@@ -471,17 +471,25 @@ TabChild::RecvloadURL(const nsCString& u
> 
>     nsresult rv = mWebNav->LoadURI(NS_ConvertUTF8toUTF16(uri).get(),
>                                    nsIWebNavigation::LOAD_FLAGS_NONE,
>                                    NULL, NULL, NULL);
>     if (NS_FAILED(rv)) {
>         NS_WARNING("mWebNav->LoadURI failed. Eating exception, what else can I do?");
>     }
> 
>-    SendPContextWrapperConstructor()->SendPObjectWrapperConstructor(true);
>+    NS_ASSERTION(mContextWrapper, "Should have created mContextWrapper already");
>+    if (mContextWrapper) {
>+        JSContext* cx = mContextWrapper->GetContext();
>+        JSAutoRequest ar(cx);
>+        jsval cval;
>+        if (JS_GetProperty(cx, JS_GetGlobalObject(cx), "content", &cval) &&
>+            JSVAL_IS_OBJECT(cval))
>+            mContextWrapper->GetOrCreateWrapper(JSVAL_TO_OBJECT(cval), true);
>+    }

Do we really want to do this every time a new page is started to load?
Could we do this all just when mContextWrapper is created?
Docshell should be creating the global object when needed.

Comment 3

[Ben Newman (:bnewman) (:benjamn)]

Attachment: Simpler patch that sorts things out.

(In reply to comment #2)
> Do we really want to do this every time a new page is started to load?
> Could we do this all just when mContextWrapper is created?
> Docshell should be creating the global object when needed.

Yes, yes.  Much simpler that way!  Thanks.

Comment 4

[Olli Pettay [:smaug][[e-mail]]]

Comment on attachment 435672 [details] [diff] [review]
Simpler patch that sorts things out.

If you have tested this all works ;)  r=me.

Comment 5

[Ben Newman (:bnewman) (:benjamn)]

Pushed to projects/electrolysis:
http://hg.mozilla.org/projects/electrolysis/rev/f22ca0221b4e