Title: [126223] trunk/Source/WebCore
Revision
126223
Author
[email protected]
Date
2012-08-21 16:53:21 -0700 (Tue, 21 Aug 2012)

Log Message

[BlackBerry] Use new method to create socket handle
https://bugs.webkit.org/show_bug.cgi?id=89991

Patch by Lianghui Chen <[email protected]> on 2012-08-21
Reviewed by Rob Buis.

Update the way to create SocketStream as the platform API has changed.

No new tests as no change of behaviour, just API change adaptation.

* platform/network/blackberry/SocketStreamHandleBlackBerry.cpp:
(WebCore::SocketStreamHandle::SocketStreamHandle):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (126222 => 126223)


--- trunk/Source/WebCore/ChangeLog	2012-08-21 23:51:39 UTC (rev 126222)
+++ trunk/Source/WebCore/ChangeLog	2012-08-21 23:53:21 UTC (rev 126223)
@@ -1,3 +1,17 @@
+2012-08-21  Lianghui Chen  <[email protected]>
+
+        [BlackBerry] Use new method to create socket handle
+        https://bugs.webkit.org/show_bug.cgi?id=89991
+
+        Reviewed by Rob Buis.
+
+        Update the way to create SocketStream as the platform API has changed.
+
+        No new tests as no change of behaviour, just API change adaptation.
+
+        * platform/network/blackberry/SocketStreamHandleBlackBerry.cpp:
+        (WebCore::SocketStreamHandle::SocketStreamHandle):
+
 2012-08-21  Kentaro Hara  <[email protected]>
 
         [V8] Move compileScript() from V8Proxy to ScriptSourceCode

Modified: trunk/Source/WebCore/platform/network/blackberry/SocketStreamHandleBlackBerry.cpp (126222 => 126223)


--- trunk/Source/WebCore/platform/network/blackberry/SocketStreamHandleBlackBerry.cpp	2012-08-21 23:51:39 UTC (rev 126222)
+++ trunk/Source/WebCore/platform/network/blackberry/SocketStreamHandleBlackBerry.cpp	2012-08-21 23:53:21 UTC (rev 126223)
@@ -65,17 +65,15 @@
     // Create a platform socket stream
     BlackBerry::Platform::NetworkStreamFactory* factory = page->chrome()->platformPageClient()->networkStreamFactory();
     ASSERT(factory);
-    m_socketStream = adoptPtr(factory->createSocketStream(playerId));
-    ASSERT(m_socketStream);
-    m_socketStream->setListener(this);
 
     // Open the socket
     BlackBerry::Platform::NetworkRequest request;
     ReadOnlyLatin1String latin1URL(url.string());
     request.setRequestUrl(latin1URL.data(), latin1URL.length(), "CONNECT", 7);
+    m_socketStream = adoptPtr(factory->createNetworkStream(request, playerId));
+    ASSERT(m_socketStream);
 
-    m_socketStream->setRequest(request);
-
+    m_socketStream->setListener(this);
     m_socketStream->streamOpen();
 }
 
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to