Use BookmarkAppConfirmationView on Mac. Delete the NSAlert Mac uses currently.

chrome/browser/ui/views/extensions/bookmark_app_confirmation_view.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/views/extensions/bookmark_app_confirmation_view.h"
 
 #include "base/callback_helpers.h"
 #include "base/strings/string16.h"
 #include "base/strings/string_util.h"
 #include "build/build_config.h"
-#include "chrome/browser/ui/browser_dialogs.h"
+#include "chrome/browser/extensions/extension_util.h"
 #include "chrome/browser/ui/views/harmony/chrome_layout_provider.h"
 #include "chrome/grit/generated_resources.h"
 #include "components/constrained_window/constrained_window_views.h"
 #include "components/strings/grit/components_strings.h"
 #include "content/public/browser/web_contents.h"
 #include "extensions/common/constants.h"
 #include "ui/accessibility/ax_node_data.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "ui/gfx/image/image_skia.h"
@@ skipping 27 matching lines @@
   }
 
   int dip_size_;
   WebApplicationInfo info_;
 };
 
 }  // namespace
 
 BookmarkAppConfirmationView::~BookmarkAppConfirmationView() {}
 
-// static
-void BookmarkAppConfirmationView::CreateAndShow(
-    gfx::NativeWindow parent,
-    const WebApplicationInfo& web_app_info,
-    const BrowserWindow::ShowBookmarkAppBubbleCallback& callback) {
-  constrained_window::CreateBrowserModalDialogViews(
-      new BookmarkAppConfirmationView(web_app_info, callback), parent)
-      ->Show();
-}
-
 BookmarkAppConfirmationView::BookmarkAppConfirmationView(
     const WebApplicationInfo& web_app_info,
-    const BrowserWindow::ShowBookmarkAppBubbleCallback& callback)
+    chrome::ShowBookmarkAppDialogCallback callback)
     : web_app_info_(web_app_info),
-      callback_(callback),
+      callback_(std::move(callback)),
       open_as_window_checkbox_(nullptr),
       title_tf_(nullptr) {
   const ChromeLayoutProvider* layout_provider = ChromeLayoutProvider::Get();
   views::GridLayout* layout = views::GridLayout::CreatePanel(this);
   const int column_set_id = 0;
 
   views::ColumnSet* column_set = layout->AddColumnSet(column_set_id);
   column_set->AddColumn(views::GridLayout::FILL, views::GridLayout::CENTER, 0,
                         views::GridLayout::USE_PREF, 0, 0);
   column_set->AddPaddingColumn(0,
@@ skipping 18 matching lines @@
   title_tf_ = new views::Textfield();
   title_tf_->SetText(web_app_info_.title);
   title_tf_->SetAccessibleName(
       l10n_util::GetStringUTF16(IDS_BOOKMARK_APP_AX_BUBBLE_NAME_LABEL));
   title_tf_->set_controller(this);
   layout->AddView(title_tf_);
 
   layout->AddPaddingRow(
       0, layout_provider->GetDistanceMetric(DISTANCE_CONTROL_LIST_VERTICAL));
 
-  open_as_window_checkbox_ = new views::Checkbox(
-      l10n_util::GetStringUTF16(IDS_BOOKMARK_APP_BUBBLE_OPEN_AS_WINDOW));
-  open_as_window_checkbox_->SetChecked(web_app_info_.open_as_window);
-  layout->StartRow(0, column_set_id);
-  layout->SkipColumns(1);
-  layout->AddView(open_as_window_checkbox_);
+  // When CanHostedAppsOpenInWindows() returns false, do not show the open as
+  // window checkbox to avoid confusing users.
+  if (extensions::util::CanHostedAppsOpenInWindows()) {
+    open_as_window_checkbox_ = new views::Checkbox(
+        l10n_util::GetStringUTF16(IDS_BOOKMARK_APP_BUBBLE_OPEN_AS_WINDOW));
+    open_as_window_checkbox_->SetChecked(web_app_info_.open_as_window);
+    layout->StartRow(0, column_set_id);
+    layout->SkipColumns(1);
+    layout->AddView(open_as_window_checkbox_);
+  }
 
   title_tf_->SelectAll(true);
   chrome::RecordDialogCreation(
       chrome::DialogIdentifier::BOOKMARK_APP_CONFIRMATION);
 }
 
 views::View* BookmarkAppConfirmationView::GetInitiallyFocusedView() {
   return title_tf_;
 }
 
@@ skipping 10 matching lines @@
 
   return l10n_util::GetStringUTF16(ids);
 }
 
 bool BookmarkAppConfirmationView::ShouldShowCloseButton() const {
   return false;
 }
 
 void BookmarkAppConfirmationView::WindowClosing() {
   if (!callback_.is_null())
-    callback_.Run(false, web_app_info_);
+    std::move(callback_).Run(false, web_app_info_);
 }
 
 bool BookmarkAppConfirmationView::Accept() {
   web_app_info_.title = GetTrimmedTitle();
-  web_app_info_.open_as_window = open_as_window_checkbox_->checked();
+  web_app_info_.open_as_window =
+      open_as_window_checkbox_ && open_as_window_checkbox_->checked();
   base::ResetAndReturn(&callback_).Run(true, web_app_info_);
   return true;
 }
 
 base::string16 BookmarkAppConfirmationView::GetDialogButtonLabel(
     ui::DialogButton button) const {
   return l10n_util::GetStringUTF16(button == ui::DIALOG_BUTTON_OK ? IDS_ADD
                                                                   : IDS_CANCEL);
 }
 
@@ skipping 13 matching lines @@
     const base::string16& new_contents) {
   DCHECK_EQ(title_tf_, sender);
   GetDialogClientView()->UpdateDialogButtons();
 }
 
 base::string16 BookmarkAppConfirmationView::GetTrimmedTitle() const {
   base::string16 title(title_tf_->text());
   base::TrimWhitespace(title, base::TRIM_ALL, &title);
   return title;
 }
+
+namespace chrome {
+
+void ShowBookmarkAppDialog(gfx::NativeWindow parent,
+                           const WebApplicationInfo& web_app_info,
+                           ShowBookmarkAppDialogCallback callback) {
+  constrained_window::CreateBrowserModalDialogViews(
+      new BookmarkAppConfirmationView(web_app_info, std::move(callback)),
+      parent)
+      ->Show();
+}
+
+}  // namespace chrome