Adding BrowsingDataPartitionClient

ios/web/browsing_data_partition_impl.mm

 // Copyright 2015 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.
 
 #import "ios/web/browsing_data_partition_impl.h"
 
 #include "base/logging.h"
 #include "ios/web/public/browser_state.h"
+#include "ios/web/public/browsing_data_partition_client.h"
 #import "ios/web/public/crw_browsing_data_store.h"
 #include "ios/web/public/web_thread.h"
 
 // A class that observes the |mode| changes in a CRWBrowsingDataStore using KVO.
 // Maintains a count of all the CRWBrowsingDataStores whose mode is out of sync
 // with their corresponding ActiveStateManager's active state.
 @interface CRWBrowsingDataStoreModeObserver : NSObject
 
 // The total count of the CRWBrowsingDataStores -- that are being observed --
 // whose mode is out of sync with their associated ActiveStateManager's active
@@ skipping 58 matching lines @@
   // Check if the |browsingDataStore|'s associated ActiveStateManager's active
   // state is still out of sync.
   if (activeState && browsingDataStore.mode == web::INACTIVE) {
     [browsingDataStore makeActiveWithCompletionHandler:nil];
   } else if (!activeState && browsingDataStore.mode == web::ACTIVE) {
     [browsingDataStore makeInactiveWithCompletionHandler:nil];
   }
 
   DCHECK_GE(self.outOfSyncStoreCount, 1U);
   --self.outOfSyncStoreCount;
-  // TODO(shreyasv): Have a BrowsingDataPartitionClient be informed when
-  // |self.outOfSyncStoreCount| goes down to 0. crbug.com/480654.
+  web::BrowsingDataPartitionClient* client =

[Eugene But (OOO till 7-30), 2015/07/21 21:48:03]

NIT: Please either use auto or inline web::GetBrowsingDataPartitionClient()
call. This linebreak does not look very well. I would personally go with
inlining web::GetBrowsingDataPartitionClient() call, but it's up to you.

[shreyasv1, 2015/07/21 22:12:06]

Acknowledged.

+      web::GetBrowsingDataPartitionClient();
+  if (client) {
+    client->DidBecomeSynchronized();
+  }
 }
 
 @end
 
 namespace web {
 
 namespace {
 // The global observer that tracks the number of CRWBrowsingDataStores whose
 // modes are out of sync with their associated ActiveStateManager's active
 // state.
@@ skipping 56 matching lines @@
 }
 
 void BrowsingDataPartitionImpl::WillBeDestroyed() {
   DCHECK_CURRENTLY_ON_WEB_THREAD(WebThread::UI);
 
   active_state_manager_->RemoveObserver(this);
   active_state_manager_ = nullptr;
 }
 
 }  // namespace web