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

Unified Diff: chrome/browser/ui/views/extensions/chooser_dialog_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/extensions/chooser_dialog_view_unittest.cc
diff --git a/chrome/browser/ui/views/extensions/chooser_dialog_view_unittest.cc b/chrome/browser/ui/views/extensions/chooser_dialog_view_unittest.cc
index efd22b174a5639688528f978b9911e9c2b0e9c50..f63b11ca400c96095ef19366ff419fd3fa0de7c2 100644
--- a/chrome/browser/ui/views/extensions/chooser_dialog_view_unittest.cc
+++ b/chrome/browser/ui/views/extensions/chooser_dialog_view_unittest.cc
@@ -30,8 +30,7 @@ class ChooserDialogViewTest : 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();
std::unique_ptr<ChooserDialogView> chooser_dialog_view(
new ChooserDialogView(std::move(mock_chooser_controller)));
« no previous file with comments | « chrome/browser/ui/views/device_chooser_content_view_unittest.cc ('k') | chrome/browser/usb/usb_chooser_context.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698