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

Side by Side Diff: content/browser/loader/resource_scheduler_unittest.cc

Issue 678073006: Standardize usage of virtual/override/final specifiers. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix bogus formatting 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 "content/browser/loader/resource_scheduler.h" 5 #include "content/browser/loader/resource_scheduler.h"
6 6
7 #include "base/memory/scoped_vector.h" 7 #include "base/memory/scoped_vector.h"
8 #include "base/message_loop/message_loop.h" 8 #include "base/message_loop/message_loop.h"
9 #include "base/run_loop.h" 9 #include "base/run_loop.h"
10 #include "base/strings/string_number_conversions.h" 10 #include "base/strings/string_number_conversions.h"
(...skipping 138 matching lines...) Expand 10 before | Expand all | Expand 10 after
149 // TODO(aiolos): Remove when throttling and coalescing have both landed. 149 // TODO(aiolos): Remove when throttling and coalescing have both landed.
150 scheduler_.SetThrottleOptionsForTesting(true /* should_throttle */, 150 scheduler_.SetThrottleOptionsForTesting(true /* should_throttle */,
151 false /* should_coalesce */); 151 false /* should_coalesce */);
152 152
153 scheduler_.OnClientCreated(kChildId, kRouteId, true, false); 153 scheduler_.OnClientCreated(kChildId, kRouteId, true, false);
154 scheduler_.OnClientCreated( 154 scheduler_.OnClientCreated(
155 kBackgroundChildId, kBackgroundRouteId, false, false); 155 kBackgroundChildId, kBackgroundRouteId, false, false);
156 context_.set_http_server_properties(http_server_properties_.GetWeakPtr()); 156 context_.set_http_server_properties(http_server_properties_.GetWeakPtr());
157 } 157 }
158 158
159 virtual ~ResourceSchedulerTest() { 159 ~ResourceSchedulerTest() override {
160 scheduler_.OnClientDeleted(kChildId, kRouteId); 160 scheduler_.OnClientDeleted(kChildId, kRouteId);
161 scheduler_.OnClientDeleted(kBackgroundChildId, kBackgroundRouteId); 161 scheduler_.OnClientDeleted(kBackgroundChildId, kBackgroundRouteId);
162 } 162 }
163 163
164 scoped_ptr<net::URLRequest> NewURLRequestWithChildAndRoute( 164 scoped_ptr<net::URLRequest> NewURLRequestWithChildAndRoute(
165 const char* url, 165 const char* url,
166 net::RequestPriority priority, 166 net::RequestPriority priority,
167 int child_id, 167 int child_id,
168 int route_id, 168 int route_id,
169 bool is_async) { 169 bool is_async) {
(...skipping 2033 matching lines...) Expand 10 before | Expand all | Expand 10 after
2203 web_contents_2.reset(); 2203 web_contents_2.reset();
2204 base::RunLoop().RunUntilIdle(); 2204 base::RunLoop().RunUntilIdle();
2205 2205
2206 browser_context.reset(); 2206 browser_context.reset();
2207 render_process_host_factory.reset(); 2207 render_process_host_factory.reset();
2208 } 2208 }
2209 2209
2210 } // unnamed namespace 2210 } // unnamed namespace
2211 2211
2212 } // namespace content 2212 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/loader/resource_loader_unittest.cc ('k') | content/browser/manifest/manifest_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698