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

Unified Diff: chrome/browser/extensions/bookmark_app_helper.cc

Issue 2916753004: Use BookmarkAppConfirmationView on Mac. Delete the NSAlert Mac uses currently. (Closed)
Patch Set: using, OnceCallback Created 3 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/bookmark_app_helper.cc
diff --git a/chrome/browser/extensions/bookmark_app_helper.cc b/chrome/browser/extensions/bookmark_app_helper.cc
index 671c980ee7e6eb46ad5d7e00420fcbd1cd1ddd1b..a667d14de2d1119e873e4d44b52df566fbb6ff8c 100644
--- a/chrome/browser/extensions/bookmark_app_helper.cc
+++ b/chrome/browser/extensions/bookmark_app_helper.cc
@@ -25,6 +25,7 @@
#include "chrome/browser/profiles/profile.h"
#include "chrome/browser/ui/app_list/app_list_service.h"
#include "chrome/browser/ui/app_list/app_list_util.h"
+#include "chrome/browser/ui/browser_dialogs.h"
#include "chrome/browser/ui/browser_finder.h"
#include "chrome/browser/ui/browser_window.h"
#include "chrome/browser/web_applications/web_app.h"
@@ -667,9 +668,10 @@ void BookmarkAppHelper::OnIconsDownloaded(
OnBubbleCompleted(true, web_app_info_);
return;
}
- browser->window()->ShowBookmarkAppBubble(
- web_app_info_, base::Bind(&BookmarkAppHelper::OnBubbleCompleted,
- weak_factory_.GetWeakPtr()));
+ chrome::ShowBookmarkAppDialog(
+ browser->window()->GetNativeWindow(), web_app_info_,
+ base::Bind(&BookmarkAppHelper::OnBubbleCompleted,
+ weak_factory_.GetWeakPtr()));
}
void BookmarkAppHelper::OnBubbleCompleted(
« no previous file with comments | « chrome/browser/extensions/bookmark_app_helper.h ('k') | chrome/browser/extensions/bookmark_app_helper_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698