Delete cookies for site when deleting locally stored data.

chrome/browser/browsing_data/cookies_tree_model.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_BROWSING_DATA_COOKIES_TREE_MODEL_H_
 #define CHROME_BROWSER_BROWSING_DATA_COOKIES_TREE_MODEL_H_
 
 // TODO(viettrungluu): This header file #includes far too much and has too much
 // inline code (which shouldn't be inline).
 
@@ skipping 701 matching lines @@
   void PopulateQuotaInfo(LocalDataContainer* container);
   void PopulateChannelIDInfo(LocalDataContainer* container);
   void PopulateServiceWorkerUsageInfo(LocalDataContainer* container);
   void PopulateFlashLSOInfo(LocalDataContainer* container);
 
   BrowsingDataCookieHelper* GetCookieHelper(const std::string& app_id);
   LocalDataContainer* data_container() {
     return data_container_.get();
   }
 
+  // Set the number of |batches_expected| this class should expect to receive.
+  // If |reset| is true, then this is a new set of batches, but if false, then
+  // this is a revised number (batches originally counted should no longer be
+  // expected).
+  void SetBatchExpectation(int batches_expected, bool reset);
+
  private:
   enum CookieIconIndex {
     ORIGIN = 0,
     COOKIE = 1,
     DATABASE = 2
   };
 
+  // Reset the counters for batches.
+  void ResetBatches();
+
+  // Record that one batch has been delivered.
+  void RecordBatchSeen();
+
+  // Record that one batch has begun processing. If this is the first batch then
+  // observers will be notified that batch processing has started.
   void NotifyObserverBeginBatch();
+
+  // Record that one batch has finished processing. If this is the last batch
+  // then observers will be notified that batch processing has ended.
   void NotifyObserverEndBatch();
 
+  // Notifies observers if expected batch count has been delievered and all
+  // batches have finished processing.
+  void MaybeNotifyBatchesEnded();
+
   void PopulateAppCacheInfoWithFilter(LocalDataContainer* container,
                                       ScopedBatchUpdateNotifier* notifier,
                                       const base::string16& filter);
   void PopulateCookieInfoWithFilter(LocalDataContainer* container,
                                     ScopedBatchUpdateNotifier* notifier,
                                     const base::string16& filter);
   void PopulateDatabaseInfoWithFilter(LocalDataContainer* container,
                                       ScopedBatchUpdateNotifier* notifier,
                                       const base::string16& filter);
   void PopulateLocalStorageInfoWithFilter(LocalDataContainer* container,
@@ skipping 33 matching lines @@
 #endif
 
   // The CookiesTreeModel maintains a separate list of observers that are
   // specifically of the type CookiesTreeModel::Observer.
   ObserverList<Observer> cookies_observer_list_;
 
   // If true, use the CanonicalCookie::Source attribute to group cookies.
   // Otherwise, use the CanonicalCookie::Domain attribute.
   bool group_by_cookie_source_;
 
-  // If this is non-zero, then this model is batching updates (there's a lot of
-  // notifications coming down the pipe). This is an integer is used to balance
-  // calls to Begin/EndBatch() if they're called in a nested manner.
-  int batch_update_;
+  // Keeps track of how many batches the consumer of this class says it is going
+  // to send.
+  int batches_expected_;
+
+  // Keeps track of how many batches we've seen.
+  int batches_seen_;
+
+  // Counts how many batches have started already. If this is non-zero and lower
+  // than batches_ended_, then this model is still batching updates.
+  int batches_started_;
+
+  // Counts how many batches have finished.
+  int batches_ended_;
 };
 
 #endif  // CHROME_BROWSER_BROWSING_DATA_COOKIES_TREE_MODEL_H_