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

Unified Diff: net/spdy/spdy_buffer.cc

Issue 992733002: Remove //net (except for Android test stuff) and sdch (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 5 years, 9 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « net/spdy/spdy_buffer.h ('k') | net/spdy/spdy_buffer_producer.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/spdy/spdy_buffer.cc
diff --git a/net/spdy/spdy_buffer.cc b/net/spdy/spdy_buffer.cc
deleted file mode 100644
index a1fd8c8c7bc04415f98b3b8c788c6b15b6048543..0000000000000000000000000000000000000000
--- a/net/spdy/spdy_buffer.cc
+++ /dev/null
@@ -1,113 +0,0 @@
-// Copyright (c) 2013 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "net/spdy/spdy_buffer.h"
-
-#include <cstring>
-
-#include "base/callback.h"
-#include "base/logging.h"
-#include "base/profiler/scoped_tracker.h"
-#include "net/base/io_buffer.h"
-#include "net/spdy/spdy_protocol.h"
-
-namespace net {
-
-namespace {
-
-// Bound on largest frame any SPDY version has allowed.
-const size_t kMaxSpdyFrameSize = 0x00ffffff;
-
-// Makes a SpdyFrame with |size| bytes of data copied from
-// |data|. |data| must be non-NULL and |size| must be positive.
-scoped_ptr<SpdyFrame> MakeSpdyFrame(const char* data, size_t size) {
- DCHECK(data);
- CHECK_GT(size, 0u);
- CHECK_LE(size, kMaxSpdyFrameSize);
- scoped_ptr<char[]> frame_data(new char[size]);
- std::memcpy(frame_data.get(), data, size);
- scoped_ptr<SpdyFrame> frame(
- new SpdyFrame(frame_data.release(), size, true /* owns_buffer */));
- return frame.Pass();
-}
-
-} // namespace
-
-// This class is an IOBuffer implementation that simply holds a
-// reference to a SharedFrame object and a fixed offset. Used by
-// SpdyBuffer::GetIOBufferForRemainingData().
-class SpdyBuffer::SharedFrameIOBuffer : public IOBuffer {
- public:
- SharedFrameIOBuffer(const scoped_refptr<SharedFrame>& shared_frame,
- size_t offset)
- : IOBuffer(shared_frame->data->data() + offset),
- shared_frame_(shared_frame) {}
-
- private:
- ~SharedFrameIOBuffer() override {
- // Prevent ~IOBuffer() from trying to delete |data_|.
- data_ = NULL;
- }
-
- const scoped_refptr<SharedFrame> shared_frame_;
-
- DISALLOW_COPY_AND_ASSIGN(SharedFrameIOBuffer);
-};
-
-SpdyBuffer::SpdyBuffer(scoped_ptr<SpdyFrame> frame)
- : shared_frame_(new SharedFrame()),
- offset_(0) {
- shared_frame_->data = frame.Pass();
-}
-
-// The given data may not be strictly a SPDY frame; we (ab)use
-// |frame_| just as a container.
-SpdyBuffer::SpdyBuffer(const char* data, size_t size) :
- shared_frame_(new SharedFrame()),
- offset_(0) {
- CHECK_GT(size, 0u);
- CHECK_LE(size, kMaxSpdyFrameSize);
- shared_frame_->data = MakeSpdyFrame(data, size);
-}
-
-SpdyBuffer::~SpdyBuffer() {
- if (GetRemainingSize() > 0)
- ConsumeHelper(GetRemainingSize(), DISCARD);
-}
-
-const char* SpdyBuffer::GetRemainingData() const {
- return shared_frame_->data->data() + offset_;
-}
-
-size_t SpdyBuffer::GetRemainingSize() const {
- return shared_frame_->data->size() - offset_;
-}
-
-void SpdyBuffer::AddConsumeCallback(const ConsumeCallback& consume_callback) {
- consume_callbacks_.push_back(consume_callback);
-}
-
-void SpdyBuffer::Consume(size_t consume_size) {
- // TODO(pkasting): Remove ScopedTracker below once crbug.com/457517 is fixed.
- tracked_objects::ScopedTracker tracking_profile(
- FROM_HERE_WITH_EXPLICIT_FUNCTION("457517 SpdyBuffer::Consume"));
- ConsumeHelper(consume_size, CONSUME);
-};
-
-IOBuffer* SpdyBuffer::GetIOBufferForRemainingData() {
- return new SharedFrameIOBuffer(shared_frame_, offset_);
-}
-
-void SpdyBuffer::ConsumeHelper(size_t consume_size,
- ConsumeSource consume_source) {
- DCHECK_GE(consume_size, 1u);
- DCHECK_LE(consume_size, GetRemainingSize());
- offset_ += consume_size;
- for (std::vector<ConsumeCallback>::const_iterator it =
- consume_callbacks_.begin(); it != consume_callbacks_.end(); ++it) {
- it->Run(consume_size, consume_source);
- }
-};
-
-} // namespace net
« no previous file with comments | « net/spdy/spdy_buffer.h ('k') | net/spdy/spdy_buffer_producer.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698