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

Unified Diff: chrome/browser/ui/views/device_chooser_content_view_unittest.cc

Issue 2746313002: Remove RenderFrameHost pointer from ChooserController. (Closed)
Patch Set: Fix Android build and juncai@ comment. Created 3 years, 9 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/browser/ui/views/device_chooser_content_view_unittest.cc
diff --git a/chrome/browser/ui/views/device_chooser_content_view_unittest.cc b/chrome/browser/ui/views/device_chooser_content_view_unittest.cc
index 047d238a6bf2565d93e14d551ff2a7806be65848..7d6cce1f5ee608b999a9dcdef38b406cb7211cf6 100644
--- a/chrome/browser/ui/views/device_chooser_content_view_unittest.cc
+++ b/chrome/browser/ui/views/device_chooser_content_view_unittest.cc
@@ -45,8 +45,7 @@ class DeviceChooserContentViewTest : public views::ViewsTestBase {
// views::ViewsTestBase:
void SetUp() override {
views::ViewsTestBase::SetUp();
- std::unique_ptr<MockChooserController> mock_chooser_controller(
- new MockChooserController(nullptr));
+ auto mock_chooser_controller = base::MakeUnique<MockChooserController>();
mock_chooser_controller_ = mock_chooser_controller.get();
mock_table_view_observer_ = base::MakeUnique<MockTableViewObserver>();
device_chooser_content_view_ = base::MakeUnique<DeviceChooserContentView>(

Powered by Google App Engine
This is Rietveld 408576698