26e73cfd90
Instead, give it as an argument of mojom.WebSocketHandshakeClient.OnConnectionEstablished. This is a follow up CL for https://chromium-review.googlesource.com/c/chromium/src/+/1728917. Passing the mojo::InterfaceRequest<WebSocketClient> at OnConnectionEstablished is less error prone because 1) It is unable to bind the implementation before that, and 2) It is now clear that we should detect connection errors on |handshake_client| until the connection is established, and |client| after that. Bug: 989406, 967524 Change-Id: Id9a7f184b678303d5b0ef8ad3b8637056a495933 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1728534 Commit-Queue: Yutaka Hirano <yhirano@chromium.org> Reviewed-by: Yoichi Osato <yoichio@chromium.org> Reviewed-by: Adam Rice <ricea@chromium.org> Reviewed-by: Kinuko Yasuda <kinuko@chromium.org> Reviewed-by: Ken Rockot <rockot@google.com> Cr-Commit-Position: refs/heads/master@{#684215}