Switch to composition for TestBrowserThread.

This avoids exposing BrowserThreadImpl internals outside of content/,
and goes most of the way to not exposing the fact that underneath, it
is a base::Thread.  There are only three test files that use the
TestBrowserThread::DeprecatedGetThreadObject method.

TBR=owners
BUG=98716

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

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

1 files changed, 2 insertions, 5 deletions

diff --git a/chrome/browser/autofill/autofill_download_unittest.cc b/chrome/browser/autofill/autofill_download_unittest.cc
index b90ce98..c8919cc 100644
--- a/chrome/browser/autofill/autofill_download_unittest.cc
+++ b/chrome/browser/autofill/autofill_download_unittest.cc
@@ -71,9 +71,7 @@ class AutofillDownloadTest : public AutofillDownloadManager::Observer,
   }
 
   virtual void SetUp() {
-    base::Thread::Options options;
-    options.message_loop_type = MessageLoop::TYPE_IO;
-    io_thread_.StartWithOptions(options);
+    io_thread_.StartIOThread();
     profile_.CreateRequestContext();
   }
 
@@ -135,8 +133,7 @@ class AutofillDownloadTest : public AutofillDownloadManager::Observer,
   AutofillDownloadManager download_manager_;
 
  private:
-  // |request_context_getter_| must be released on the IO thread.
-  MessageLoop* io_thread_loop() { return io_thread_.message_loop(); }
+  // The profile's request context must be released on the IO thread.
   content::TestBrowserThread io_thread_;
 };