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

Side by Side Diff: net/quic/quic_data_stream.cc

Issue 612323013: QUIC - (no behavior change) s/NULL/nullptr/g in .../quic/... (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 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/quic/quic_crypto_stream.cc ('k') | net/quic/quic_data_stream_test.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 2013 The Chromium Authors. All rights reserved. 1 // Copyright 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/quic/quic_data_stream.h" 5 #include "net/quic/quic_data_stream.h"
6 6
7 #include "base/logging.h" 7 #include "base/logging.h"
8 #include "net/quic/quic_session.h" 8 #include "net/quic/quic_session.h"
9 #include "net/quic/quic_utils.h" 9 #include "net/quic/quic_utils.h"
10 #include "net/quic/quic_write_blocked_list.h" 10 #include "net/quic/quic_write_blocked_list.h"
(...skipping 11 matching lines...) Expand all
22 // to set a priority client-side, or cancel a stream before stripping the 22 // to set a priority client-side, or cancel a stream before stripping the
23 // priority from the wire server-side. In either case, start out with a 23 // priority from the wire server-side. In either case, start out with a
24 // priority in the middle. 24 // priority in the middle.
25 QuicPriority kDefaultPriority = 3; 25 QuicPriority kDefaultPriority = 3;
26 26
27 } // namespace 27 } // namespace
28 28
29 QuicDataStream::QuicDataStream(QuicStreamId id, 29 QuicDataStream::QuicDataStream(QuicStreamId id,
30 QuicSession* session) 30 QuicSession* session)
31 : ReliableQuicStream(id, session), 31 : ReliableQuicStream(id, session),
32 visitor_(NULL), 32 visitor_(nullptr),
33 headers_decompressed_(false), 33 headers_decompressed_(false),
34 priority_(kDefaultPriority), 34 priority_(kDefaultPriority),
35 decompression_failed_(false), 35 decompression_failed_(false),
36 priority_parsed_(false) { 36 priority_parsed_(false) {
37 DCHECK_NE(kCryptoStreamId, id); 37 DCHECK_NE(kCryptoStreamId, id);
38 // Don't receive any callbacks from the sequencer until headers 38 // Don't receive any callbacks from the sequencer until headers
39 // are complete. 39 // are complete.
40 sequencer()->SetBlockedUntilFlush(); 40 sequencer()->SetBlockedUntilFlush();
41 } 41 }
42 42
(...skipping 125 matching lines...) Expand 10 before | Expand all | Expand 10 after
168 } 168 }
169 } 169 }
170 170
171 void QuicDataStream::OnClose() { 171 void QuicDataStream::OnClose() {
172 ReliableQuicStream::OnClose(); 172 ReliableQuicStream::OnClose();
173 173
174 if (visitor_) { 174 if (visitor_) {
175 Visitor* visitor = visitor_; 175 Visitor* visitor = visitor_;
176 // Calling Visitor::OnClose() may result the destruction of the visitor, 176 // Calling Visitor::OnClose() may result the destruction of the visitor,
177 // so we need to ensure we don't call it again. 177 // so we need to ensure we don't call it again.
178 visitor_ = NULL; 178 visitor_ = nullptr;
179 visitor->OnClose(this); 179 visitor->OnClose(this);
180 } 180 }
181 } 181 }
182 182
183 bool QuicDataStream::FinishedReadingHeaders() { 183 bool QuicDataStream::FinishedReadingHeaders() {
184 return headers_decompressed_ && decompressed_headers_.empty(); 184 return headers_decompressed_ && decompressed_headers_.empty();
185 } 185 }
186 186
187 } // namespace net 187 } // namespace net
OLDNEW
« no previous file with comments | « net/quic/quic_crypto_stream.cc ('k') | net/quic/quic_data_stream_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698