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

Unified Diff: chrome/browser/search_engines/chrome_template_url_service_client.cc

Issue 367863005: No chrome dependencies in TemplateURLService (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rename to TemplateURLServiceClient Created 6 years, 6 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/search_engines/chrome_template_url_service_client.cc
diff --git a/chrome/browser/search_engines/chrome_template_url_service_client.cc b/chrome/browser/search_engines/chrome_template_url_service_client.cc
new file mode 100644
index 0000000000000000000000000000000000000000..0183636a212c18164a45065bbf4bbe3fee5f6b5a
--- /dev/null
+++ b/chrome/browser/search_engines/chrome_template_url_service_client.cc
@@ -0,0 +1,82 @@
+// Copyright 2014 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "chrome/browser/search_engines/chrome_template_url_service_client.h"
+
+#include "chrome/browser/chrome_notification_types.h"
+#include "chrome/browser/history/history_service.h"
+#include "chrome/browser/history/history_service_factory.h"
+#include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/search_engines/template_url_service.h"
+#include "content/public/browser/notification_details.h"
+#include "content/public/browser/notification_source.h"
+
+ChromeTemplateURLServiceClient::ChromeTemplateURLServiceClient(Profile* profile)
+ : profile_(profile),
+ owner_(NULL) {
+ DCHECK(profile);
+
+ // Register for notifications.
+ // TODO(sky): bug 1166191. The keywords should be moved into the history
+ // db, which will mean we no longer need this notification and the history
+ // backend can handle automatically adding the search terms as the user
+ // navigates.
+ content::Source<Profile> profile_source(profile->GetOriginalProfile());
+ notification_registrar_.Add(this, chrome::NOTIFICATION_HISTORY_URL_VISITED,
+ profile_source);
+}
+
+ChromeTemplateURLServiceClient::~ChromeTemplateURLServiceClient() {
+}
+
+void ChromeTemplateURLServiceClient::SetOwner(TemplateURLService* owner) {
+ DCHECK(!owner_);
+ owner_ = owner;
+}
+
+void ChromeTemplateURLServiceClient::DeleteAllSearchTermsForKeyword(
+ TemplateURLID id) {
+ HistoryService* history_service =
+ HistoryServiceFactory::GetForProfile(profile_, Profile::EXPLICIT_ACCESS);
+ if (history_service)
+ history_service->DeleteAllSearchTermsForKeyword(id);
+}
+
+void ChromeTemplateURLServiceClient::SetKeywordSearchTermsForURL(
+ const GURL& url,
+ TemplateURLID id,
+ const base::string16& term) {
+ HistoryService* history_service =
+ HistoryServiceFactory::GetForProfile(profile_, Profile::EXPLICIT_ACCESS);
+ if (history_service)
+ history_service->SetKeywordSearchTermsForURL(url, id, term);
+}
+
+void ChromeTemplateURLServiceClient::AddKeywordGeneratedVisit(const GURL& url) {
+ HistoryService* history_service =
+ HistoryServiceFactory::GetForProfile(profile_, Profile::EXPLICIT_ACCESS);
+ if (history_service)
+ history_service->AddPage(url, base::Time::Now(), NULL, 0, GURL(),
+ history::RedirectList(),
+ content::PAGE_TRANSITION_KEYWORD_GENERATED,
+ history::SOURCE_BROWSED, false);
+}
+
+void ChromeTemplateURLServiceClient::Observe(
+ int type,
+ const content::NotificationSource& source,
+ const content::NotificationDetails& details) {
+ DCHECK_EQ(type, chrome::NOTIFICATION_HISTORY_URL_VISITED);
+
+ if (!owner_)
+ return;
+
+ content::Details<history::URLVisitedDetails> history_details(details);
+ TemplateURLService::URLVisitedDetails visited_details;
+ visited_details.url = history_details->row.url();
+ visited_details.is_keyword_transition =
+ content::PageTransitionStripQualifier(history_details->transition) ==
+ content::PAGE_TRANSITION_KEYWORD;
+ owner_->OnHistoryURLVisited(visited_details);
+}

Powered by Google App Engine
This is Rietveld 408576698