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

Unified Diff: chrome/browser/autocomplete/shortcuts_provider.cc

Issue 7327007: Moving notification types which are chrome specific to a new header file chrome_notification_type... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 9 years, 5 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
« no previous file with comments | « chrome/browser/autocomplete/shortcuts_provider.h ('k') | chrome/browser/autofill/autofill_browsertest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/autocomplete/shortcuts_provider.cc
===================================================================
--- chrome/browser/autocomplete/shortcuts_provider.cc (revision 91968)
+++ chrome/browser/autocomplete/shortcuts_provider.cc (working copy)
@@ -22,11 +22,11 @@
#include "chrome/browser/net/url_fixer_upper.h"
#include "chrome/browser/prefs/pref_service.h"
#include "chrome/browser/profiles/profile.h"
+#include "chrome/common/chrome_notification_types.h"
#include "chrome/common/pref_names.h"
#include "chrome/common/url_constants.h"
#include "content/common/notification_details.h"
#include "content/common/notification_source.h"
-#include "content/common/notification_type.h"
#include "googleurl/src/url_parse.h"
#include "googleurl/src/url_util.h"
#include "net/base/escape.h"
@@ -81,9 +81,9 @@
Profile* profile)
: AutocompleteProvider(listener, profile, "ShortcutsProvider"),
languages_(profile_->GetPrefs()->GetString(prefs::kAcceptLanguages)) {
- notification_registrar_.Add(this, NotificationType::OMNIBOX_OPENED_URL,
+ notification_registrar_.Add(this, chrome::NOTIFICATION_OMNIBOX_OPENED_URL,
Source<Profile>(profile));
- notification_registrar_.Add(this, NotificationType::HISTORY_URLS_DELETED,
+ notification_registrar_.Add(this, chrome::NOTIFICATION_HISTORY_URLS_DELETED,
Source<Profile>(profile));
}
@@ -129,17 +129,17 @@
history_service->DeleteURL(match.destination_url);
}
-void ShortcutsProvider::Observe(NotificationType type,
+void ShortcutsProvider::Observe(int type,
const NotificationSource& source,
const NotificationDetails& details) {
- if (type == NotificationType::HISTORY_URLS_DELETED) {
+ if (type == chrome::NOTIFICATION_HISTORY_URLS_DELETED) {
const std::set<GURL>& urls =
Details<const history::URLsDeletedDetails>(details)->urls;
DeleteShortcutsWithURLs(urls);
return;
}
- DCHECK(type == NotificationType::OMNIBOX_OPENED_URL);
+ DCHECK(type == chrome::NOTIFICATION_OMNIBOX_OPENED_URL);
AutocompleteLog* log = Details<AutocompleteLog>(details).ptr();
string16 text_lowercase(base::i18n::ToLower(log->text));
« no previous file with comments | « chrome/browser/autocomplete/shortcuts_provider.h ('k') | chrome/browser/autofill/autofill_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698