Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2945)

Unified Diff: chrome/browser/extensions/activity_log/counting_policy_unittest.cc

Issue 1395103003: Don't use base::MessageLoop::{Quit,QuitClosure} in chrome/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/activity_log/counting_policy_unittest.cc
diff --git a/chrome/browser/extensions/activity_log/counting_policy_unittest.cc b/chrome/browser/extensions/activity_log/counting_policy_unittest.cc
index 45ac54161a363347a602eb59b2cb9b1b7b8e57f1..51218dbdc8928aad974b3041b13799221aa72a88 100644
--- a/chrome/browser/extensions/activity_log/counting_policy_unittest.cc
+++ b/chrome/browser/extensions/activity_log/counting_policy_unittest.cc
@@ -69,10 +69,8 @@ class CountingPolicyTest : public testing::Test {
// Wait for the task queue for the specified thread to empty.
void WaitOnThread(const BrowserThread::ID& thread) {
BrowserThread::PostTaskAndReply(
- thread,
- FROM_HERE,
- base::Bind(&base::DoNothing),
- base::MessageLoop::current()->QuitClosure());
+ thread, FROM_HERE, base::Bind(&base::DoNothing),
+ base::MessageLoop::current()->QuitWhenIdleClosure());
base::MessageLoop::current()->Run();
}
@@ -102,15 +100,9 @@ class CountingPolicyTest : public testing::Test {
// checker function when results are available. This will happen on the
// database thread.
policy->ReadFilteredData(
- extension_id,
- type,
- api_name,
- page_url,
- arg_url,
- day,
- base::Bind(&CountingPolicyTest::CheckWrapper,
- checker,
- base::MessageLoop::current()->QuitClosure()));
+ extension_id, type, api_name, page_url, arg_url, day,
+ base::Bind(&CountingPolicyTest::CheckWrapper, checker,
+ base::MessageLoop::current()->QuitWhenIdleClosure()));
// Set up a timeout for receiving results; if we haven't received anything
// when the timeout triggers then assume that the test is broken.

Powered by Google App Engine
This is Rietveld 408576698