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

Unified Diff: chrome/browser/browsing_data/browsing_data_remover_unittest.cc

Issue 2554413002: Extract embedder-specific data types from BrowsingDataRemover (Closed)
Patch Set: Android fix Created 4 years 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/browsing_data/browsing_data_remover_unittest.cc
diff --git a/chrome/browser/browsing_data/browsing_data_remover_unittest.cc b/chrome/browser/browsing_data/browsing_data_remover_unittest.cc
index 3adb0b589b0ba2222a2f09102bd02de8d1ceee20..190a4bad4033e6afe7626d6e06236b0aa482d15a 100644
--- a/chrome/browser/browsing_data/browsing_data_remover_unittest.cc
+++ b/chrome/browser/browsing_data/browsing_data_remover_unittest.cc
@@ -34,6 +34,7 @@
#include "chrome/browser/browsing_data/browsing_data_helper.h"
#include "chrome/browser/browsing_data/browsing_data_remover_factory.h"
#include "chrome/browser/browsing_data/browsing_data_remover_test_util.h"
+#include "chrome/browser/browsing_data/chrome_browsing_data_remover_delegate.h"
#include "chrome/browser/browsing_data/registrable_domain_filter_builder.h"
#include "chrome/browser/content_settings/host_content_settings_map_factory.h"
#include "chrome/browser/domain_reliability/service_factory.h"
@@ -1126,8 +1127,9 @@ class BrowsingDataRemoverTest : public testing::Test {
BrowsingDataRemoverFactory::GetForBrowserContext(profile_.get());
#if BUILDFLAG(ANDROID_JAVA_UI)
- remover_->OverrideWebappRegistryForTesting(
- std::unique_ptr<WebappRegistry>(new TestWebappRegistry()));
+ static_cast<ChromeBrowsingDataRemoverDelegate*>(
+ remover_->get_embedder_delegate())->OverrideWebappRegistryForTesting(
+ std::unique_ptr<WebappRegistry>(new TestWebappRegistry()));
Bernhard Bauer 2016/12/12 14:52:06 Small nit while you're here: Use base::WrapUnique?
msramek 2016/12/12 20:48:56 Done.
#endif
}

Powered by Google App Engine
This is Rietveld 408576698