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

Side by Side Diff: chrome/browser/net/predictor_unittest.cc

Issue 684613002: Standardize usage of virtual/override/final specifiers. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 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 #include <time.h> 5 #include <time.h>
6 6
7 #include <algorithm> 7 #include <algorithm>
8 #include <sstream> 8 #include <sstream>
9 #include <string> 9 #include <string>
10 10
(...skipping 63 matching lines...) Expand 10 before | Expand all | Expand 10 after
74 74
75 class PredictorTest : public testing::Test { 75 class PredictorTest : public testing::Test {
76 public: 76 public:
77 PredictorTest() 77 PredictorTest()
78 : ui_thread_(BrowserThread::UI, &loop_), 78 : ui_thread_(BrowserThread::UI, &loop_),
79 io_thread_(BrowserThread::IO, &loop_), 79 io_thread_(BrowserThread::IO, &loop_),
80 host_resolver_(new net::MockCachingHostResolver()) { 80 host_resolver_(new net::MockCachingHostResolver()) {
81 } 81 }
82 82
83 protected: 83 protected:
84 virtual void SetUp() { 84 void SetUp() override {
85 #if defined(OS_WIN) 85 #if defined(OS_WIN)
86 net::EnsureWinsockInit(); 86 net::EnsureWinsockInit();
87 #endif 87 #endif
88 Predictor::set_max_parallel_resolves( 88 Predictor::set_max_parallel_resolves(
89 Predictor::kMaxSpeculativeParallelResolves); 89 Predictor::kMaxSpeculativeParallelResolves);
90 Predictor::set_max_queueing_delay( 90 Predictor::set_max_queueing_delay(
91 Predictor::kMaxSpeculativeResolveQueueDelayMs); 91 Predictor::kMaxSpeculativeResolveQueueDelayMs);
92 // Since we are using a caching HostResolver, the following latencies will 92 // Since we are using a caching HostResolver, the following latencies will
93 // only be incurred by the first request, after which the result will be 93 // only be incurred by the first request, after which the result will be
94 // cached internally by |host_resolver_|. 94 // cached internally by |host_resolver_|.
(...skipping 839 matching lines...) Expand 10 before | Expand all | Expand 10 after
934 934
935 // Proxy may not be in use (the PAC script has not yet been evaluated), so the 935 // Proxy may not be in use (the PAC script has not yet been evaluated), so the
936 // name has been registered for pre-resolve. 936 // name has been registered for pre-resolve.
937 EXPECT_FALSE(testing_master.work_queue_.IsEmpty()); 937 EXPECT_FALSE(testing_master.work_queue_.IsEmpty());
938 938
939 delete testing_master.proxy_service_; 939 delete testing_master.proxy_service_;
940 testing_master.Shutdown(); 940 testing_master.Shutdown();
941 } 941 }
942 942
943 } // namespace chrome_browser_net 943 } // namespace chrome_browser_net
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698