Changeset 147915 in webkit

Timestamp:

Apr 8, 2013 8:30:04 AM (11 years ago)

Author:

commit-queue@webkit.org

Message:

[BlackBerry] Provide more specific error description for SocketStreamError
​https://bugs.webkit.org/show_bug.cgi?id=114134

Patch by Seokju Kwon <Seokju Kwon> on 2013-04-08
Reviewed by Rob Buis.

Pass the reason of a failure to SocketStreamError instance
and notify it via SocketStreamHandleClient::didFailSocketStream().

• platform/network/blackberry/SocketStreamError.h:

(WebCore::SocketStreamError::SocketStreamError):

• platform/network/blackberry/SocketStreamHandleBlackBerry.cpp:

(WebCore::SocketStreamHandle::notifyStatusReceived):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/network/blackberry/SocketStreamError.h (modified) (2 diffs)
• platform/network/blackberry/SocketStreamHandleBlackBerry.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2013-04-08  Seokju Kwon  <seokju.kwon@gmail.com>
+
+        [BlackBerry] Provide more specific error description for SocketStreamError
+        https://bugs.webkit.org/show_bug.cgi?id=114134
+
+        Reviewed by Rob Buis.
+
+        Pass the reason of a failure to SocketStreamError instance
+        and notify it via SocketStreamHandleClient::didFailSocketStream().
+
+        * platform/network/blackberry/SocketStreamError.h:
+        (WebCore::SocketStreamError::SocketStreamError):
+        * platform/network/blackberry/SocketStreamHandleBlackBerry.cpp:
+        (WebCore::SocketStreamHandle::notifyStatusReceived):
+
 2013-04-08  Youenn Fablet  <youennf@gmail.com>
 

trunk/Source/WebCore/platform/network/blackberry/SocketStreamError.h

@@ -1 +1 @@
 /*
  * Copyright (C) 2009 Google Inc.  All rights reserved.
+ * Copyright (C) 2013 Seokju Kwon (seokju.kwon@gmail.com)
  *
  * Redistribution and use in source and binary forms, with or without
@@ -43 +44 @@
     {
     }
+    SocketStreamError(int errorCode, const BlackBerry::Platform::String& description)
+        : SocketStreamErrorBase(errorCode, String(), description)
+    {
+    }
 
 };

trunk/Source/WebCore/platform/network/blackberry/SocketStreamHandleBlackBerry.cpp

@@ -106 +106 @@
     m_status = status;
     if (FilterStream::StatusSuccess != status)
-        m_client->didFailSocketStream(this, SocketStreamError(status));
+        m_client->didFailSocketStream(this, SocketStreamError(status, message));
     else {
         m_state = Open;