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

Side by Side Diff: chrome/browser/extensions/api/omnibox/omnibox_api.h

Issue 12379095: Fix up unused includes from chrome/browser/extensions to the rest of chrome/browser. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 9 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 unified diff | Download patch | Annotate | Revision Log
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 #ifndef CHROME_BROWSER_EXTENSIONS_API_OMNIBOX_OMNIBOX_API_H_ 5 #ifndef CHROME_BROWSER_EXTENSIONS_API_OMNIBOX_OMNIBOX_API_H_
6 #define CHROME_BROWSER_EXTENSIONS_API_OMNIBOX_OMNIBOX_API_H_ 6 #define CHROME_BROWSER_EXTENSIONS_API_OMNIBOX_OMNIBOX_API_H_
7 7
8 #include <set> 8 #include <set>
9 #include <string> 9 #include <string>
10 #include <vector> 10 #include <vector>
(...skipping 112 matching lines...) Expand 10 before | Expand all | Expand 10 after
123 123
124 content::NotificationRegistrar registrar_; 124 content::NotificationRegistrar registrar_;
125 125
126 // Keeps track of favicon-sized omnibox icons for extensions. 126 // Keeps track of favicon-sized omnibox icons for extensions.
127 ExtensionIconManager omnibox_icon_manager_; 127 ExtensionIconManager omnibox_icon_manager_;
128 ExtensionIconManager omnibox_popup_icon_manager_; 128 ExtensionIconManager omnibox_popup_icon_manager_;
129 129
130 DISALLOW_COPY_AND_ASSIGN(OmniboxAPI); 130 DISALLOW_COPY_AND_ASSIGN(OmniboxAPI);
131 }; 131 };
132 132
133 template <>
134 void ProfileKeyedAPIFactory<OmniboxAPI>::DeclareFactoryDependencies();
Matt Perry 2013/03/05 02:34:15 Did you mean to include this in this CL?
Yoyo Zhou 2013/03/05 02:41:08 Yes - this moves the TemplateURLServiceFactory fro
135
133 class OmniboxSetDefaultSuggestionFunction : public SyncExtensionFunction { 136 class OmniboxSetDefaultSuggestionFunction : public SyncExtensionFunction {
134 public: 137 public:
135 DECLARE_EXTENSION_FUNCTION("omnibox.setDefaultSuggestion", 138 DECLARE_EXTENSION_FUNCTION("omnibox.setDefaultSuggestion",
136 OMNIBOX_SETDEFAULTSUGGESTION) 139 OMNIBOX_SETDEFAULTSUGGESTION)
137 140
138 protected: 141 protected:
139 virtual ~OmniboxSetDefaultSuggestionFunction() {} 142 virtual ~OmniboxSetDefaultSuggestionFunction() {}
140 143
141 // ExtensionFunction: 144 // ExtensionFunction:
142 virtual bool RunImpl() OVERRIDE; 145 virtual bool RunImpl() OVERRIDE;
(...skipping 41 matching lines...) Expand 10 before | Expand all | Expand 10 after
184 // omnibox.setDefaultSuggestion, apply that to |match|. Otherwise, do nothing. 187 // omnibox.setDefaultSuggestion, apply that to |match|. Otherwise, do nothing.
185 void ApplyDefaultSuggestionForExtensionKeyword( 188 void ApplyDefaultSuggestionForExtensionKeyword(
186 Profile* profile, 189 Profile* profile,
187 const TemplateURL* keyword, 190 const TemplateURL* keyword,
188 const string16& remaining_input, 191 const string16& remaining_input,
189 AutocompleteMatch* match); 192 AutocompleteMatch* match);
190 193
191 } // namespace extensions 194 } // namespace extensions
192 195
193 #endif // CHROME_BROWSER_EXTENSIONS_API_OMNIBOX_OMNIBOX_API_H_ 196 #endif // CHROME_BROWSER_EXTENSIONS_API_OMNIBOX_OMNIBOX_API_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698