Eliminate sending NOTIFICATION_HISTORY_KEYWORD_SEARCH_TERM_DELETED

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 177 matching lines @@
   // Override history::HistoryServiceObserver.
   void OnURLVisited(HistoryService* history_service,
                     ui::PageTransition transition,
                     const history::URLRow& row,
                     const history::RedirectList& redirects,
                     base::Time visit_time) override;
   void OnKeywordSearchTermUpdated(HistoryService* history_service,
                                   const history::URLRow& row,
                                   history::KeywordID keyword_id,
                                   const base::string16& term) override;
+  void OnKeywordSearchTermDeleted(HistoryService* history_service,
+                                  history::URLID url_id) override{};

[newt (away), 2015/01/16 18:58:03]

space before {

[sdefresne, 2015/01/17 18:23:23]

Nit: no ; after the {} too, and could you not inline this method but put it in
the implementation file?

[Abhishek, 2015/01/19 06:13:01]

Done.

[Abhishek, 2015/01/19 06:13:01]

Done.

 
   // 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.
   //
@@ skipping 13 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_