No chrome dependencies in TemplateURLService

chrome/browser/search_engines/keyword_history_service.h

Index: chrome/browser/search_engines/keyword_history_service.h
diff --git a/chrome/browser/search_engines/keyword_history_service.h b/chrome/browser/search_engines/keyword_history_service.h
new file mode 100644
index 0000000000000000000000000000000000000000..eaf0239175b8d1b16f33b05cc2e6446e406ba62c
--- /dev/null
+++ b/chrome/browser/search_engines/keyword_history_service.h
@@ -0,0 +1,44 @@
+// 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.
+
+#ifndef CHROME_BROWSER_SEARCH_ENGINES_KEYWORD_HISTORY_SERVICE_H_
+#define CHROME_BROWSER_SEARCH_ENGINES_KEYWORD_HISTORY_SERVICE_H_
+
+#include "chrome/browser/search_engines/template_url_service.h"
+#include "content/public/browser/notification_observer.h"
+#include "content/public/browser/notification_registrar.h"
+
+class HistoryService;
+class Profile;
+
+// KeywordHistoryService provides keyword related history functionalities.

[Peter Kasting, 2014/07/02 20:45:56]

Nit: functionalities -> functionality

[hashimoto, 2014/07/03 00:05:07]

Done.

+class KeywordHistoryService : public KeywordHistoryServiceInterface,
+                              public content::NotificationObserver {
+ public:
+  explicit KeywordHistoryService(Profile* profile);
+  virtual ~KeywordHistoryService();
+
+  // KeywordHistoryServiceInterface overrides:

[Peter Kasting, 2014/07/02 20:45:56]

Nit: No need for "overrides" here

[hashimoto, 2014/07/03 00:05:07]

Done.

+  virtual void SetOwner(TemplateURLService* owner) OVERRIDE;
+  virtual void DeleteAllSearchTermsForKeyword(
+      history::KeywordID keyword_Id) OVERRIDE;
+  virtual void SetKeywordSearchTermsForURL(const GURL& url,
+                                           TemplateURLID id,
+                                           const base::string16& term) OVERRIDE;
+  virtual void AddKeywordGeneratedVisit(const GURL& url) OVERRIDE;
+
+  // content::NotificationObserver implementation.

[Peter Kasting, 2014/07/02 20:45:56]

Nit: " implementation." -> ":"

[hashimoto, 2014/07/03 00:05:07]

Done.

+  virtual void Observe(int type,
+                       const content::NotificationSource& source,
+                       const content::NotificationDetails& details) OVERRIDE;
+
+ private:
+  Profile* profile_;
+  TemplateURLService* owner_;
+  content::NotificationRegistrar notification_registrar_;
+
+  DISALLOW_COPY_AND_ASSIGN(KeywordHistoryService);
+};
+
+#endif  // CHROME_BROWSER_SEARCH_ENGINES_KEYWORD_HISTORY_SERVICE_H_