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

Unified Diff: chrome/browser/web_applications/web_app_mac.mm

Issue 2929773002: Mac: Remove create_application_shortcut_cocoa.mm (Closed)
Patch Set: One less file 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/web_applications/web_app_mac.mm
diff --git a/chrome/browser/web_applications/web_app_mac.mm b/chrome/browser/web_applications/web_app_mac.mm
index 178a16b90854ff813818f599cb99799797040c47..b51db206aaaae46e02be7f8ce14d21750fbdc184 100644
--- a/chrome/browser/web_applications/web_app_mac.mm
+++ b/chrome/browser/web_applications/web_app_mac.mm
@@ -1113,3 +1113,20 @@ void DeleteAllShortcutsForProfile(const base::FilePath& profile_path) {
} // namespace internals
} // namespace web_app
+
+namespace chrome {
+
+void ShowCreateChromeAppShortcutsDialog(
benwells 2017/06/09 04:28:52 Hmmm .. this name is unfortunate. I guess it doesn
tapted 2017/06/09 05:47:03 Acknowledged.
+ gfx::NativeWindow /*parent_window*/,
+ Profile* profile,
+ const extensions::Extension* app,
+ const base::Callback<void(bool)>& close_callback) {
+ // On Mac, the Applications folder is the only option, so don't bother asking
+ // the user anything. Just create shortcuts.
+ CreateShortcuts(web_app::SHORTCUT_CREATION_BY_USER,
+ web_app::ShortcutLocations(), profile, app);
+ if (!close_callback.is_null())
+ close_callback.Run(true);
+}
+
+} // namespace chrome

Powered by Google App Engine
This is Rietveld 408576698