Convert CookieMonster tests to use base::RunLoop

net/cookies/cookie_store_test_callbacks.cc

Index: net/cookies/cookie_store_test_callbacks.cc
diff --git a/net/cookies/cookie_store_test_callbacks.cc b/net/cookies/cookie_store_test_callbacks.cc
index 2a8a659e79f3f601f5ac2c610467628051b5dc46..48335fdb319d61c106eb1707fe0e562298459e91 100644
--- a/net/cookies/cookie_store_test_callbacks.cc
+++ b/net/cookies/cookie_store_test_callbacks.cc
@@ -16,15 +16,15 @@ CookieCallback::CookieCallback(base::Thread* run_in_thread)
     : did_run_(false),
       run_in_thread_(run_in_thread),
       run_in_loop_(NULL),
-      parent_loop_(base::MessageLoop::current()),
-      loop_to_quit_(base::MessageLoop::current()) {}
+      loop_to_quit_(new base::RunLoop) {}
 
 CookieCallback::CookieCallback()
     : did_run_(false),
       run_in_thread_(NULL),
       run_in_loop_(base::MessageLoop::current()),
-      parent_loop_(NULL),
-      loop_to_quit_(base::MessageLoop::current()) {}
+      loop_to_quit_(new base::RunLoop) {}

[mmenke, 2016/01/27 16:55:42]

nit:  Can just replace this with:

CookieCallback::CookieCallback() : CookieCallback(base::MessageLoop::current())
...

[xunjieli, 2016/01/27 18:35:02]

Hmm.. Not sure if I understand.  I don't think I can stick a message loop as a
base::Thread*, right? Are you suggesting changing the argument? 

[mmenke, 2016/01/27 18:52:17]

No, I'm suggest defining this constructor in terms of the other constructor.

CookieCallback::CookieCallback() : CookieCallback(base::MessageLoop::current())
{
}

Will say when this constructor is called, call the other one, with the current
message loop as the argument.  So don't have to duplicate shared initialization.

[xunjieli, 2016/01/27 18:54:53]

The other constructor takes in a base::Thread, not a MessageLoop.

[mmenke, 2016/01/27 21:36:25]

Oops...sorry about that.

+
+CookieCallback::~CookieCallback() {}
 
 void CookieCallback::CallbackEpilogue() {
   base::MessageLoop* expected_loop = NULL;
@@ -38,8 +38,11 @@ void CookieCallback::CallbackEpilogue() {
 
   did_run_ = true;
   EXPECT_EQ(expected_loop, base::MessageLoop::current());
-  loop_to_quit_->task_runner()->PostTask(
-      FROM_HERE, base::MessageLoop::QuitWhenIdleClosure());
+  loop_to_quit_->Quit();
+}
+
+void CookieCallback::WaitUntilDone() {
+  loop_to_quit_->Run();
 }
 
 StringResultCookieCallback::StringResultCookieCallback() {}