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

Side by Side Diff: net/quic/chromium/quic_chromium_client_session.h

Issue 2456713002: Server push cancellation: add methods in session layer to cancel push given the pushed url. Not in … (Closed)
Patch Set: remove todo Created 4 years, 1 month 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
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 // A client specific QuicSession subclass. This class owns the underlying 5 // A client specific QuicSession subclass. This class owns the underlying
6 // QuicConnection and QuicConnectionHelper objects. The connection stores 6 // QuicConnection and QuicConnectionHelper objects. The connection stores
7 // a non-owning pointer to the helper so this session needs to ensure that 7 // a non-owning pointer to the helper so this session needs to ensure that
8 // the helper outlives the connection. 8 // the helper outlives the connection.
9 9
10 #ifndef NET_QUIC_QUIC_CHROMIUM_CLIENT_SESSION_H_ 10 #ifndef NET_QUIC_QUIC_CHROMIUM_CLIENT_SESSION_H_
(...skipping 287 matching lines...) Expand 10 before | Expand all | Expand 10 after
298 bool HasNonMigratableStreams() const; 298 bool HasNonMigratableStreams() const;
299 299
300 void HandlePromised(QuicStreamId associated_id, 300 void HandlePromised(QuicStreamId associated_id,
301 QuicStreamId promised_id, 301 QuicStreamId promised_id,
302 const SpdyHeaderBlock& headers) override; 302 const SpdyHeaderBlock& headers) override;
303 303
304 void DeletePromised(QuicClientPromisedInfo* promised) override; 304 void DeletePromised(QuicClientPromisedInfo* promised) override;
305 305
306 void OnPushStreamTimedOut(QuicStreamId stream_id) override; 306 void OnPushStreamTimedOut(QuicStreamId stream_id) override;
307 307
308 void CancelPush(const GURL& url) override;
Ryan Hamilton 2016/10/27 17:00:25 nit: please comment, although I imagine this is pr
Zhongyi Shi 2016/10/27 17:57:29 Done.
309
308 const LoadTimingInfo::ConnectTiming& GetConnectTiming(); 310 const LoadTimingInfo::ConnectTiming& GetConnectTiming();
309 311
310 protected: 312 protected:
311 // QuicSession methods: 313 // QuicSession methods:
312 bool ShouldCreateIncomingDynamicStream(QuicStreamId id) override; 314 bool ShouldCreateIncomingDynamicStream(QuicStreamId id) override;
313 bool ShouldCreateOutgoingDynamicStream() override; 315 bool ShouldCreateOutgoingDynamicStream() override;
314 316
315 QuicChromiumClientStream* CreateIncomingDynamicStream( 317 QuicChromiumClientStream* CreateIncomingDynamicStream(
316 QuicStreamId id) override; 318 QuicStreamId id) override;
317 319
(...skipping 77 matching lines...) Expand 10 before | Expand all | Expand 10 after
395 // the current sockets_.size() == the passed in value. 397 // the current sockets_.size() == the passed in value.
396 bool migration_pending_; // True while migration is underway. 398 bool migration_pending_; // True while migration is underway.
397 base::WeakPtrFactory<QuicChromiumClientSession> weak_factory_; 399 base::WeakPtrFactory<QuicChromiumClientSession> weak_factory_;
398 400
399 DISALLOW_COPY_AND_ASSIGN(QuicChromiumClientSession); 401 DISALLOW_COPY_AND_ASSIGN(QuicChromiumClientSession);
400 }; 402 };
401 403
402 } // namespace net 404 } // namespace net
403 405
404 #endif // NET_QUIC_QUIC_CHROMIUM_CLIENT_SESSION_H_ 406 #endif // NET_QUIC_QUIC_CHROMIUM_CLIENT_SESSION_H_
OLDNEW
« no previous file with comments | « no previous file | net/quic/chromium/quic_chromium_client_session.cc » ('j') | net/quic/chromium/quic_chromium_client_session.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698