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

Side by Side Diff: net/quic/quic_chromium_connection_helper.h

Issue 1572353002: QuicAlarms are now allocated out of an arena in QuicConnection (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Merge recent changes. Created 4 years, 11 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_arena_scoped_ptr_test.cc ('k') | net/quic/quic_chromium_connection_helper.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) 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 // The Chrome-specific helper for QuicConnection which uses 5 // The Chrome-specific helper for QuicConnection which uses
6 // a TaskRunner for alarms, and uses a DatagramClientSocket for writing data. 6 // a TaskRunner for alarms, and uses a DatagramClientSocket for writing data.
7 7
8 #ifndef NET_QUIC_QUIC_CONNECTION_HELPER_H_ 8 #ifndef NET_QUIC_QUIC_CONNECTION_HELPER_H_
9 #define NET_QUIC_QUIC_CONNECTION_HELPER_H_ 9 #define NET_QUIC_QUIC_CONNECTION_HELPER_H_
10 10
(...skipping 22 matching lines...) Expand all
33 public: 33 public:
34 QuicChromiumConnectionHelper(base::TaskRunner* task_runner, 34 QuicChromiumConnectionHelper(base::TaskRunner* task_runner,
35 const QuicClock* clock, 35 const QuicClock* clock,
36 QuicRandom* random_generator); 36 QuicRandom* random_generator);
37 ~QuicChromiumConnectionHelper() override; 37 ~QuicChromiumConnectionHelper() override;
38 38
39 // QuicConnectionHelperInterface 39 // QuicConnectionHelperInterface
40 const QuicClock* GetClock() const override; 40 const QuicClock* GetClock() const override;
41 QuicRandom* GetRandomGenerator() override; 41 QuicRandom* GetRandomGenerator() override;
42 QuicAlarm* CreateAlarm(QuicAlarm::Delegate* delegate) override; 42 QuicAlarm* CreateAlarm(QuicAlarm::Delegate* delegate) override;
43 QuicArenaScopedPtr<QuicAlarm> CreateAlarm(
44 QuicArenaScopedPtr<QuicAlarm::Delegate> delegate,
45 QuicConnectionArena* arena) override;
43 QuicBufferAllocator* GetBufferAllocator() override; 46 QuicBufferAllocator* GetBufferAllocator() override;
44 47
45 private: 48 private:
46 base::TaskRunner* task_runner_; 49 base::TaskRunner* task_runner_;
47 const QuicClock* clock_; 50 const QuicClock* clock_;
48 QuicRandom* random_generator_; 51 QuicRandom* random_generator_;
49 SimpleBufferAllocator buffer_allocator_; 52 SimpleBufferAllocator buffer_allocator_;
50 base::WeakPtrFactory<QuicChromiumConnectionHelper> weak_factory_; 53 base::WeakPtrFactory<QuicChromiumConnectionHelper> weak_factory_;
51 54
52 DISALLOW_COPY_AND_ASSIGN(QuicChromiumConnectionHelper); 55 DISALLOW_COPY_AND_ASSIGN(QuicChromiumConnectionHelper);
53 }; 56 };
54 57
55 } // namespace net 58 } // namespace net
56 59
57 #endif // NET_QUIC_QUIC_CONNECTION_HELPER_H_ 60 #endif // NET_QUIC_QUIC_CONNECTION_HELPER_H_
OLDNEW
« no previous file with comments | « net/quic/quic_arena_scoped_ptr_test.cc ('k') | net/quic/quic_chromium_connection_helper.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698