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

Unified Diff: chrome/browser/ui/app_list/search/omnibox_provider.cc

Issue 621823004: Simplifies the structure of app_list search a bit. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 3 months 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/app_list/search/omnibox_provider.cc
diff --git a/chrome/browser/ui/app_list/search/omnibox_provider.cc b/chrome/browser/ui/app_list/search/omnibox_provider.cc
index 483bddde53ceac5d07917c138317500576208a8e..25fcfc92bea69bfc7de4c5ae8c1ba099b8304640 100644
--- a/chrome/browser/ui/app_list/search/omnibox_provider.cc
+++ b/chrome/browser/ui/app_list/search/omnibox_provider.cc
@@ -9,13 +9,13 @@
#include "chrome/browser/autocomplete/chrome_autocomplete_scheme_classifier.h"
#include "chrome/browser/bookmarks/bookmark_model_factory.h"
#include "chrome/browser/search_engines/template_url_service_factory.h"
-#include "chrome/browser/ui/app_list/search/chrome_search_result.h"
#include "chrome/browser/ui/browser_navigator.h"
#include "components/bookmarks/browser/bookmark_model.h"
#include "components/metrics/proto/omnibox_event.pb.h"
#include "components/omnibox/autocomplete_input.h"
#include "components/omnibox/autocomplete_match.h"
#include "grit/theme_resources.h"
+#include "ui/app_list/search_result.h"
#include "ui/base/resource/resource_bundle.h"
namespace app_list {
@@ -65,7 +65,7 @@ void ACMatchClassificationsToTags(
}
}
-class OmniboxResult : public ChromeSearchResult {
+class OmniboxResult : public SearchResult {
public:
OmniboxResult(Profile* profile,
AutocompleteController* autocomplete_controller,
@@ -90,7 +90,7 @@ class OmniboxResult : public ChromeSearchResult {
}
virtual ~OmniboxResult() {}
- // ChromeSearchResult overides:
+ // SearchResult overides:
virtual void Open(int event_flags) OVERRIDE {
chrome::NavigateParams params(profile_,
match_.destination_url,
@@ -99,14 +99,12 @@ class OmniboxResult : public ChromeSearchResult {
chrome::Navigate(&params);
}
- virtual void InvokeAction(int action_index, int event_flags) OVERRIDE {}
-
- virtual scoped_ptr<ChromeSearchResult> Duplicate() OVERRIDE {
- return scoped_ptr<ChromeSearchResult>(
+ virtual scoped_ptr<SearchResult> Duplicate() OVERRIDE {
+ return scoped_ptr<SearchResult>(
new OmniboxResult(profile_, autocomplete_controller_, match_)).Pass();
}
- virtual ChromeSearchResultType GetType() OVERRIDE {
+ virtual SearchResultType GetType() OVERRIDE {
return OMNIBOX_SEARCH_RESULT;
}

Powered by Google App Engine
This is Rietveld 408576698