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

Side by Side Diff: chrome/browser/ui/views/extensions/chooser_dialog_view.cc

Issue 2086663003: Change ChooserController ownership model (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: address reillyg@'s comments Created 4 years, 6 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/ui/views/extensions/chooser_dialog_view.h" 5 #include "chrome/browser/ui/views/extensions/chooser_dialog_view.h"
6 6
7 #include "base/strings/utf_string_conversions.h" 7 #include "base/strings/utf_string_conversions.h"
8 #include "chrome/browser/extensions/chrome_extension_chooser_dialog.h" 8 #include "chrome/browser/extensions/chrome_extension_chooser_dialog.h"
9 #include "chrome/browser/ui/views/chooser_content_view.h" 9 #include "chrome/browser/ui/views/chooser_content_view.h"
10 #include "chrome/grit/generated_resources.h" 10 #include "chrome/grit/generated_resources.h"
11 #include "components/chooser_controller/chooser_controller.h" 11 #include "components/chooser_controller/chooser_controller.h"
12 #include "components/constrained_window/constrained_window_views.h" 12 #include "components/constrained_window/constrained_window_views.h"
13 #include "components/url_formatter/elide_url.h" 13 #include "components/url_formatter/elide_url.h"
14 #include "components/web_modal/web_contents_modal_dialog_manager.h" 14 #include "components/web_modal/web_contents_modal_dialog_manager.h"
15 #include "content/public/browser/browser_thread.h" 15 #include "content/public/browser/browser_thread.h"
16 #include "extensions/browser/extension_registry.h" 16 #include "extensions/browser/extension_registry.h"
17 #include "ui/base/l10n/l10n_util.h" 17 #include "ui/base/l10n/l10n_util.h"
18 #include "ui/views/controls/styled_label.h" 18 #include "ui/views/controls/styled_label.h"
19 #include "ui/views/window/dialog_client_view.h" 19 #include "ui/views/window/dialog_client_view.h"
20 #include "url/origin.h"
21 20
22 ChooserDialogView::ChooserDialogView(content::WebContents* web_contents, 21 ChooserDialogView::ChooserDialogView(
23 ChooserController* chooser_controller) 22 content::WebContents* web_contents,
24 : chooser_content_view_(nullptr), 23 std::unique_ptr<ChooserController> chooser_controller)
25 web_contents_(web_contents), 24 : web_contents_(web_contents), chooser_content_view_(nullptr) {
26 chooser_controller_(chooser_controller) {
27 // ------------------------------------ 25 // ------------------------------------
28 // | Chooser dialog title | 26 // | Chooser dialog title |
29 // | -------------------------------- | 27 // | -------------------------------- |
30 // | | option 0 | | 28 // | | option 0 | |
31 // | | option 1 | | 29 // | | option 1 | |
32 // | | option 2 | | 30 // | | option 2 | |
33 // | | | | 31 // | | | |
34 // | | | | 32 // | | | |
35 // | | | | 33 // | | | |
36 // | -------------------------------- | 34 // | -------------------------------- |
37 // | [ Connect ] [ Cancel ] | 35 // | [ Connect ] [ Cancel ] |
38 // |----------------------------------| 36 // |----------------------------------|
39 // | Not seeing your device? Get help | 37 // | Not seeing your device? Get help |
40 // ------------------------------------ 38 // ------------------------------------
41 39
42 DCHECK(web_contents_); 40 DCHECK(web_contents_);
43 41 DCHECK(chooser_controller);
44 chooser_content_view_ = new ChooserContentView(this, chooser_controller_); 42 origin_ = chooser_controller->GetOrigin();
43 chooser_content_view_ =
44 new ChooserContentView(this, std::move(chooser_controller));
45 } 45 }
46 46
47 ChooserDialogView::~ChooserDialogView() {} 47 ChooserDialogView::~ChooserDialogView() {}
48 48
49 base::string16 ChooserDialogView::GetWindowTitle() const { 49 base::string16 ChooserDialogView::GetWindowTitle() const {
50 base::string16 chooser_title; 50 base::string16 chooser_title;
51 url::Origin origin = chooser_controller_->GetOrigin();
52 content::BrowserContext* browser_context = web_contents_->GetBrowserContext(); 51 content::BrowserContext* browser_context = web_contents_->GetBrowserContext();
53 extensions::ExtensionRegistry* extension_registry = 52 extensions::ExtensionRegistry* extension_registry =
54 extensions::ExtensionRegistry::Get(browser_context); 53 extensions::ExtensionRegistry::Get(browser_context);
55 if (extension_registry) { 54 if (extension_registry) {
56 const extensions::Extension* extension = 55 const extensions::Extension* extension =
57 extension_registry->enabled_extensions().GetExtensionOrAppByURL( 56 extension_registry->enabled_extensions().GetExtensionOrAppByURL(
58 GURL(origin.Serialize())); 57 GURL(origin_.Serialize()));
59 if (extension) 58 if (extension)
60 chooser_title = base::UTF8ToUTF16(extension->name()); 59 chooser_title = base::UTF8ToUTF16(extension->name());
61 } 60 }
62 61
63 if (chooser_title.empty()) { 62 if (chooser_title.empty()) {
64 chooser_title = url_formatter::FormatOriginForSecurityDisplay( 63 chooser_title = url_formatter::FormatOriginForSecurityDisplay(
65 origin, url_formatter::SchemeDisplay::OMIT_CRYPTOGRAPHIC); 64 origin_, url_formatter::SchemeDisplay::OMIT_CRYPTOGRAPHIC);
66 } 65 }
67 66
68 return l10n_util::GetStringFUTF16(IDS_DEVICE_CHOOSER_PROMPT, chooser_title); 67 return l10n_util::GetStringFUTF16(IDS_DEVICE_CHOOSER_PROMPT, chooser_title);
69 } 68 }
70 69
71 bool ChooserDialogView::ShouldShowCloseButton() const { 70 bool ChooserDialogView::ShouldShowCloseButton() const {
72 return false; 71 return false;
73 } 72 }
74 73
75 ui::ModalType ChooserDialogView::GetModalType() const { 74 ui::ModalType ChooserDialogView::GetModalType() const {
(...skipping 48 matching lines...) Expand 10 before | Expand all | Expand 10 after
124 123
125 void ChooserDialogView::OnSelectionChanged() { 124 void ChooserDialogView::OnSelectionChanged() {
126 GetDialogClientView()->UpdateDialogButtons(); 125 GetDialogClientView()->UpdateDialogButtons();
127 } 126 }
128 127
129 views::TableView* ChooserDialogView::table_view_for_test() const { 128 views::TableView* ChooserDialogView::table_view_for_test() const {
130 return chooser_content_view_->table_view_for_test(); 129 return chooser_content_view_->table_view_for_test();
131 } 130 }
132 131
133 void ChromeExtensionChooserDialog::ShowDialogImpl( 132 void ChromeExtensionChooserDialog::ShowDialogImpl(
134 ChooserController* chooser_controller) const { 133 std::unique_ptr<ChooserController> chooser_controller) const {
135 DCHECK_CURRENTLY_ON(content::BrowserThread::UI); 134 DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
136 DCHECK(chooser_controller); 135 DCHECK(chooser_controller);
137 136
138 web_modal::WebContentsModalDialogManager* manager = 137 web_modal::WebContentsModalDialogManager* manager =
139 web_modal::WebContentsModalDialogManager::FromWebContents(web_contents_); 138 web_modal::WebContentsModalDialogManager::FromWebContents(web_contents_);
140 if (manager) { 139 if (manager) {
141 constrained_window::ShowWebModalDialogViews( 140 constrained_window::ShowWebModalDialogViews(
142 new ChooserDialogView(web_contents_, chooser_controller), 141 new ChooserDialogView(web_contents_, std::move(chooser_controller)),
143 web_contents_); 142 web_contents_);
144 } 143 }
145 } 144 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698