Remove NOTIFICATION_HISTORY_URL_VISITED

chrome/browser/search_engines/chrome_template_url_service_client.h

 // 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_CHROME_TEMPLATE_URL_SERVICE_CLIENT_H_
 #define CHROME_BROWSER_SEARCH_ENGINES_CHROME_TEMPLATE_URL_SERVICE_CLIENT_H_
 
+#include "components/history/core/browser/history_service_observer.h"
 #include "components/search_engines/template_url_service_client.h"
 #include "content/public/browser/notification_observer.h"
 #include "content/public/browser/notification_registrar.h"

[droger, 2014/10/14 15:15:24]

Remove these includes.

[sdefresne, 2014/10/14 16:02:05]

Done.

 
 class HistoryService;
 class Profile;

[droger, 2014/10/14 15:15:24]

Remove.

[sdefresne, 2014/10/14 16:02:05]

Done.

 
 // ChromeTemplateURLServiceClient provides keyword related history
 // functionality for TemplateURLService.
 class ChromeTemplateURLServiceClient : public TemplateURLServiceClient,
-                                       public content::NotificationObserver {
+                                       public history::HistoryServiceObserver {
  public:
-  explicit ChromeTemplateURLServiceClient(Profile* profile);
+  explicit ChromeTemplateURLServiceClient(HistoryService* history_service);
   virtual ~ChromeTemplateURLServiceClient();
 
   // TemplateURLServiceClient:
   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;
   virtual void RestoreExtensionInfoIfNecessary(
       TemplateURL* template_url) override;
 
-  // content::NotificationObserver:
-  virtual void Observe(int type,
-                       const content::NotificationSource& source,
-                       const content::NotificationDetails& details) override;
+  // history::HistoryServiceObserver:
+  virtual void OnURLVisited(HistoryService* history_service,
+                            ui::PageTransition transition,
+                            const history::URLRow& row,
+                            const history::RedirectList& redirects,
+                            base::Time visit_time) override;

[droger, 2014/10/14 15:15:24]

You need includes or forward declarations for these types.

[sdefresne, 2014/10/14 16:02:05]

They are forward-declared / included in
"components/history/core/browser/history_service_observer.h". Do I really need
to re-foward-declare / include them?

[droger, 2014/10/14 16:11:09]

Ok, you are probably right.

 
  private:
-  Profile* profile_;
   TemplateURLService* owner_;
-  content::NotificationRegistrar notification_registrar_;
+  HistoryService* history_service_;
 
   DISALLOW_COPY_AND_ASSIGN(ChromeTemplateURLServiceClient);
 };
 
 #endif  // CHROME_BROWSER_SEARCH_ENGINES_CHROME_TEMPLATE_URL_SERVICE_CLIENT_H_