Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(156)

Side by Side Diff: net/spdy/spdy_buffer.cc

Issue 1852423004: Implement SpdySerializedFrame move semantics. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « net/spdy/spdy_buffer.h ('k') | net/spdy/spdy_buffer_unittest.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "net/spdy/spdy_buffer.h" 5 #include "net/spdy/spdy_buffer.h"
6 6
7 #include <cstring> 7 #include <cstring>
8 #include <utility> 8 #include <utility>
9 9
10 #include "base/callback.h" 10 #include "base/callback.h"
11 #include "base/logging.h" 11 #include "base/logging.h"
12 #include "base/macros.h" 12 #include "base/macros.h"
13 #include "net/base/io_buffer.h" 13 #include "net/base/io_buffer.h"
14 #include "net/spdy/spdy_protocol.h" 14 #include "net/spdy/spdy_protocol.h"
15 15
16 namespace net { 16 namespace net {
17 17
18 namespace { 18 namespace {
19 19
20 // Bound on largest frame any SPDY version has allowed. 20 // Bound on largest frame any SPDY version has allowed.
21 const size_t kMaxSpdyFrameSize = 0x00ffffff; 21 const size_t kMaxSpdyFrameSize = 0x00ffffff;
22 22
23 // Makes a SpdyFrame with |size| bytes of data copied from 23 // Makes a SpdySerializedFrame with |size| bytes of data copied from |data|.
24 // |data|. |data| must be non-NULL and |size| must be positive. 24 // |data| must be non-NULL and |size| must be positive.
25 scoped_ptr<SpdyFrame> MakeSpdyFrame(const char* data, size_t size) { 25 scoped_ptr<SpdySerializedFrame> MakeSpdySerializedFrame(const char* data,
26 size_t size) {
26 DCHECK(data); 27 DCHECK(data);
27 CHECK_GT(size, 0u); 28 CHECK_GT(size, 0u);
28 CHECK_LE(size, kMaxSpdyFrameSize); 29 CHECK_LE(size, kMaxSpdyFrameSize);
29 scoped_ptr<char[]> frame_data(new char[size]); 30 scoped_ptr<char[]> frame_data(new char[size]);
30 std::memcpy(frame_data.get(), data, size); 31 std::memcpy(frame_data.get(), data, size);
31 scoped_ptr<SpdyFrame> frame( 32 scoped_ptr<SpdySerializedFrame> frame(new SpdySerializedFrame(
32 new SpdyFrame(frame_data.release(), size, true /* owns_buffer */)); 33 frame_data.release(), size, true /* owns_buffer */));
33 return frame; 34 return frame;
34 } 35 }
35 36
36 } // namespace 37 } // namespace
37 38
38 // This class is an IOBuffer implementation that simply holds a 39 // This class is an IOBuffer implementation that simply holds a
39 // reference to a SharedFrame object and a fixed offset. Used by 40 // reference to a SharedFrame object and a fixed offset. Used by
40 // SpdyBuffer::GetIOBufferForRemainingData(). 41 // SpdyBuffer::GetIOBufferForRemainingData().
41 class SpdyBuffer::SharedFrameIOBuffer : public IOBuffer { 42 class SpdyBuffer::SharedFrameIOBuffer : public IOBuffer {
42 public: 43 public:
43 SharedFrameIOBuffer(const scoped_refptr<SharedFrame>& shared_frame, 44 SharedFrameIOBuffer(const scoped_refptr<SharedFrame>& shared_frame,
44 size_t offset) 45 size_t offset)
45 : IOBuffer(shared_frame->data->data() + offset), 46 : IOBuffer(shared_frame->data->data() + offset),
46 shared_frame_(shared_frame) {} 47 shared_frame_(shared_frame) {}
47 48
48 private: 49 private:
49 ~SharedFrameIOBuffer() override { 50 ~SharedFrameIOBuffer() override {
50 // Prevent ~IOBuffer() from trying to delete |data_|. 51 // Prevent ~IOBuffer() from trying to delete |data_|.
51 data_ = NULL; 52 data_ = NULL;
52 } 53 }
53 54
54 const scoped_refptr<SharedFrame> shared_frame_; 55 const scoped_refptr<SharedFrame> shared_frame_;
55 56
56 DISALLOW_COPY_AND_ASSIGN(SharedFrameIOBuffer); 57 DISALLOW_COPY_AND_ASSIGN(SharedFrameIOBuffer);
57 }; 58 };
58 59
59 SpdyBuffer::SpdyBuffer(scoped_ptr<SpdyFrame> frame) 60 SpdyBuffer::SpdyBuffer(scoped_ptr<SpdySerializedFrame> frame)
60 : shared_frame_(new SharedFrame()), 61 : shared_frame_(new SharedFrame()), offset_(0) {
61 offset_(0) {
62 shared_frame_->data = std::move(frame); 62 shared_frame_->data = std::move(frame);
63 } 63 }
64 64
65 // The given data may not be strictly a SPDY frame; we (ab)use 65 // The given data may not be strictly a SPDY frame; we (ab)use
66 // |frame_| just as a container. 66 // |frame_| just as a container.
67 SpdyBuffer::SpdyBuffer(const char* data, size_t size) : 67 SpdyBuffer::SpdyBuffer(const char* data, size_t size) :
68 shared_frame_(new SharedFrame()), 68 shared_frame_(new SharedFrame()),
69 offset_(0) { 69 offset_(0) {
70 CHECK_GT(size, 0u); 70 CHECK_GT(size, 0u);
71 CHECK_LE(size, kMaxSpdyFrameSize); 71 CHECK_LE(size, kMaxSpdyFrameSize);
72 shared_frame_->data = MakeSpdyFrame(data, size); 72 shared_frame_->data = MakeSpdySerializedFrame(data, size);
73 } 73 }
74 74
75 SpdyBuffer::~SpdyBuffer() { 75 SpdyBuffer::~SpdyBuffer() {
76 if (GetRemainingSize() > 0) 76 if (GetRemainingSize() > 0)
77 ConsumeHelper(GetRemainingSize(), DISCARD); 77 ConsumeHelper(GetRemainingSize(), DISCARD);
78 } 78 }
79 79
80 const char* SpdyBuffer::GetRemainingData() const { 80 const char* SpdyBuffer::GetRemainingData() const {
81 return shared_frame_->data->data() + offset_; 81 return shared_frame_->data->data() + offset_;
82 } 82 }
(...skipping 19 matching lines...) Expand all
102 DCHECK_GE(consume_size, 1u); 102 DCHECK_GE(consume_size, 1u);
103 DCHECK_LE(consume_size, GetRemainingSize()); 103 DCHECK_LE(consume_size, GetRemainingSize());
104 offset_ += consume_size; 104 offset_ += consume_size;
105 for (std::vector<ConsumeCallback>::const_iterator it = 105 for (std::vector<ConsumeCallback>::const_iterator it =
106 consume_callbacks_.begin(); it != consume_callbacks_.end(); ++it) { 106 consume_callbacks_.begin(); it != consume_callbacks_.end(); ++it) {
107 it->Run(consume_size, consume_source); 107 it->Run(consume_size, consume_source);
108 } 108 }
109 }; 109 };
110 110
111 } // namespace net 111 } // namespace net
OLDNEW
« no previous file with comments | « net/spdy/spdy_buffer.h ('k') | net/spdy/spdy_buffer_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698