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

Unified Diff: chrome/browser/ui/views/extensions/chooser_dialog_view.cc

Issue 2478863003: Fix the Web Bluetooth chooser when it is used on Chrome apps on non-Mac (Closed)
Patch Set: added code to handle the footnote link pointer ownership Created 4 years, 1 month 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.cc
diff --git a/chrome/browser/ui/views/extensions/chooser_dialog_view.cc b/chrome/browser/ui/views/extensions/chooser_dialog_view.cc
index da13e010d7215e23d6e69636f4ea58295043d320..af8b36842831ef4a01a1b3d370c5cc70564eedde 100644
--- a/chrome/browser/ui/views/extensions/chooser_dialog_view.cc
+++ b/chrome/browser/ui/views/extensions/chooser_dialog_view.cc
@@ -11,8 +11,10 @@
#include "components/constrained_window/constrained_window_views.h"
#include "components/web_modal/web_contents_modal_dialog_manager.h"
#include "content/public/browser/browser_thread.h"
+#include "ui/gfx/geometry/insets.h"
#include "ui/views/controls/link.h"
#include "ui/views/controls/styled_label.h"
+#include "ui/views/layout/layout_constants.h"
#include "ui/views/window/dialog_client_view.h"
ChooserDialogView::ChooserDialogView(
@@ -64,6 +66,23 @@ views::View* ChooserDialogView::CreateFootnoteView() {
return chooser_content_view_->CreateFootnoteView();
}
+views::ClientView* ChooserDialogView::CreateClientView(views::Widget* widget) {
+ views::DialogClientView* client =
+ new views::DialogClientView(widget, GetContentsView());
+ client->set_button_row_insets(gfx::Insets());
+ return client;
+}
+
+views::NonClientFrameView* ChooserDialogView::CreateNonClientFrameView(
+ views::Widget* widget) {
+ if (ShouldUseCustomFrame()) {
+ return views::DialogDelegate::CreateDialogFrameView(
+ widget, gfx::Insets(views::kPanelVertMargin, views::kPanelHorizMargin,
+ views::kPanelVertMargin, views::kPanelHorizMargin));
+ }
+ return DialogDelegate::CreateNonClientFrameView(widget);
+}
+
bool ChooserDialogView::Accept() {
chooser_content_view_->Accept();
return true;

Powered by Google App Engine
This is Rietveld 408576698