Minor cleanup to UsageTracker.IsWorking().

Review URL: http://codereview.chromium.org/7634013

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@96583 0039d316-1c4b-4281-b951-d872f2087c98

webkit/quota/quota_manager.cc

@@ -1183,14 +1183,14 @@ void QuotaManager::DeleteOriginData(
 bool QuotaManager::ResetUsageTracker(StorageType type) {
   switch (type) {
     case kStorageTypeTemporary:
-      if (!temporary_usage_tracker_->IsWorking())
+      if (temporary_usage_tracker_->IsWorking())
         return false;
       temporary_usage_tracker_.reset(
           new UsageTracker(clients_, kStorageTypeTemporary,
                            special_storage_policy_));
       return true;
     case kStorageTypePersistent:
-      if (!persistent_usage_tracker_->IsWorking())
+      if (persistent_usage_tracker_->IsWorking())
         return false;
       persistent_usage_tracker_.reset(
           new UsageTracker(clients_, kStorageTypePersistent,

webkit/quota/quota_types.h

@@ -163,6 +163,10 @@ class CallbackQueueMapBase {
     return (callback_map_.find(key) != callback_map_.end());
   }
 
+  bool HasAnyCallbacks() const {
+    return !callback_map_.empty();
+  }
+
   iterator Begin() { return callback_map_.begin(); }
   iterator End() { return callback_map_.end(); }
 

webkit/quota/usage_tracker.cc

@@ -323,19 +323,6 @@ void UsageTracker::DidGetClientHostUsage(const std::string& host,
   }
 }
 
-bool UsageTracker::IsWorking() {
-  if (global_usage_.pending_clients > 0)
-    return false;
-  for (std::map<std::string, TrackingInfo>::iterator iter =
-           outstanding_host_usage_.begin();
-       iter != outstanding_host_usage_.end();
-       ++iter) {
-    if (iter->second.pending_clients > 0)
-      return false;
-  }
-  return true;
-}
-
 // ClientUsageTracker ----------------------------------------------------
 
 ClientUsageTracker::ClientUsageTracker(

webkit/quota/usage_tracker.h

@@ -42,9 +42,11 @@ class UsageTracker : public QuotaTaskObserver {
   void UpdateUsageCache(QuotaClient::ID client_id,
                         const GURL& origin,
                         int64 delta);
-
   void GetCachedOrigins(std::set<GURL>* origins) const;
-  bool IsWorking();
+  bool IsWorking() const {
+    return global_usage_callbacks_.HasCallbacks() ||
+           host_usage_callbacks_.HasAnyCallbacks();
+  }
 
  private:
   struct TrackingInfo {