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

Side by Side Diff: chrome/browser/prerender/prerender_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 <map> 5 #include <map>
6 #include <utility> 6 #include <utility>
7 7
8 #include "base/command_line.h" 8 #include "base/command_line.h"
9 #include "base/format_macros.h" 9 #include "base/format_macros.h"
10 #include "base/memory/scoped_vector.h" 10 #include "base/memory/scoped_vector.h"
(...skipping 310 matching lines...) Expand 10 before | Expand all | Expand 10 after
321 prerender_link_manager_( 321 prerender_link_manager_(
322 new PrerenderLinkManager(prerender_manager_.get())), 322 new PrerenderLinkManager(prerender_manager_.get())),
323 last_prerender_id_(0), 323 last_prerender_id_(0),
324 field_trial_list_(NULL) { 324 field_trial_list_(NULL) {
325 // Enable omnibox prerendering. 325 // Enable omnibox prerendering.
326 CommandLine::ForCurrentProcess()->AppendSwitchASCII( 326 CommandLine::ForCurrentProcess()->AppendSwitchASCII(
327 switches::kPrerenderFromOmnibox, 327 switches::kPrerenderFromOmnibox,
328 switches::kPrerenderFromOmniboxSwitchValueEnabled); 328 switches::kPrerenderFromOmniboxSwitchValueEnabled);
329 } 329 }
330 330
331 virtual ~PrerenderTest() { 331 ~PrerenderTest() override {
332 prerender_link_manager_->OnChannelClosing(kDefaultChildId); 332 prerender_link_manager_->OnChannelClosing(kDefaultChildId);
333 prerender_link_manager_->Shutdown(); 333 prerender_link_manager_->Shutdown();
334 prerender_manager_->Shutdown(); 334 prerender_manager_->Shutdown();
335 } 335 }
336 336
337 UnitTestPrerenderManager* prerender_manager() { 337 UnitTestPrerenderManager* prerender_manager() {
338 return prerender_manager_.get(); 338 return prerender_manager_.get();
339 } 339 }
340 340
341 PrerenderLinkManager* prerender_link_manager() { 341 PrerenderLinkManager* prerender_link_manager() {
(...skipping 1231 matching lines...) Expand 10 before | Expand all | Expand 10 after
1573 CHECK(prerender_handle.get()); 1573 CHECK(prerender_handle.get());
1574 EXPECT_TRUE(prerender_handle->IsPrerendering()); 1574 EXPECT_TRUE(prerender_handle->IsPrerendering());
1575 EXPECT_TRUE(prerender_contents->prerendering_has_started()); 1575 EXPECT_TRUE(prerender_contents->prerendering_has_started());
1576 EXPECT_EQ(prerender_contents, prerender_handle->contents()); 1576 EXPECT_EQ(prerender_contents, prerender_handle->contents());
1577 EXPECT_EQ(ORIGIN_INSTANT, prerender_handle->contents()->origin()); 1577 EXPECT_EQ(ORIGIN_INSTANT, prerender_handle->contents()->origin());
1578 ASSERT_EQ(prerender_contents, prerender_manager()->FindAndUseEntry(url)); 1578 ASSERT_EQ(prerender_contents, prerender_manager()->FindAndUseEntry(url));
1579 EXPECT_FALSE(prerender_handle->IsPrerendering()); 1579 EXPECT_FALSE(prerender_handle->IsPrerendering());
1580 } 1580 }
1581 1581
1582 } // namespace prerender 1582 } // namespace prerender
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698