Quota: Minor clean up for QuotaTask.

Replace deprecated DISALLOW_COPY_AND_ASSIGN macro with deleted copy
constructor and assignment operator on the base class.

Change-Id: Id6ce85518a1db717784a9a6d5f38ea2f191a1d30
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2590691Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Commit-Queue: Jarryd Goodman <jarrydg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#837131}

storage/browser/quota/quota_manager.cc

@@ -386,7 +386,6 @@ class QuotaManager::UsageAndQuotaInfoGatherer : public QuotaTask {
 
   // Weak pointers are used to support cancelling work.
   base::WeakPtrFactory<UsageAndQuotaInfoGatherer> weak_factory_{this};
-  DISALLOW_COPY_AND_ASSIGN(UsageAndQuotaInfoGatherer);
 };
 
 class QuotaManager::EvictionRoundInfoHelper : public QuotaTask {
@@ -475,7 +474,6 @@ class QuotaManager::EvictionRoundInfoHelper : public QuotaTask {
   int64_t global_usage_ = 0;
   bool global_usage_is_complete_ = false;
   base::WeakPtrFactory<EvictionRoundInfoHelper> weak_factory_{this};
-  DISALLOW_COPY_AND_ASSIGN(EvictionRoundInfoHelper);
 };
 
 class QuotaManager::GetUsageInfoTask : public QuotaTask {
@@ -539,7 +537,6 @@ class QuotaManager::GetUsageInfoTask : public QuotaTask {
   int remaining_trackers_;
   base::WeakPtrFactory<GetUsageInfoTask> weak_factory_{this};
 
-  DISALLOW_COPY_AND_ASSIGN(GetUsageInfoTask);
 };
 
 class QuotaManager::OriginDataDeleter : public QuotaTask {
@@ -633,7 +630,6 @@ class QuotaManager::OriginDataDeleter : public QuotaTask {
   StatusCallback callback_;
 
   base::WeakPtrFactory<OriginDataDeleter> weak_factory_{this};
-  DISALLOW_COPY_AND_ASSIGN(OriginDataDeleter);
 };
 
 class QuotaManager::HostDataDeleter : public QuotaTask {
@@ -730,7 +726,6 @@ class QuotaManager::HostDataDeleter : public QuotaTask {
   StatusCallback callback_;
 
   base::WeakPtrFactory<HostDataDeleter> weak_factory_{this};
-  DISALLOW_COPY_AND_ASSIGN(HostDataDeleter);
 };
 
 class QuotaManager::StorageCleanupHelper : public QuotaTask {
@@ -788,7 +783,6 @@ class QuotaManager::StorageCleanupHelper : public QuotaTask {
   const QuotaClientTypes quota_client_types_;
   base::OnceClosure callback_;
   base::WeakPtrFactory<StorageCleanupHelper> weak_factory_{this};
-  DISALLOW_COPY_AND_ASSIGN(StorageCleanupHelper);
 };
 
 // Fetch origins that have been modified since the specified time. This is used

storage/browser/quota/quota_task.h

@@ -30,6 +30,9 @@ class QuotaTaskObserver;
 // TODO(kinuko): Revise this using base::OnceCallback.
 class QuotaTask {
  public:
+  QuotaTask(const QuotaTask&) = delete;
+  QuotaTask& operator=(const QuotaTask&) = delete;
+
   void Start();
 
  protected: