1294381 - [e10s-multi] Web socket failure in test_bug803225.html with multiple content processes

Status: RESOLVED FIXED

(Core :: Networking: WebSockets, defect)

Description

[Gabor Krizsanits (INACTIVE)]

open() does not seem to be called on the websocket for some reason.

Comment 1

[u408661]

Michal, IIRC you know websockets. Can you take a look when you get a chance?

Comment 2

[Gabor Krizsanits (INACTIVE)]

After a recent update on my repo the test fails differently... Now the mailto part times out again, and the reason for it seems to be that when the webhandler opens http://example.com/tests/dom/security/test/mixedcontentblocker/file_bug803225_test_mailto.html?s=% the "content-document-global-created" observer reports https://example.com as origin instead of http://example.com.

Clearing needinfo since I cannot reproduce the websocket failure anymore.

Comment 3

[Gabor Krizsanits (INACTIVE)]

Attachment: Delayed process script. v1

sigh... I forgot to enable the process script to be attached to future content processes too.

Comment 4

[Blake Kaplan (:mrbkap) (inactive)]

Comment on attachment 8785180 [details] [diff] [review]
Delayed process script. v1

Review of attachment 8785180 [details] [diff] [review]:
-----------------------------------------------------------------

::: dom/security/test/mixedcontentblocker/file_main_bug803225.html
@@ +132,5 @@
>            }
>          }
>        };
>        os.addObserver(observer, "content-document-global-created", false);
> +    }, true);

Do we have to worry about this observer being attached to every following process and sending useless messages? I guess it's not that much work, so probably not.

Comment 5

[Gabor Krizsanits (INACTIVE)]

(In reply to Blake Kaplan (:mrbkap) from comment #4)
> Do we have to worry about this observer being attached to every following
> process and sending useless messages? I guess it's not that much work, so
> probably not.

Good catch. Probably it's not a huge deal but it's bothering enough, especially if
people start copying it and it becomes a bad habit over time. I will try to add a
removeDelayedProcessScript call in a cleanup section.

Comment 6

[Gabor Krizsanits (INACTIVE)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/2c31f51e6237db885cc14baccd30e13b2034d959
Bug 1294381 - Delayed process script for test_bug803225.html. r=mrbkap

Comment 7

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/2c31f51e6237

Comment 8

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/2c31f51e6237