WebSocket Pepper API: C++ utility class implementation.

webkit/plugins/ppapi/ppb_websocket_impl.cc

Index: webkit/plugins/ppapi/ppb_websocket_impl.cc
diff --git a/webkit/plugins/ppapi/ppb_websocket_impl.cc b/webkit/plugins/ppapi/ppb_websocket_impl.cc
index a7fc973b2a8ec0009437f0290b5af59938bb784a..dc52f379773e9727a5f0cda34eeba052a395afb6 100644
--- a/webkit/plugins/ppapi/ppb_websocket_impl.cc
+++ b/webkit/plugins/ppapi/ppb_websocket_impl.cc
@@ -425,9 +425,11 @@ PP_Bool PPB_WebSocket_Impl::SetBinaryType(
     default:
       return PP_FALSE;
   }
-  if (!websocket_.get())
-    return PP_FALSE;
-  websocket_->setBinaryType(binary_type_);
+  // WebKit API setBinaryType() is called when Connect() is called.
+  // If the websocket_ contains an object; it means Connect() is already
+  // called, call WebKit API here to reflect the setting as soon as possible.
+  if (websocket_.get())
+    websocket_->setBinaryType(binary_type_);

[dmichael (off chromium), 2012/02/01 23:05:18]

This looks like leftovers from another CL?

[Takashi Toyoshima, 2012/02/03 07:58:27]

Yes.
This difference was removed in the next patch set 16.

   return PP_TRUE;
 }