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

Side by Side Diff: chrome/browser/prerender/prerender_tracker_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 <set> 5 #include <set>
6 #include <utility> 6 #include <utility>
7 7
8 #include "base/files/file_path.h" 8 #include "base/files/file_path.h"
9 #include "base/logging.h" 9 #include "base/logging.h"
10 #include "base/message_loop/message_loop.h" 10 #include "base/message_loop/message_loop.h"
(...skipping 157 matching lines...) Expand 10 before | Expand all | Expand 10 after
168 static const int kDefaultRouteId = 100; 168 static const int kDefaultRouteId = 100;
169 169
170 PrerenderTrackerTest() : 170 PrerenderTrackerTest() :
171 ui_thread_(BrowserThread::UI, &message_loop_), 171 ui_thread_(BrowserThread::UI, &message_loop_),
172 io_thread_(BrowserThread::IO, &message_loop_), 172 io_thread_(BrowserThread::IO, &message_loop_),
173 prerender_manager_(prerender_tracker()), 173 prerender_manager_(prerender_tracker()),
174 test_contents_(&prerender_manager_, kDefaultChildId, kDefaultRouteId) { 174 test_contents_(&prerender_manager_, kDefaultChildId, kDefaultRouteId) {
175 chrome_browser_net::SetUrlRequestMocksEnabled(true); 175 chrome_browser_net::SetUrlRequestMocksEnabled(true);
176 } 176 }
177 177
178 virtual ~PrerenderTrackerTest() { 178 ~PrerenderTrackerTest() override {
179 chrome_browser_net::SetUrlRequestMocksEnabled(false); 179 chrome_browser_net::SetUrlRequestMocksEnabled(false);
180 180
181 // Cleanup work so the file IO tasks from URLRequestMockHTTPJob 181 // Cleanup work so the file IO tasks from URLRequestMockHTTPJob
182 // are gone. 182 // are gone.
183 content::BrowserThread::GetBlockingPool()->FlushForTesting(); 183 content::BrowserThread::GetBlockingPool()->FlushForTesting();
184 RunEvents(); 184 RunEvents();
185 } 185 }
186 186
187 PrerenderTracker* prerender_tracker() { 187 PrerenderTracker* prerender_tracker() {
188 return g_browser_process->prerender_tracker(); 188 return g_browser_process->prerender_tracker();
(...skipping 143 matching lines...) Expand 10 before | Expand all | Expand 10 after
332 // We should have cancelled the prerender. 332 // We should have cancelled the prerender.
333 EXPECT_EQ(FINAL_STATUS_BAD_DEFERRED_REDIRECT, 333 EXPECT_EQ(FINAL_STATUS_BAD_DEFERRED_REDIRECT,
334 test_contents()->final_status()); 334 test_contents()->final_status());
335 335
336 // Cleanup work so the prerender is gone. 336 // Cleanup work so the prerender is gone.
337 test_contents()->Cancel(); 337 test_contents()->Cancel();
338 RunEvents(); 338 RunEvents();
339 } 339 }
340 340
341 } // namespace prerender 341 } // namespace prerender
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698