Index: chrome/browser/extensions/api/history/history_api.cc |
diff --git a/chrome/browser/extensions/api/history/history_api.cc b/chrome/browser/extensions/api/history/history_api.cc |
index 37ca08ed639d707322bd19121b27153073e4989d..ec1c74876b7ace5978972f3e574891097ca2efc8 100644 |
--- a/chrome/browser/extensions/api/history/history_api.cc |
+++ b/chrome/browser/extensions/api/history/history_api.cc |
@@ -18,7 +18,6 @@ |
#include "base/task/cancelable_task_tracker.h" |
#include "base/time/time.h" |
#include "base/values.h" |
-#include "chrome/browser/chrome_notification_types.h" |
#include "chrome/browser/extensions/activity_log/activity_log.h" |
#include "chrome/browser/history/history_service.h" |
#include "chrome/browser/history/history_service_factory.h" |
@@ -27,8 +26,6 @@ |
#include "chrome/common/extensions/api/history.h" |
#include "chrome/common/pref_names.h" |
#include "components/history/core/browser/history_types.h" |
-#include "content/public/browser/notification_details.h" |
-#include "content/public/browser/notification_source.h" |
#include "extensions/browser/event_router.h" |
#include "extensions/browser/extension_system_provider.h" |
#include "extensions/browser/extensions_browser_client.h" |
@@ -135,24 +132,12 @@ HistoryEventRouter::HistoryEventRouter(Profile* profile, |
HistoryService* history_service) |
: profile_(profile), history_service_observer_(this) { |
DCHECK(profile); |
- registrar_.Add(this, |
- chrome::NOTIFICATION_HISTORY_URLS_DELETED, |
- content::Source<Profile>(profile)); |
history_service_observer_.Add(history_service); |
} |
HistoryEventRouter::~HistoryEventRouter() { |
} |
-void HistoryEventRouter::Observe(int type, |
- const content::NotificationSource& source, |
- const content::NotificationDetails& details) { |
- DCHECK_EQ(type, chrome::NOTIFICATION_HISTORY_URLS_DELETED); |
- HistoryUrlsRemoved( |
- content::Source<Profile>(source).ptr(), |
- content::Details<const history::URLsDeletedDetails>(details).ptr()); |
-} |
- |
void HistoryEventRouter::OnURLVisited(HistoryService* history_service, |
ui::PageTransition transition, |
const history::URLRow& row, |
@@ -163,21 +148,22 @@ void HistoryEventRouter::OnURLVisited(HistoryService* history_service, |
DispatchEvent(profile_, api::history::OnVisited::kEventName, args.Pass()); |
} |
-void HistoryEventRouter::HistoryUrlsRemoved( |
- Profile* profile, |
- const history::URLsDeletedDetails* details) { |
+void HistoryEventRouter::OnURLsDeleted(HistoryService* history_service, |
+ bool all_history, |
+ bool expired, |
+ const history::URLRows& deleted_rows, |
+ const std::set<GURL>& favicon_urls) { |
OnVisitRemoved::Removed removed; |
- removed.all_history = details->all_history; |
+ removed.all_history = all_history; |
std::vector<std::string>* urls = new std::vector<std::string>(); |
- for (history::URLRows::const_iterator iterator = details->rows.begin(); |
- iterator != details->rows.end(); ++iterator) { |
- urls->push_back(iterator->url().spec()); |
- } |
+ for (const auto& row : deleted_rows) |
+ urls->push_back(row.url().spec()); |
removed.urls.reset(urls); |
scoped_ptr<base::ListValue> args = OnVisitRemoved::Create(removed); |
- DispatchEvent(profile, api::history::OnVisitRemoved::kEventName, args.Pass()); |
+ DispatchEvent(profile_, api::history::OnVisitRemoved::kEventName, |
+ args.Pass()); |
} |
void HistoryEventRouter::DispatchEvent( |