[chrome/browser/chromeos/extensions] Deprecation of loop proxy usage with TTRH

chrome/browser/chromeos/extensions/device_local_account_external_policy_loader_unittest.cc

 // Copyright 2013 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.
 
 #include "chrome/browser/chromeos/extensions/device_local_account_external_policy_loader.h"
 
 #include <string>
 
 #include "base/callback.h"
 #include "base/files/file_util.h"
 #include "base/files/scoped_temp_dir.h"
 #include "base/message_loop/message_loop.h"
-#include "base/message_loop/message_loop_proxy.h"
 #include "base/path_service.h"
 #include "base/run_loop.h"
+#include "base/single_thread_task_runner.h"

[benwells, 2015/05/07 00:05:26]

Nit: this include isn't needed.

[Pranay, 2015/05/07 03:16:50]

Done.

 #include "base/strings/stringprintf.h"
+#include "base/thread_task_runner_handle.h"
 #include "base/values.h"
 #include "base/version.h"
 #include "chrome/browser/extensions/external_provider_impl.h"
 #include "chrome/common/chrome_paths.h"
 #include "chrome/test/base/testing_browser_process.h"
 #include "components/policy/core/common/cloud/mock_cloud_policy_store.h"
 #include "components/policy/core/common/policy_map.h"
 #include "components/policy/core/common/policy_types.h"
 #include "content/public/browser/notification_service.h"
 #include "content/public/browser/notification_source.h"
@@ skipping 108 matching lines @@
 
 DeviceLocalAccountExternalPolicyLoaderTest::
     ~DeviceLocalAccountExternalPolicyLoaderTest() {
 }
 
 void DeviceLocalAccountExternalPolicyLoaderTest::SetUp() {
   ASSERT_TRUE(temp_dir_.CreateUniqueTempDir());
   cache_dir_ = temp_dir_.path().Append(kCacheDir);
   ASSERT_TRUE(base::CreateDirectoryAndGetError(cache_dir_, NULL));
   request_context_getter_ =
-      new net::TestURLRequestContextGetter(base::MessageLoopProxy::current());
+      new net::TestURLRequestContextGetter(base::ThreadTaskRunnerHandle::Get());
   TestingBrowserProcess::GetGlobal()->SetSystemRequestContext(
       request_context_getter_.get());
   ASSERT_TRUE(PathService::Get(chrome::DIR_TEST_DATA, &test_dir_));
 
   loader_ = new DeviceLocalAccountExternalPolicyLoader(&store_, cache_dir_);
   provider_.reset(new extensions::ExternalProviderImpl(
       &visitor_,
       loader_,
       NULL,
       extensions::Manifest::EXTERNAL_POLICY,
@@ skipping 48 matching lines @@
 }
 
 // Verifies that the cache can be started and stopped correctly.
 TEST_F(DeviceLocalAccountExternalPolicyLoaderTest, ForceInstallListEmpty) {
   // Set an empty force-install list policy.
   store_.NotifyStoreLoaded();
 
   // Start the cache. Verify that the loader announces an empty extension list.
   EXPECT_CALL(visitor_, OnExternalProviderReady(provider_.get()))
       .Times(1);
-  loader_->StartCache(base::MessageLoopProxy::current());
+  loader_->StartCache(base::ThreadTaskRunnerHandle::Get());
   base::RunLoop().RunUntilIdle();
   VerifyAndResetVisitorCallExpectations();
 
   // Stop the cache. Verify that the loader announces an empty extension list.
   EXPECT_CALL(visitor_, OnExternalProviderReady(provider_.get()))
       .Times(1);
   base::RunLoop run_loop;
   loader_->StopCache(run_loop.QuitClosure());
   VerifyAndResetVisitorCallExpectations();
 
   // Spin the loop until the cache shutdown callback is invoked. Verify that at
   // that point, no further file I/O tasks are pending.
   run_loop.Run();
   EXPECT_TRUE(base::MessageLoop::current()->IsIdleForTesting());
 }
 
 // Verifies that when a force-install list policy referencing an extension is
 // set and the cache is started, the loader downloads, caches and serves the
 // extension.
 TEST_F(DeviceLocalAccountExternalPolicyLoaderTest, ForceInstallListSet) {
   // Set a force-install list policy that contains an invalid entry (which
   // should be ignored) and a valid reference to an extension.
   SetForceInstallListPolicy();
 
   // Start the cache.
-  loader_->StartCache(base::MessageLoopProxy::current());
+  loader_->StartCache(base::ThreadTaskRunnerHandle::Get());
 
   // Spin the loop, allowing the loader to process the force-install list.
   // Verify that the loader announces an empty extension list.
   net::TestURLFetcherFactory factory;
   EXPECT_CALL(visitor_, OnExternalProviderReady(provider_.get()))
       .Times(1);
   base::MessageLoop::current()->RunUntilIdle();
 
   // Verify that a downloader has started and is attempting to download an
   // update manifest.
@@ skipping 61 matching lines @@
   loader_->StopCache(shutdown_run_loop.QuitClosure());
   VerifyAndResetVisitorCallExpectations();
 
   // Spin the loop until the cache shutdown callback is invoked. Verify that at
   // that point, no further file I/O tasks are pending.
   shutdown_run_loop.Run();
   EXPECT_TRUE(base::MessageLoop::current()->IsIdleForTesting());
 }
 
 }  // namespace chromeos