Bug 121315 – Inspector Server: queue messages sent before socket connection is established

• Component: New Bugs
• Status: RESOLVED DUPLICATE

[Gustavo Noronha (kov)]

Inspector Server: queue messages sent before socket connection is established

[Gustavo Noronha (kov)]

Created attachment 211576 [details]
Patch

[Timothy Hatcher]

Comment on attachment 211576 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=211576&action=review

> Source/WebInspectorUI/UserInterface/InspectorFrontendHostStub.js:118
> +        remoteConnected: function()
> +        {
> +            for (i = 0; i < this._messagesQueue.length; i++)
> +                WebInspector.socket.send(this._messagesQueue[i]);
> +
> +            delete this._messagesQueue;
> +        },
> +
>          sendMessageToBackend: function(message)
>          {
> +            // Messages may be sent before the websocket is connected, in which case
> +            // queue them for sending as soon as the connection is established.
>              if (WebInspector.socket)
>                  WebInspector.socket.send(message);
> +            else
> +                this._messagesQueue.push(message);

This duplicates a patch I have on bug 121391.

[Timothy Hatcher]

*** This bug has been marked as a duplicate of bug 121391 ***