Replace ScopedObserver with base::ScopedObservation in...

Replace ScopedObserver with base::ScopedObservation in /content/browser/storage_partition_impl_unittest.cc.

ScopedObserver is being deprecated in favor of two new classes:
- base::ScopedObservation for observers that only ever observe
  a single source.
- base::ScopedMultiSourceObservation for observers that do or may
  observe more than a single source.
This CL was uploaded by git cl split.

R=kinuko@chromium.org

Change-Id: I87a97adbc8cd1ad3e9bb6131289df74f9afb691a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2526167
Auto-Submit: Sigurður Ásgeirsson <siggi@chromium.org>
Commit-Queue: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Cr-Commit-Position: refs/heads/master@{#825735}

content/browser/storage_partition_impl_unittest.cc

@@ -15,6 +15,7 @@
 #include "base/files/file_util.h"
 #include "base/location.h"
 #include "base/run_loop.h"
+#include "base/scoped_observation.h"
 #include "base/single_thread_task_runner.h"
 #include "base/stl_util.h"
 #include "base/test/bind.h"
@@ -593,7 +594,7 @@ class RemovePluginPrivateDataTester {
 class MockDataRemovalObserver : public StoragePartition::DataRemovalObserver {
  public:
   explicit MockDataRemovalObserver(StoragePartition* partition) {
-    observer_.Add(partition);
+    observation_.Observe(partition);
   }
 
   MOCK_METHOD4(OnOriginDataCleared,
@@ -603,8 +604,9 @@ class MockDataRemovalObserver : public StoragePartition::DataRemovalObserver {
                     base::Time));
 
  private:
-  ScopedObserver<StoragePartition, StoragePartition::DataRemovalObserver>
-      observer_{this};
+  base::ScopedObservation<StoragePartition,
+                          StoragePartition::DataRemovalObserver>
+      observation_{this};
 };
 
 bool IsWebSafeSchemeForTest(const std::string& scheme) {