Remove NOTIFICATION_HISTORY_URL_VISITED

chrome/browser/history/chrome_history_client.cc

 // 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/history/chrome_history_client.h"
 
 #include "base/logging.h"
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/history/history_notifications.h"
 #include "chrome/browser/history/history_service.h"
@@ skipping 75 matching lines @@
   // it can release the signal history is waiting on, allowing history to
   // shutdown (HistoryService::Cleanup to complete). In such a scenario history
   // sees an incorrect view of bookmarks, but it's better than a deadlock.
   bookmark_model_->Shutdown();
   if (history_service_) {
     history_service_->RemoveObserver(this);
     history_service_ = nullptr;
   }
 }
 
-// TODO(sdefresne): port client listening for NOTIFICATION_HISTORY_URL_VISITED
-// to use the HistoryService::Observer pattern instead and remove this method
-// when this is complete, http://crbug.com/42178
-void ChromeHistoryClient::OnURLVisited(HistoryService* history_service,
-                                       ui::PageTransition transition,
-                                       const history::URLRow& row,
-                                       const history::RedirectList& redirects,
-                                       base::Time visit_time) {
-  DCHECK(history_service == history_service_);
-  scoped_ptr<history::URLVisitedDetails> details(
-      new history::URLVisitedDetails);
-  details->transition = transition;
-  details->row = row;
-  details->redirects = redirects;
-  details->visit_time = visit_time;
-  content::NotificationService::current()->Notify(
-      chrome::NOTIFICATION_HISTORY_URL_VISITED,
-      content::Source<Profile>(profile_),
-      content::Details<history::HistoryDetails>(details.get()));
-}
-
 void ChromeHistoryClient::TopSitesLoaded(history::TopSites* top_sites) {
   content::NotificationService::current()->Notify(
       chrome::NOTIFICATION_TOP_SITES_LOADED,
       content::Source<Profile>(profile_),
       content::Details<history::TopSites>(top_sites));
 }
 
 void ChromeHistoryClient::TopSitesChanged(history::TopSites* top_sites) {
   content::NotificationService::current()->Notify(
       chrome::NOTIFICATION_TOP_SITES_CHANGED,
       content::Source<history::TopSites>(top_sites),
       content::NotificationService::NoDetails());
 }