Remove NOTIFICATION_HISTORY_URLS_MODIFIED

chrome/browser/history/android/android_provider_backend_unittest.cc

 // 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.
 
 #include "chrome/browser/history/android/android_provider_backend.h"
 
 #include <vector>
 
 #include "base/files/file_path.h"
 #include "base/files/file_util.h"
@@ skipping 51 matching lines @@
   bitmap.eraseColor(SK_ColorBLUE);
   return bitmap;
 }
 
 }  // namespace
 
 class AndroidProviderBackendDelegate : public HistoryBackend::Delegate {
  public:
   AndroidProviderBackendDelegate() {}
 
-  virtual void NotifyProfileError(sql::InitStatus init_status) override {}
-  virtual void SetInMemoryBackend(
+  void NotifyProfileError(sql::InitStatus init_status) override {}
+  void SetInMemoryBackend(
       scoped_ptr<InMemoryHistoryBackend> backend) override {}
-  virtual void NotifyAddVisit(const history::BriefVisitInfo& info) override {}
-  virtual void NotifyFaviconChanged(const std::set<GURL>& url) override {
+  void NotifyAddVisit(const history::BriefVisitInfo& info) override {}
+  void NotifyFaviconChanged(const std::set<GURL>& url) override {
     favicon_changed_.reset(new std::set<GURL>(url.begin(), url.end()));
   }
-  virtual void NotifyURLVisited(ui::PageTransition,
-                                const history::URLRow& row,
-                                const history::RedirectList& redirects,
-                                base::Time visit_time) override {}
-  virtual void BroadcastNotifications(
+  void NotifyURLVisited(ui::PageTransition,
+                        const history::URLRow& row,
+                        const history::RedirectList& redirects,
+                        base::Time visit_time) override {}
+  void NotifyURLsModified(const history::URLRows& rows) override {
+    modified_details_.reset(new history::URLRows(rows));
+  }
+  void BroadcastNotifications(
       int type,
       scoped_ptr<HistoryDetails> details) override {
-    switch (type) {
-      case chrome::NOTIFICATION_HISTORY_URLS_DELETED: {
-        scoped_ptr<URLsDeletedDetails> urls_deleted_details(
-            static_cast<URLsDeletedDetails*>(details.release()));
-        deleted_details_.reset(new history::URLRows(
-            urls_deleted_details->rows));
-        break;
-      }
-      case chrome::NOTIFICATION_HISTORY_URLS_MODIFIED: {
-        scoped_ptr<URLsModifiedDetails> urls_modified_details(
-            static_cast<URLsModifiedDetails*>(details.release()));
-        modified_details_.reset(new history::URLRows(
-            urls_modified_details->changed_urls));
-        break;
-      }
-    }
+    DCHECK_EQ(type, chrome::NOTIFICATION_HISTORY_URLS_DELETED);
+    scoped_ptr<URLsDeletedDetails> urls_deleted_details(
+        static_cast<URLsDeletedDetails*>(details.release()));
+    deleted_details_.reset(new history::URLRows(urls_deleted_details->rows));
   }
-  virtual void DBLoaded() override {}
+  void DBLoaded() override {}
 
   history::URLRows* deleted_details() const { return deleted_details_.get(); }
 
   history::URLRows* modified_details() const { return modified_details_.get(); }
 
   std::set<GURL>* favicon_changed() const { return favicon_changed_.get(); }
 
   void ResetDetails() {
     deleted_details_.reset();
     modified_details_.reset();
@@ skipping 53 matching lines @@
 class AndroidProviderBackendTest : public testing::Test {
  public:
   AndroidProviderBackendTest()
       : thumbnail_db_(NULL),
         profile_manager_(
           TestingBrowserProcess::GetGlobal()),
         bookmark_model_(NULL),
         ui_thread_(BrowserThread::UI, &message_loop_),
         file_thread_(BrowserThread::FILE, &message_loop_) {
   }
-  virtual ~AndroidProviderBackendTest() {}
+  ~AndroidProviderBackendTest() override {}
 
  protected:
-  virtual void SetUp() override {
+  void SetUp() override {
     // Setup the testing profile, so the bookmark_model_sql_handler could
     // get the bookmark model from it.
     ASSERT_TRUE(profile_manager_.SetUp());
     // It seems that the name has to be chrome::kInitialProfile, so it
     // could be found by ProfileManager::GetLastUsedProfile().
     TestingProfile* testing_profile = profile_manager_.CreateTestingProfile(
         chrome::kInitialProfile);
     testing_profile->CreateBookmarkModel(true);
     bookmark_model_ = BookmarkModelFactory::GetForProfile(testing_profile);
     history_client_ =
@@ skipping 1960 matching lines @@
                                                  update_args, &update_count));
   // Verify notifications.
   EXPECT_FALSE(notifier_.deleted_details());
   ASSERT_TRUE(notifier_.modified_details());
   ASSERT_EQ(1u, notifier_.modified_details()->size());
   // No favicon will be updated as thumbnail database is missing.
   EXPECT_FALSE(notifier_.favicon_changed());
 }
 
 }  // namespace history