Refactor HistoryService to not send NOTIFICATION_FAVICON_CHANGED

chrome/browser/history/android/android_provider_backend.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_HISTORY_ANDROID_ANDROID_PROVIDER_BACKEND_H_
 #define CHROME_BROWSER_HISTORY_ANDROID_ANDROID_PROVIDER_BACKEND_H_
 
+#include <list>
 #include <set>
 
 #include "base/containers/hash_tables.h"
 #include "base/files/file_path.h"
 #include "base/gtest_prod_util.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/scoped_vector.h"
 #include "chrome/browser/history/android/android_cache_database.h"
 #include "chrome/browser/history/android/android_history_types.h"
@@ skipping 121 matching lines @@
   bool DeleteSearchTerms(const std::string& selection,
                          const std::vector<base::string16>& selection_args,
                          int * deleted_count);
 
  private:
   friend class AndroidProviderBackendTest;
 
   FRIEND_TEST_ALL_PREFIXES(AndroidProviderBackendTest, UpdateTables);
   FRIEND_TEST_ALL_PREFIXES(AndroidProviderBackendTest, UpdateSearchTermTable);
 
-  class HistoryNotifications {
-   public:
-    HistoryNotifications();
-    ~HistoryNotifications();
-
-    void PushBack(int type, scoped_ptr<HistoryDetails> detail);
-    int PopBackType();
-    scoped_ptr<HistoryDetails> PopBackDetails();
-
-    bool empty() const { return types_.empty(); }
-
-   private:
-    std::vector<int> types_;
-    ScopedVector<HistoryDetails> details_;
-
-    DISALLOW_COPY_AND_ASSIGN(HistoryNotifications);
-  };
+  typedef std::list<base::Closure> HistoryNotifications;
 
   // The scoped transaction for AndroidProviderBackend.
   //
   // The new transactions are started automatically in both history and
   // thumbnail database and could be a nesting transaction, if so, rolling back
   // of this transaction will cause the exsting and subsequent nesting
   // transactions failed.
   //
   // Commit() is used to commit the transaction, otherwise the transaction will
   // be rolled back when the object is out of scope. This transaction could
@@ skipping 180 matching lines @@
   bool initialized_;
 
   HistoryBackend::Delegate* delegate_;
 
   DISALLOW_COPY_AND_ASSIGN(AndroidProviderBackend);
 };
 
 }  // namespace history
 
 #endif  // CHROME_BROWSER_HISTORY_ANDROID_ANDROID_PROVIDER_BACKEND_H_