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

Unified Diff: components/previews/previews_io_data_unittest.cc

Issue 2335023002: Adding a previews IO-thread blacklist (Closed)
Patch Set: tbansal comments Created 4 years, 3 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: components/previews/previews_io_data_unittest.cc
diff --git a/components/previews/previews_io_data_unittest.cc b/components/previews/previews_io_data_unittest.cc
index 4d99c92cf5cf6663184aba8fe442713af44e50de..8b36868d4dd7066adb512a198039e02f24fd3764 100644
--- a/components/previews/previews_io_data_unittest.cc
+++ b/components/previews/previews_io_data_unittest.cc
@@ -23,23 +23,24 @@ class TestPreviewsIOData : public PreviewsIOData {
const scoped_refptr<base::SingleThreadTaskRunner>& io_task_runner,
const scoped_refptr<base::SingleThreadTaskRunner>& ui_task_runner)
: PreviewsIOData(io_task_runner, ui_task_runner), initialized_(false) {}
~TestPreviewsIOData() override {}
// Whether Initialize was called.
bool initialized() { return initialized_; }
private:
// Set |initialized_| to true and use base class functionality.
- void InitializeOnIOThread() override {
+ void InitializeOnIOThread(
+ std::unique_ptr<PreviewsOptOutStore> previews_opt_out_store) override {
initialized_ = true;
- PreviewsIOData::InitializeOnIOThread();
+ PreviewsIOData::InitializeOnIOThread(std::move(previews_opt_out_store));
}
// Whether Initialize was called.
bool initialized_;
};
class PreviewsIODataTest : public testing::Test {
public:
PreviewsIODataTest() {}
@@ -62,19 +63,19 @@ class PreviewsIODataTest : public testing::Test {
private:
std::unique_ptr<TestPreviewsIOData> io_data_;
std::unique_ptr<PreviewsUIService> ui_service_;
};
} // namespace
TEST_F(PreviewsIODataTest, TestInitialization) {
set_io_data(base::WrapUnique(
new TestPreviewsIOData(loop_.task_runner(), loop_.task_runner())));
- set_ui_service(
- base::WrapUnique(new PreviewsUIService(io_data(), loop_.task_runner())));
+ set_ui_service(base::WrapUnique(
+ new PreviewsUIService(io_data(), loop_.task_runner(), nullptr)));
loop_.RunUntilIdle();
// After the outstanding posted tasks have run, |io_data_| should be fully
// initialized.
EXPECT_TRUE(io_data()->initialized());
}
} // namespace previews

Powered by Google App Engine
This is Rietveld 408576698