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

Side by Side Diff: net/http/bidirectional_stream.cc

Issue 2837863004: Use base::OneShotTimer instead of base::Timer(false, false)
Patch Set: rebase Created 3 years, 7 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/dns/mdns_client_impl.cc ('k') | net/http/bidirectional_stream_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 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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/http/bidirectional_stream.h" 5 #include "net/http/bidirectional_stream.h"
6 6
7 #include <memory> 7 #include <memory>
8 #include <string> 8 #include <string>
9 9
10 #include "base/bind.h" 10 #include "base/bind.h"
(...skipping 55 matching lines...) Expand 10 before | Expand all | Expand 10 after
66 66
67 BidirectionalStream::BidirectionalStream( 67 BidirectionalStream::BidirectionalStream(
68 std::unique_ptr<BidirectionalStreamRequestInfo> request_info, 68 std::unique_ptr<BidirectionalStreamRequestInfo> request_info,
69 HttpNetworkSession* session, 69 HttpNetworkSession* session,
70 bool send_request_headers_automatically, 70 bool send_request_headers_automatically,
71 Delegate* delegate) 71 Delegate* delegate)
72 : BidirectionalStream(std::move(request_info), 72 : BidirectionalStream(std::move(request_info),
73 session, 73 session,
74 send_request_headers_automatically, 74 send_request_headers_automatically,
75 delegate, 75 delegate,
76 base::MakeUnique<base::Timer>(false, false)) {} 76 base::MakeUnique<base::OneShotTimer>()) {}
77 77
78 BidirectionalStream::BidirectionalStream( 78 BidirectionalStream::BidirectionalStream(
79 std::unique_ptr<BidirectionalStreamRequestInfo> request_info, 79 std::unique_ptr<BidirectionalStreamRequestInfo> request_info,
80 HttpNetworkSession* session, 80 HttpNetworkSession* session,
81 bool send_request_headers_automatically, 81 bool send_request_headers_automatically,
82 Delegate* delegate, 82 Delegate* delegate,
83 std::unique_ptr<base::Timer> timer) 83 std::unique_ptr<base::Timer> timer)
84 : request_info_(std::move(request_info)), 84 : request_info_(std::move(request_info)),
85 net_log_(NetLogWithSource::Make(session->net_log(), 85 net_log_(NetLogWithSource::Make(session->net_log(),
86 NetLogSourceType::BIDIRECTIONAL_STREAM)), 86 NetLogSourceType::BIDIRECTIONAL_STREAM)),
(...skipping 355 matching lines...) Expand 10 before | Expand all | Expand 10 after
442 "Net.BidirectionalStream.TimeToSendEnd.QUIC", 442 "Net.BidirectionalStream.TimeToSendEnd.QUIC",
443 load_timing_info_.send_end - load_timing_info_.request_start); 443 load_timing_info_.send_end - load_timing_info_.request_start);
444 UMA_HISTOGRAM_COUNTS("Net.BidirectionalStream.ReceivedBytes.QUIC", 444 UMA_HISTOGRAM_COUNTS("Net.BidirectionalStream.ReceivedBytes.QUIC",
445 stream_impl_->GetTotalReceivedBytes()); 445 stream_impl_->GetTotalReceivedBytes());
446 UMA_HISTOGRAM_COUNTS("Net.BidirectionalStream.SentBytes.QUIC", 446 UMA_HISTOGRAM_COUNTS("Net.BidirectionalStream.SentBytes.QUIC",
447 stream_impl_->GetTotalSentBytes()); 447 stream_impl_->GetTotalSentBytes());
448 } 448 }
449 } 449 }
450 450
451 } // namespace net 451 } // namespace net
OLDNEW
« no previous file with comments | « net/dns/mdns_client_impl.cc ('k') | net/http/bidirectional_stream_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698