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

Unified Diff: content/browser/indexed_db/indexed_db_quota_client_unittest.cc

Issue 2930183002: Let IndexedDBContextImpl create its own task runner (Closed)
Patch Set: Actually compile Created 3 years, 5 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: content/browser/indexed_db/indexed_db_quota_client_unittest.cc
diff --git a/content/browser/indexed_db/indexed_db_quota_client_unittest.cc b/content/browser/indexed_db/indexed_db_quota_client_unittest.cc
index a5a11cee301d7bfd322eaec132f0731312f46122..8474e399864592c7a0ae65cf0fa3a8fbb7c58627 100644
--- a/content/browser/indexed_db/indexed_db_quota_client_unittest.cc
+++ b/content/browser/indexed_db/indexed_db_quota_client_unittest.cc
@@ -12,7 +12,7 @@
#include "base/files/scoped_temp_dir.h"
#include "base/macros.h"
#include "base/run_loop.h"
-#include "base/test/test_simple_task_runner.h"
+#include "base/threading/sequenced_task_runner_handle.h"
#include "base/threading/thread.h"
#include "base/threading/thread_task_runner_handle.h"
#include "content/browser/browser_thread_impl.h"
@@ -21,6 +21,7 @@
#include "content/public/browser/storage_partition.h"
#include "content/public/test/test_browser_context.h"
#include "content/public/test/test_browser_thread_bundle.h"
+#include "content/public/test/test_utils.h"
#include "storage/browser/test/mock_quota_manager.h"
#include "testing/gtest/include/gtest/gtest.h"
@@ -42,27 +43,22 @@ class IndexedDBQuotaClientTest : public testing::Test {
kOriginB("http://host:8000"),
kOriginOther("http://other"),
usage_(0),
- task_runner_(new base::TestSimpleTaskRunner),
weak_factory_(this) {
browser_context_.reset(new TestBrowserContext());
scoped_refptr<storage::QuotaManager> quota_manager =
new MockQuotaManager(false /*in_memory*/, browser_context_->GetPath(),
base::ThreadTaskRunnerHandle::Get(),
- base::ThreadTaskRunnerHandle::Get(),
+ base::SequencedTaskRunnerHandle::Get(),
browser_context_->GetSpecialStoragePolicy());
- idb_context_ =
- new IndexedDBContextImpl(browser_context_->GetPath(),
- browser_context_->GetSpecialStoragePolicy(),
- quota_manager->proxy(),
- task_runner_.get());
+ idb_context_ = new IndexedDBContextImpl(
+ browser_context_->GetPath(),
+ browser_context_->GetSpecialStoragePolicy(), quota_manager->proxy());
base::RunLoop().RunUntilIdle();
setup_temp_dir();
}
- void FlushIndexedDBTaskRunner() { task_runner_->RunUntilIdle(); }
-
void setup_temp_dir() {
ASSERT_TRUE(temp_dir_.CreateUniqueTempDir());
base::FilePath indexeddb_dir =
@@ -72,7 +68,7 @@ class IndexedDBQuotaClientTest : public testing::Test {
}
~IndexedDBQuotaClientTest() override {
- FlushIndexedDBTaskRunner();
+ RunAllBlockingPoolTasksUntilIdle();
idb_context_ = NULL;
browser_context_.reset();
base::RunLoop().RunUntilIdle();
@@ -87,8 +83,7 @@ class IndexedDBQuotaClientTest : public testing::Test {
type,
base::Bind(&IndexedDBQuotaClientTest::OnGetOriginUsageComplete,
weak_factory_.GetWeakPtr()));
- FlushIndexedDBTaskRunner();
- base::RunLoop().RunUntilIdle();
+ RunAllBlockingPoolTasksUntilIdle();
EXPECT_GT(usage_, -1);
return usage_;
}
@@ -100,8 +95,7 @@ class IndexedDBQuotaClientTest : public testing::Test {
type,
base::Bind(&IndexedDBQuotaClientTest::OnGetOriginsComplete,
weak_factory_.GetWeakPtr()));
- FlushIndexedDBTaskRunner();
- base::RunLoop().RunUntilIdle();
+ RunAllBlockingPoolTasksUntilIdle();
return origins_;
}
@@ -114,8 +108,7 @@ class IndexedDBQuotaClientTest : public testing::Test {
host,
base::Bind(&IndexedDBQuotaClientTest::OnGetOriginsComplete,
weak_factory_.GetWeakPtr()));
- FlushIndexedDBTaskRunner();
- base::RunLoop().RunUntilIdle();
+ RunAllBlockingPoolTasksUntilIdle();
return origins_;
}
@@ -127,8 +120,7 @@ class IndexedDBQuotaClientTest : public testing::Test {
kTemp,
base::Bind(&IndexedDBQuotaClientTest::OnDeleteOriginComplete,
weak_factory_.GetWeakPtr()));
- FlushIndexedDBTaskRunner();
- base::RunLoop().RunUntilIdle();
+ RunAllBlockingPoolTasksUntilIdle();
return delete_status_;
}
@@ -148,7 +140,7 @@ class IndexedDBQuotaClientTest : public testing::Test {
}
file_path_origin = file_path_origin.Append(FILE_PATH_LITERAL("fake_file"));
SetFileSizeTo(file_path_origin, size);
- idb_context()->ResetCaches();
+ idb_context()->ResetCachesForTesting();
}
private:
@@ -162,12 +154,11 @@ class IndexedDBQuotaClientTest : public testing::Test {
delete_status_ = code;
}
+ content::TestBrowserThreadBundle thread_bundle_;
base::ScopedTempDir temp_dir_;
int64_t usage_;
std::set<GURL> origins_;
- scoped_refptr<base::TestSimpleTaskRunner> task_runner_;
scoped_refptr<IndexedDBContextImpl> idb_context_;
- content::TestBrowserThreadBundle thread_bundle_;
std::unique_ptr<TestBrowserContext> browser_context_;
storage::QuotaStatusCode delete_status_;
base::WeakPtrFactory<IndexedDBQuotaClientTest> weak_factory_;

Powered by Google App Engine
This is Rietveld 408576698