From a691b6c34bcceab491224c90359875550996e14b Mon Sep 17 00:00:00 2001 From: "tyoshino@chromium.org" Date: Mon, 24 Mar 2014 16:09:08 +0000 Subject: [PATCH] Rename WebSocketChannel::SendIOBuffer to SendFrameFromIOBuffer SendIOBuffer sounds like sending raw data from an IOBuffer. I'd like to add Frame to make it look like dealing with structured data representing a frame. BUG=none Review URL: https://codereview.chromium.org/196473009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@258943 0039d316-1c4b-4281-b951-d872f2087c98 --- net/websockets/websocket_channel.cc | 9 +++++---- net/websockets/websocket_channel.h | 8 ++++---- 2 files changed, 9 insertions(+), 8 deletions(-) diff --git a/net/websockets/websocket_channel.cc b/net/websockets/websocket_channel.cc index 18e1708ae311d8..25df3a4b7fe087 100644 --- a/net/websockets/websocket_channel.cc +++ b/net/websockets/websocket_channel.cc @@ -387,7 +387,7 @@ void WebSocketChannel::SendFrame(bool fin, // server is not saturated. scoped_refptr buffer(new IOBuffer(data.size())); std::copy(data.begin(), data.end(), buffer->data()); - AllowUnused(SendIOBuffer(fin, op_code, buffer, data.size())); + AllowUnused(SendFrameFromIOBuffer(fin, op_code, buffer, data.size())); // |this| may have been deleted. } @@ -768,7 +768,7 @@ ChannelState WebSocketChannel::HandleFrameByState( case WebSocketFrameHeader::kOpCodePing: VLOG(1) << "Got Ping of size " << size; if (state_ == CONNECTED) - return SendIOBuffer( + return SendFrameFromIOBuffer( true, WebSocketFrameHeader::kOpCodePong, data_buffer, size); VLOG(3) << "Ignored ping in state " << state_; return CHANNEL_ALIVE; @@ -907,7 +907,7 @@ ChannelState WebSocketChannel::HandleDataFrame( return event_interface_->OnDataFrame(final, opcode_to_send, data); } -ChannelState WebSocketChannel::SendIOBuffer( +ChannelState WebSocketChannel::SendFrameFromIOBuffer( bool fin, WebSocketFrameHeader::OpCode op_code, const scoped_refptr& buffer, @@ -985,7 +985,8 @@ ChannelState WebSocketChannel::SendClose(uint16 code, FROM_HERE, timeout_, base::Bind(&WebSocketChannel::CloseTimeout, base::Unretained(this))); - if (SendIOBuffer(true, WebSocketFrameHeader::kOpCodeClose, body, size) == + if (SendFrameFromIOBuffer( + true, WebSocketFrameHeader::kOpCodeClose, body, size) == CHANNEL_DELETED) return CHANNEL_DELETED; return CHANNEL_ALIVE; diff --git a/net/websockets/websocket_channel.h b/net/websockets/websocket_channel.h index 928ba68481c6e3..80c4290976efa6 100644 --- a/net/websockets/websocket_channel.h +++ b/net/websockets/websocket_channel.h @@ -262,10 +262,10 @@ class NET_EXPORT WebSocketChannel { // when the current write finishes. |fin| and |op_code| are defined as for // SendFrame() above, except that |op_code| may also be a control frame // opcode. - ChannelState SendIOBuffer(bool fin, - WebSocketFrameHeader::OpCode op_code, - const scoped_refptr& buffer, - size_t size) WARN_UNUSED_RESULT; + ChannelState SendFrameFromIOBuffer(bool fin, + WebSocketFrameHeader::OpCode op_code, + const scoped_refptr& buffer, + size_t size) WARN_UNUSED_RESULT; // Performs the "Fail the WebSocket Connection" operation as defined in // RFC6455. A NotifyFailure message is sent to the renderer with |message|.