Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(817)

Unified Diff: third_party/WebKit/Source/modules/websockets/DocumentWebSocketChannel.cpp

Issue 2564493002: Make WebSocket available again in service workers (Closed)
Patch Set: a Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/modules/websockets/DocumentWebSocketChannel.cpp
diff --git a/third_party/WebKit/Source/modules/websockets/DocumentWebSocketChannel.cpp b/third_party/WebKit/Source/modules/websockets/DocumentWebSocketChannel.cpp
index 95c0ded764028caec622a5dee76bd01b8675ae98..841bc63fd0b1a7847bb42a57f22721b4f243b255 100644
--- a/third_party/WebKit/Source/modules/websockets/DocumentWebSocketChannel.cpp
+++ b/third_party/WebKit/Source/modules/websockets/DocumentWebSocketChannel.cpp
@@ -174,15 +174,7 @@ bool DocumentWebSocketChannel::connect(const KURL& url,
protocol.split(", ", true, protocols);
}
- if (document()->frame()) {
- // Initialize the WebSocketHandle with the frame's InterfaceProvider to
- // provide the WebSocket implementation with context about this frame.
- // This is important so that the browser can show UI associated with
- // the WebSocket (e.g., for certificate errors).
- m_handle->initialize(document()->frame()->interfaceProvider());
- } else {
- m_handle->initialize(Platform::current()->interfaceProvider());
- }
+ m_handle->initialize(Platform::current()->interfaceProvider());
m_handle->connect(url, protocols, document()->getSecurityOrigin(),
document()->firstPartyForCookies(), document()->userAgent(),
this);

Powered by Google App Engine
This is Rietveld 408576698