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

Unified Diff: chrome/browser/ui/cocoa/tab_dialogs_views_mac.mm

Issue 2808823002: MacViews: Allows the toolkit-views Manage Passwords Dialog to be used (Closed)
Patch Set: MacViews: Allows the toolkit-views Manage Passwords Dialog to be used (comments) Created 3 years, 8 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/cocoa/tab_dialogs_views_mac.mm
diff --git a/chrome/browser/ui/cocoa/tab_dialogs_views_mac.mm b/chrome/browser/ui/cocoa/tab_dialogs_views_mac.mm
index d71c1f239b9f957b6f8c11b4e3a784cb5a5031a8..3be5da4f5fddd7e5c17cfeca23f01826723135db 100644
--- a/chrome/browser/ui/cocoa/tab_dialogs_views_mac.mm
+++ b/chrome/browser/ui/cocoa/tab_dialogs_views_mac.mm
@@ -4,8 +4,13 @@
#include "chrome/browser/ui/cocoa/tab_dialogs_views_mac.h"
+#include "chrome/browser/ui/cocoa/browser_window_controller.h"
+#import "chrome/browser/ui/cocoa/passwords/passwords_bubble_cocoa.h"
#include "chrome/browser/ui/views/collected_cookies_views.h"
+#include "chrome/browser/ui/views/passwords/manage_passwords_bubble_view.h"
#include "chrome/browser/ui/views/sync/profile_signin_confirmation_dialog_views.h"
+#include "content/public/browser/web_contents.h"
+#include "ui/base/material_design/material_design_controller.h"
TabDialogsViewsMac::TabDialogsViewsMac(content::WebContents* contents)
: TabDialogsCocoa(contents) {}
@@ -25,3 +30,29 @@ void TabDialogsViewsMac::ShowProfileSigninConfirmation(
ProfileSigninConfirmationDialogViews::ShowDialog(browser, profile, username,
std::move(delegate));
}
+
+void TabDialogsViewsMac::ShowManagePasswordsBubble(bool user_action) {
+ if (!ui::MaterialDesignController::IsSecondaryUiMaterial()) {
+ TabDialogsCocoa::ShowManagePasswordsBubble(user_action);
+ return;
+ }
+ NSWindow* window = [web_contents()->GetNativeView() window];
+ if (!window) {
+ // The tab isn't active right now.
+ return;
+ }
+ BrowserWindowController* bwc =
+ [BrowserWindowController browserWindowControllerForWindow:window];
+ NSPoint anchor_point = [bwc bookmarkBubblePoint];
+ ShowViewsManagePasswordsBubbleOnCocoaBrowser(anchor_point, web_contents(),
+ user_action);
+}
+
+void TabDialogsViewsMac::HideManagePasswordsBubble() {
+ // Close toolkit-views bubble.
+ if (!ui::MaterialDesignController::IsSecondaryUiMaterial()) {
+ TabDialogsCocoa::HideManagePasswordsBubble();
+ return;
+ }
+ ManagePasswordsBubbleView::CloseCurrentBubble();
+}

Powered by Google App Engine
This is Rietveld 408576698