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

Unified Diff: chrome/test/base/chrome_render_view_host_test_harness.cc

Issue 2696263002: Refactor ViewsDelegate and MD-ify the icon-to-text spacing for checkbox and radiobutton (Closed)
Patch Set: Fix unit tests to use a ViewsDelegate to guard against crashes Created 3 years, 10 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/test/base/chrome_render_view_host_test_harness.cc
diff --git a/chrome/test/base/chrome_render_view_host_test_harness.cc b/chrome/test/base/chrome_render_view_host_test_harness.cc
index 950fd1b6a6374d0db49822d4d7a7937b9b9c4cf1..e121da2af83aa3c477e95617b099df016f1dbbcf 100644
--- a/chrome/test/base/chrome_render_view_host_test_harness.cc
+++ b/chrome/test/base/chrome_render_view_host_test_harness.cc
@@ -6,6 +6,7 @@
#include <utility>
+#include "base/memory/ptr_util.h"
#include "build/build_config.h"
#include "chrome/browser/browser_process.h"
#include "chrome/browser/signin/account_tracker_service_factory.h"
@@ -49,8 +50,8 @@ std::unique_ptr<KeyedService> BuildSigninManagerFake(
} // namespace
-ChromeRenderViewHostTestHarness::ChromeRenderViewHostTestHarness() {
-}
+ChromeRenderViewHostTestHarness::ChromeRenderViewHostTestHarness()
+ : views_delegate_(base::MakeUnique<views::TestViewsDelegate>()) {}
ChromeRenderViewHostTestHarness::~ChromeRenderViewHostTestHarness() {
}

Powered by Google App Engine
This is Rietveld 408576698