Eliminate sending NOTIFICATION_HISTORY_KEYWORD_SEARCH_TERM_UPDATED

chrome/browser/android/provider/chrome_browser_provider.h

 // Copyright (c) 2012 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_ANDROID_PROVIDER_CHROME_BROWSER_PROVIDER_H_
 #define CHROME_BROWSER_ANDROID_PROVIDER_CHROME_BROWSER_PROVIDER_H_
 
 #include "base/android/jni_weak_ref.h"
 #include "base/android/scoped_java_ref.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 167 matching lines @@
   virtual ~ChromeBrowserProvider();
 
   // Override bookmarks::BaseBookmarkModelObserver.
   virtual void BookmarkModelChanged() override;
   virtual void ExtensiveBookmarkChangesBeginning(BookmarkModel* model) override;
   virtual void ExtensiveBookmarkChangesEnded(BookmarkModel* model) override;
 
   // Deals with updates to the history service.
   void OnHistoryChanged();
 
+  void OnSearchTermUpdated();

[sdefresne, 2014/12/24 09:03:43]

style: comment this method

[Abhishek, 2014/12/24 19:39:40]

Done.

+
   // Override history::HistoryServiceObserver.
   virtual void OnURLVisited(HistoryService* history_service,

[sdefresne, 2014/12/24 09:03:43]

style: method should only have one of "override", "virtual", "final". Please
remove the "virtual" keyword for all method also marked as "override", keeping
just the "override" keyword (also applicable to other methods and files in that
change list).

[Abhishek, 2014/12/24 19:39:41]

Done.

                             ui::PageTransition transition,
                             const history::URLRow& row,
                             const history::RedirectList& redirects,
                             base::Time visit_time) override;
 

[sdefresne, 2014/12/24 09:03:43]

nit: remove empty line

[Abhishek, 2014/12/24 19:39:41]

Done.

+  virtual void HistoryKeywordSearchTermUpdated(
+      HistoryService* history_service,
+      const history::URLRow& row,
+      history::KeywordID keyword_id,
+      const base::string16& term) override;
+
   // Override content::NotificationObserver.
   virtual void Observe(int type,
                        const content::NotificationSource& source,
                        const content::NotificationDetails& details) override;
 
   JavaObjectWeakGlobalRef weak_java_provider_;
 
   // Profile must outlive this object.
   //
   // BookmarkModel, HistoryService and history::TopSites lifetime is bound to
@@ skipping 12 matching lines @@
   content::NotificationRegistrar notification_registrar_;
   ScopedObserver<HistoryService, HistoryServiceObserver>
       history_service_observer_;
 
   bool handling_extensive_changes_;
 
   DISALLOW_COPY_AND_ASSIGN(ChromeBrowserProvider);
 };
 
 #endif  // CHROME_BROWSER_ANDROID_PROVIDER_CHROME_BROWSER_PROVIDER_H_