1451411 - land WebSocket event listener leak test

Status: RESOLVED DUPLICATE of bug 1451913

(Core :: DOM: Networking, enhancement, P2)

Description

[Ben Kelly [:bkelly, not reviewing]]

I'd like to land a websocket leak test, but there is an assertion problem we need to fix in WebSocket first.

Comment 1

[Ben Kelly [:bkelly, not reviewing]]

Attachment: P2 Test that WebSocket does not leak through its event listeners. r=baku

There are assertions around properly updating the window's count of web sockets with this test.  I tried a few things to fix it (calling Disconnect() from DisconnectFromOwner(), etc), but its still asserting.  Andrea, any chance you could look at this?

Comment 2

[Ben Kelly [:bkelly, not reviewing]]

Comment on attachment 8964993 [details] [diff] [review]
P2 Test that WebSocket does not leak through its event listeners. r=baku

Andrea, here is an event target leak test for WebSocket.  This passes once I fix the active websocket counter stuff.

Comment 3

[Ben Kelly [:bkelly, not reviewing]]

Comment on attachment 8964993 [details] [diff] [review]
P2 Test that WebSocket does not leak through its event listeners. r=baku

I'm going to move this to bug 1451913.