[scheduling] Use ScopedTaskEnvironment instead of MessageLoop in data_use_measurement

MessageLoop will go away, eventually.

BUG=891670

Change-Id: I85b84b92eb544b02920139aad88a4fb9a31d830f
Reviewed-on: https://chromium-review.googlesource.com/c/1352418Reviewed-by: Scott Little <sclittle@chromium.org>
Commit-Queue: Carlos Caballero <carlscab@google.com>
Cr-Commit-Position: refs/heads/master@{#613035}

components/data_use_measurement/core/data_use_measurement_unittest.cc

@@ -8,9 +8,9 @@
 #include <string>
 
 #include "base/macros.h"
-#include "base/message_loop/message_loop.h"
 #include "base/run_loop.h"
 #include "base/test/metrics/histogram_tester.h"
+#include "base/test/scoped_task_environment.h"
 #include "build/build_config.h"
 #include "components/data_use_measurement/core/data_use_ascriber.h"
 #include "components/data_use_measurement/core/data_use_recorder.h"
@@ -228,7 +228,8 @@ class DataUseMeasurementTest : public testing::Test {
     context_->Init();
   }
 
-  base::MessageLoopForIO loop_;
+  base::test::ScopedTaskEnvironment task_environment_{
+      base::test::ScopedTaskEnvironment::MainThreadType::IO};
 
   TestDataUseAscriber ascriber_;
   TestURLRequestClassifier* url_request_classifier_;

components/data_use_measurement/core/data_use_network_delegate_unittest.cc

@@ -6,9 +6,9 @@
 
 #include <memory>
 
-#include "base/message_loop/message_loop.h"
 #include "base/run_loop.h"
 #include "base/test/metrics/histogram_tester.h"
+#include "base/test/scoped_task_environment.h"
 #include "components/data_use_measurement/core/data_use_ascriber.h"
 #include "components/data_use_measurement/core/url_request_classifier.h"
 #include "components/metrics/data_use_tracker.h"
@@ -106,7 +106,8 @@ std::unique_ptr<net::URLRequest> RequestURL(
   if (redirect)
     socket_factory->AddSocketDataProvider(&redirect_socket_data_provider);
   net::MockRead response_mock_reads[] = {
-      net::MockRead("HTTP/1.1 200 OK\r\n\r\n"), net::MockRead("response body"),
+      net::MockRead("HTTP/1.1 200 OK\r\n\r\n"),
+      net::MockRead("response body"),
       net::MockRead(net::SYNCHRONOUS, net::OK),
   };
   const auto traffic_annotation =
@@ -155,7 +156,8 @@ class DataUseNetworkDelegateTest : public testing::Test {
   }
 
  private:
-  base::MessageLoopForIO message_loop_;
+  base::test::ScopedTaskEnvironment task_environment_{
+      base::test::ScopedTaskEnvironment::MainThreadType::IO};
   net::MockClientSocketFactory mock_socket_factory_;
   net::TestURLRequestContext context_;
   TestDataUseAscriber test_data_use_ascriber_;