diff options
author | rsleevi@chromium.org <rsleevi@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-06-02 21:25:43 +0000 |
---|---|---|
committer | rsleevi@chromium.org <rsleevi@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-06-02 21:25:43 +0000 |
commit | f0c8624f0b31037e930b445c015957364a53cd1e (patch) | |
tree | c2546b6266fe1a64daeaee584ee2301507a0b3f7 /ppapi/proxy/websocket_resource.cc | |
parent | 9ce15e12e930570d8486e2e9cf7d1c2f9c7e653d (diff) | |
download | chromium_src-f0c8624f0b31037e930b445c015957364a53cd1e.zip chromium_src-f0c8624f0b31037e930b445c015957364a53cd1e.tar.gz chromium_src-f0c8624f0b31037e930b445c015957364a53cd1e.tar.bz2 |
Update ppapi/ to use scoped_refptr<T>::get() rather than implicit "operator T*"
Linux fixes
BUG=110610
TBR=darin
Review URL: https://chromiumcodereview.appspot.com/15806016
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@203629 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'ppapi/proxy/websocket_resource.cc')
-rw-r--r-- | ppapi/proxy/websocket_resource.cc | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/ppapi/proxy/websocket_resource.cc b/ppapi/proxy/websocket_resource.cc index 7439826..004e648 100644 --- a/ppapi/proxy/websocket_resource.cc +++ b/ppapi/proxy/websocket_resource.cc @@ -89,7 +89,7 @@ int32_t WebSocketResource::Connect( // Get the URL. url_ = StringVar::FromPPVar(url); - if (!url_) + if (!url_.get()) return PP_ERROR_BADARGUMENT; // Get the protocols. @@ -100,7 +100,7 @@ int32_t WebSocketResource::Connect( scoped_refptr<StringVar> protocol(StringVar::FromPPVar(protocols[i])); // Check invalid and empty entries. - if (!protocol || !protocol->value().length()) + if (!protocol.get() || !protocol->value().length()) return PP_ERROR_BADARGUMENT; // Check duplicated protocol entries. @@ -157,7 +157,7 @@ int32_t WebSocketResource::Close(uint16_t code, if (reason.type != PP_VARTYPE_UNDEFINED) { // Validate |reason|. reason_string_var = StringVar::FromPPVar(reason); - if (!reason_string_var || + if (!reason_string_var.get() || reason_string_var->value().size() > kMaxReasonSizeInBytes) return PP_ERROR_BADARGUMENT; reason_string = reason_string_var->value(); @@ -246,12 +246,12 @@ int32_t WebSocketResource::SendMessage(const PP_Var& message) { uint64_t payload_size = 0; if (message.type == PP_VARTYPE_STRING) { scoped_refptr<StringVar> message_string = StringVar::FromPPVar(message); - if (message_string) + if (message_string.get()) payload_size += message_string->value().length(); } else if (message.type == PP_VARTYPE_ARRAY_BUFFER) { scoped_refptr<ArrayBufferVar> message_array_buffer = ArrayBufferVar::FromPPVar(message); - if (message_array_buffer) + if (message_array_buffer.get()) payload_size += message_array_buffer->ByteLength(); } else { // TODO(toyoshim): Support Blob. @@ -268,14 +268,14 @@ int32_t WebSocketResource::SendMessage(const PP_Var& message) { if (message.type == PP_VARTYPE_STRING) { // Convert message to std::string, then send it. scoped_refptr<StringVar> message_string = StringVar::FromPPVar(message); - if (!message_string) + if (!message_string.get()) return PP_ERROR_BADARGUMENT; Post(RENDERER, PpapiHostMsg_WebSocket_SendText(message_string->value())); } else if (message.type == PP_VARTYPE_ARRAY_BUFFER) { // Convert message to std::vector<uint8_t>, then send it. scoped_refptr<ArrayBufferVar> message_arraybuffer = ArrayBufferVar::FromPPVar(message); - if (!message_arraybuffer) + if (!message_arraybuffer.get()) return PP_ERROR_BADARGUMENT; uint8_t* message_data = static_cast<uint8_t*>(message_arraybuffer->Map()); uint32 message_length = message_arraybuffer->ByteLength(); @@ -298,7 +298,7 @@ uint16_t WebSocketResource::GetCloseCode() { } PP_Var WebSocketResource::GetCloseReason() { - if (!close_reason_) + if (!close_reason_.get()) return empty_string_->GetPPVar(); return close_reason_->GetPPVar(); } @@ -312,7 +312,7 @@ PP_Var WebSocketResource::GetExtensions() { } PP_Var WebSocketResource::GetProtocol() { - if (!protocol_) + if (!protocol_.get()) return empty_string_->GetPPVar(); return protocol_->GetPPVar(); } @@ -322,7 +322,7 @@ PP_WebSocketReadyState WebSocketResource::GetReadyState() { } PP_Var WebSocketResource::GetURL() { - if (!url_) + if (!url_.get()) return empty_string_->GetPPVar(); return url_->GetPPVar(); } |