| Index: chrome/browser/history/in_memory_url_index.cc
|
| diff --git a/chrome/browser/history/in_memory_url_index.cc b/chrome/browser/history/in_memory_url_index.cc
|
| index 74ad2a07063e068a6fc88a59cdd1585f2e5f943d..dde249e670cbee57b7360ea65b7fd2b34c787e53 100644
|
| --- a/chrome/browser/history/in_memory_url_index.cc
|
| +++ b/chrome/browser/history/in_memory_url_index.cc
|
| @@ -18,9 +18,6 @@
|
| #include "components/bookmarks/browser/bookmark_model.h"
|
| #include "components/history/core/browser/url_database.h"
|
| #include "content/public/browser/browser_thread.h"
|
| -#include "content/public/browser/notification_details.h"
|
| -#include "content/public/browser/notification_service.h"
|
| -#include "content/public/browser/notification_source.h"
|
|
|
| using in_memory_url_index::InMemoryURLIndexCacheItem;
|
|
|
| @@ -106,11 +103,7 @@ InMemoryURLIndex::InMemoryURLIndex(Profile* profile,
|
| needs_to_be_cached_(false),
|
| history_service_observer_(this) {
|
| InitializeSchemeWhitelist(&scheme_whitelist_);
|
| - if (profile) {
|
| - // TODO(mrossetti): Register for language change notifications.
|
| - content::Source<Profile> source(profile);
|
| - registrar_.Add(this, chrome::NOTIFICATION_HISTORY_URLS_DELETED, source);
|
| - }
|
| + // TODO(mrossetti): Register for language change notifications.
|
| if (history_service_)
|
| history_service_observer_.Add(history_service_);
|
| }
|
| @@ -142,7 +135,6 @@ void InMemoryURLIndex::Init() {
|
|
|
| void InMemoryURLIndex::ShutDown() {
|
| history_service_observer_.RemoveAll();
|
| - registrar_.RemoveAll();
|
| cache_reader_tracker_.TryCancelAll();
|
| shutdown_ = true;
|
| base::FilePath path;
|
| @@ -184,21 +176,6 @@ void InMemoryURLIndex::DeleteURL(const GURL& url) {
|
| private_data_->DeleteURL(url);
|
| }
|
|
|
| -void InMemoryURLIndex::Observe(int notification_type,
|
| - const content::NotificationSource& source,
|
| - const content::NotificationDetails& details) {
|
| - switch (notification_type) {
|
| - case chrome::NOTIFICATION_HISTORY_URLS_DELETED:
|
| - OnURLsDeleted(
|
| - content::Details<history::URLsDeletedDetails>(details).ptr());
|
| - break;
|
| - default:
|
| - // For simplicity, the unit tests send us all notifications, even when
|
| - // we haven't registered for them, so don't assert here.
|
| - break;
|
| - }
|
| -}
|
| -
|
| void InMemoryURLIndex::OnURLVisited(HistoryService* history_service,
|
| ui::PageTransition transition,
|
| const URLRow& row,
|
| @@ -224,17 +201,14 @@ void InMemoryURLIndex::OnURLsModified(HistoryService* history_service,
|
| }
|
| }
|
|
|
| -void InMemoryURLIndex::OnHistoryServiceLoaded(HistoryService* history_service) {
|
| - ScheduleRebuildFromHistory();
|
| -}
|
| -
|
| -void InMemoryURLIndex::OnURLsDeleted(const URLsDeletedDetails* details) {
|
| - if (details->all_history) {
|
| +void InMemoryURLIndex::OnURLsDeleted(HistoryService* history_service,
|
| + const URLsDeletedDetails& details) {
|
| + if (details.all_history) {
|
| ClearPrivateData();
|
| needs_to_be_cached_ = true;
|
| } else {
|
| - for (URLRows::const_iterator row = details->rows.begin();
|
| - row != details->rows.end(); ++row)
|
| + for (URLRows::const_iterator row = details.rows.begin();
|
| + row != details.rows.end(); ++row)
|
| needs_to_be_cached_ |= private_data_->DeleteURL(row->url());
|
| }
|
| // If we made changes, destroy the previous cache. Otherwise, if we go
|
| @@ -258,6 +232,10 @@ void InMemoryURLIndex::OnURLsDeleted(const URLsDeletedDetails* details) {
|
| }
|
| }
|
|
|
| +void InMemoryURLIndex::OnHistoryServiceLoaded(HistoryService* history_service) {
|
| + ScheduleRebuildFromHistory();
|
| +}
|
| +
|
| // Restoring from Cache --------------------------------------------------------
|
|
|
| void InMemoryURLIndex::PostRestoreFromCacheFileTask() {
|
|
|