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

Side by Side Diff: components/omnibox/autocomplete_result_unittest.cc

Issue 684513002: Standardize usage of virtual/override/final specifiers. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 "components/omnibox/autocomplete_result.h" 5 #include "components/omnibox/autocomplete_result.h"
6 6
7 #include <vector> 7 #include <vector>
8 8
9 #include "base/memory/scoped_ptr.h" 9 #include "base/memory/scoped_ptr.h"
10 #include "base/metrics/field_trial.h" 10 #include "base/metrics/field_trial.h"
(...skipping 70 matching lines...) Expand 10 before | Expand all | Expand 10 after
81 81
82 AutocompleteResultTest() { 82 AutocompleteResultTest() {
83 // Destroy the existing FieldTrialList before creating a new one to avoid 83 // Destroy the existing FieldTrialList before creating a new one to avoid
84 // a DCHECK. 84 // a DCHECK.
85 field_trial_list_.reset(); 85 field_trial_list_.reset();
86 field_trial_list_.reset(new base::FieldTrialList( 86 field_trial_list_.reset(new base::FieldTrialList(
87 new metrics::SHA1EntropyProvider("foo"))); 87 new metrics::SHA1EntropyProvider("foo")));
88 variations::testing::ClearAllVariationParams(); 88 variations::testing::ClearAllVariationParams();
89 } 89 }
90 90
91 virtual void SetUp() override { 91 void SetUp() override {
92 template_url_service_.reset(new TemplateURLService(NULL, 0)); 92 template_url_service_.reset(new TemplateURLService(NULL, 0));
93 template_url_service_->Load(); 93 template_url_service_->Load();
94 } 94 }
95 95
96 // Configures |match| from |data|. 96 // Configures |match| from |data|.
97 static void PopulateAutocompleteMatch(const TestData& data, 97 static void PopulateAutocompleteMatch(const TestData& data,
98 AutocompleteMatch* match); 98 AutocompleteMatch* match);
99 99
100 // Adds |count| AutocompleteMatches to |matches|. 100 // Adds |count| AutocompleteMatches to |matches|.
101 static void PopulateAutocompleteMatches(const TestData* data, 101 static void PopulateAutocompleteMatches(const TestData* data,
(...skipping 725 matching lines...) Expand 10 before | Expand all | Expand 10 after
827 result.Reset(); 827 result.Reset();
828 matches.clear(); 828 matches.clear();
829 829
830 // Case 5: Multiple verbatim matches found in AutocompleteResult. 830 // Case 5: Multiple verbatim matches found in AutocompleteResult.
831 PopulateAutocompleteMatchesFromTestData(kVerbatimMatches, 831 PopulateAutocompleteMatchesFromTestData(kVerbatimMatches,
832 arraysize(kVerbatimMatches), 832 arraysize(kVerbatimMatches),
833 &matches); 833 &matches);
834 result.AppendMatches(matches); 834 result.AppendMatches(matches);
835 EXPECT_FALSE(result.ShouldHideTopMatch()); 835 EXPECT_FALSE(result.ShouldHideTopMatch());
836 } 836 }
OLDNEW
« no previous file with comments | « components/network_time/network_time_tracker_unittest.cc ('k') | components/omnibox/base_search_provider_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698