Title: [147915] trunk/Source/WebCore
Revision
147915
Author
[email protected]
Date
2013-04-08 08:30:04 -0700 (Mon, 08 Apr 2013)

Log Message

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

Patch by Seokju Kwon <[email protected]> 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):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (147914 => 147915)


--- trunk/Source/WebCore/ChangeLog	2013-04-08 15:26:39 UTC (rev 147914)
+++ trunk/Source/WebCore/ChangeLog	2013-04-08 15:30:04 UTC (rev 147915)
@@ -1,3 +1,18 @@
+2013-04-08  Seokju Kwon  <[email protected]>
+
+        [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  <[email protected]>
 
         [GTK][EFL] HEAD requests changed to GET after 303 redirection

Modified: trunk/Source/WebCore/platform/network/blackberry/SocketStreamError.h (147914 => 147915)


--- trunk/Source/WebCore/platform/network/blackberry/SocketStreamError.h	2013-04-08 15:26:39 UTC (rev 147914)
+++ trunk/Source/WebCore/platform/network/blackberry/SocketStreamError.h	2013-04-08 15:30:04 UTC (rev 147915)
@@ -1,5 +1,6 @@
 /*
  * Copyright (C) 2009 Google Inc.  All rights reserved.
+ * Copyright (C) 2013 Seokju Kwon ([email protected])
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
@@ -42,6 +43,10 @@
         : SocketStreamErrorBase(errorCode)
     {
     }
+    SocketStreamError(int errorCode, const BlackBerry::Platform::String& description)
+        : SocketStreamErrorBase(errorCode, String(), description)
+    {
+    }
 
 };
 

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


--- trunk/Source/WebCore/platform/network/blackberry/SocketStreamHandleBlackBerry.cpp	2013-04-08 15:26:39 UTC (rev 147914)
+++ trunk/Source/WebCore/platform/network/blackberry/SocketStreamHandleBlackBerry.cpp	2013-04-08 15:30:04 UTC (rev 147915)
@@ -105,7 +105,7 @@
     RefPtr<SocketStreamHandle> protect(this);
     m_status = status;
     if (FilterStream::StatusSuccess != status)
-        m_client->didFailSocketStream(this, SocketStreamError(status));
+        m_client->didFailSocketStream(this, SocketStreamError(status, message));
     else {
         m_state = Open;
         m_client->didOpenSocketStream(this);
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to