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

Side by Side Diff: ui/app_list/search/mixer_unittest.cc

Issue 681873002: 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 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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 <string> 5 #include <string>
6 6
7 #include "base/memory/scoped_vector.h" 7 #include "base/memory/scoped_vector.h"
8 #include "base/strings/string16.h" 8 #include "base/strings/string16.h"
9 #include "base/strings/stringprintf.h" 9 #include "base/strings/stringprintf.h"
10 #include "base/strings/utf_string_conversions.h" 10 #include "base/strings/utf_string_conversions.h"
(...skipping 64 matching lines...) Expand 10 before | Expand all | Expand 10 after
75 private: 75 private:
76 std::string prefix_; 76 std::string prefix_;
77 size_t count_; 77 size_t count_;
78 78
79 DISALLOW_COPY_AND_ASSIGN(TestSearchProvider); 79 DISALLOW_COPY_AND_ASSIGN(TestSearchProvider);
80 }; 80 };
81 81
82 class MixerTest : public testing::Test { 82 class MixerTest : public testing::Test {
83 public: 83 public:
84 MixerTest() {} 84 MixerTest() {}
85 virtual ~MixerTest() {} 85 ~MixerTest() override {}
86 86
87 // testing::Test overrides: 87 // testing::Test overrides:
88 virtual void SetUp() override { 88 void SetUp() override {
89 results_.reset(new AppListModel::SearchResults); 89 results_.reset(new AppListModel::SearchResults);
90 90
91 providers_.push_back(new TestSearchProvider("app")); 91 providers_.push_back(new TestSearchProvider("app"));
92 providers_.push_back(new TestSearchProvider("omnibox")); 92 providers_.push_back(new TestSearchProvider("omnibox"));
93 providers_.push_back(new TestSearchProvider("webstore")); 93 providers_.push_back(new TestSearchProvider("webstore"));
94 providers_.push_back(new TestSearchProvider("people")); 94 providers_.push_back(new TestSearchProvider("people"));
95 95
96 mixer_.reset(new Mixer(results_.get())); 96 mixer_.reset(new Mixer(results_.get()));
97 mixer_->Init(); 97 mixer_->Init();
98 mixer_->AddProviderToGroup(Mixer::MAIN_GROUP, providers_[0]); 98 mixer_->AddProviderToGroup(Mixer::MAIN_GROUP, providers_[0]);
(...skipping 174 matching lines...) Expand 10 before | Expand all | Expand 10 after
273 EXPECT_EQ(old_ui_result_ids[0], 273 EXPECT_EQ(old_ui_result_ids[0],
274 TestSearchResult::GetInstanceId(ui_results.GetItemAt(3))); 274 TestSearchResult::GetInstanceId(ui_results.GetItemAt(3)));
275 EXPECT_EQ(old_ui_result_ids[1], 275 EXPECT_EQ(old_ui_result_ids[1],
276 TestSearchResult::GetInstanceId(ui_results.GetItemAt(0))); 276 TestSearchResult::GetInstanceId(ui_results.GetItemAt(0)));
277 EXPECT_EQ(old_ui_result_ids[2], 277 EXPECT_EQ(old_ui_result_ids[2],
278 TestSearchResult::GetInstanceId(ui_results.GetItemAt(2))); 278 TestSearchResult::GetInstanceId(ui_results.GetItemAt(2)));
279 } 279 }
280 280
281 } // namespace test 281 } // namespace test
282 } // namespace app_list 282 } // namespace app_list
OLDNEW
« no previous file with comments | « ui/app_list/search/history_data_store_unittest.cc ('k') | ui/app_list/test/app_list_test_view_delegate.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698