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

Unified Diff: webkit/appcache/appcache_group_unittest.cc

Issue 8673008: base::Bind: Implement CancelableCallback to replace CancelableTaske. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Test build fix. Created 9 years, 1 month 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: webkit/appcache/appcache_group_unittest.cc
diff --git a/webkit/appcache/appcache_group_unittest.cc b/webkit/appcache/appcache_group_unittest.cc
index 26eb5ca25869c9b23cb11168587aee25654be6a6..662a272791d855d7e9510f8eae4fe5b7eb88916c 100644
--- a/webkit/appcache/appcache_group_unittest.cc
+++ b/webkit/appcache/appcache_group_unittest.cc
@@ -275,14 +275,14 @@ TEST(AppCacheGroupTest, QueueUpdate) {
// sent to host.
delete group->update_job_;
EXPECT_EQ(AppCacheGroup::IDLE, group->update_status_);
- EXPECT_TRUE(group->restart_update_task_);
+ EXPECT_FALSE(group->restart_update_task_.is_null());
EXPECT_FALSE(host.update_completed_);
// Start another update. Cancels task and will run queued updates.
group->update_status_ = AppCacheGroup::CHECKING; // prevent actual fetches
group->StartUpdate();
EXPECT_TRUE(group->update_job_);
- EXPECT_FALSE(group->restart_update_task_);
+ EXPECT_TRUE(group->restart_update_task_.is_null());
EXPECT_TRUE(group->queued_updates_.empty());
EXPECT_FALSE(group->update_job_->pending_master_entries_.empty());
EXPECT_FALSE(group->FindObserver(&host, group->queued_observers_));
@@ -291,7 +291,7 @@ TEST(AppCacheGroupTest, QueueUpdate) {
// Delete update to cause it to complete. Verify host is notified.
delete group->update_job_;
EXPECT_EQ(AppCacheGroup::IDLE, group->update_status_);
- EXPECT_FALSE(group->restart_update_task_);
+ EXPECT_TRUE(group->restart_update_task_.is_null());
EXPECT_TRUE(host.update_completed_);
}

Powered by Google App Engine
This is Rietveld 408576698