Fix unit test leak in DeviceOAuth2TokenServiceTest.

chrome/browser/chromeos/settings/device_oauth2_token_service_unittest.cc

Index: chrome/browser/chromeos/settings/device_oauth2_token_service_unittest.cc
diff --git a/chrome/browser/chromeos/settings/device_oauth2_token_service_unittest.cc b/chrome/browser/chromeos/settings/device_oauth2_token_service_unittest.cc
index 61c8c578d45296115e0194585d3668068f0e9dcd..ddead26131561f7145b54f00817f0b823db40c12 100644
--- a/chrome/browser/chromeos/settings/device_oauth2_token_service_unittest.cc
+++ b/chrome/browser/chromeos/settings/device_oauth2_token_service_unittest.cc
@@ -61,9 +61,7 @@ TEST_F(DeviceOAuth2TokenServiceTest, SaveEncryptedToken) {
       .WillOnce(Return("test-token"));
 
   DeviceOAuth2TokenService oauth2_service(
-      new net::TestURLRequestContextGetter(
-          content::BrowserThread::GetMessageLoopProxyForThread(
-              content::BrowserThread::IO)),
+      new net::TestURLRequestContextGetter(message_loop_.message_loop_proxy()),

[Mattias Nissler (ping if slow), 2013/04/19 17:45:19]

I don't see how this fixes the problem, can you enlighten me?

[David Roche, 2013/04/19 22:05:03]

So, the key is in URLRequestContextGetter::OnDestruct().  If it has a task
runner on the current thread it will immediately 'delete this', but if the task
runner is on a diff thread, then it will

  network_task_runner->DeleteSoon(FROM_HERE, this)

which would only run if my test called RunUntilIdle() on that task runner, which
it wasn't.

However, since this test isn't even using the URLRequestContextGetter anyway
(it's just needed to pass a DCHECK), it was simpler to just give it a task
runner from the current message loop and thereby trigger an immediate delete.

       scoped_testing_local_state_.Get());
 
   ASSERT_EQ("", oauth2_service.GetRefreshToken());