views: Fix the button's label of the question message box.

chrome/browser/ui/views/simple_message_box_views.cc

Index: chrome/browser/ui/views/simple_message_box_views.cc
diff --git a/chrome/browser/ui/views/simple_message_box_views.cc b/chrome/browser/ui/views/simple_message_box_views.cc
index 18dd80aece719c05dae7cd231f128aa97d8aa69d..141e7a13a32093c48f1505f55ff1b38e356fd6a9 100644
--- a/chrome/browser/ui/views/simple_message_box_views.cc
+++ b/chrome/browser/ui/views/simple_message_box_views.cc
@@ -154,16 +154,19 @@ SimpleMessageBoxViews::~SimpleMessageBoxViews() {
 }
 
 int SimpleMessageBoxViews::GetDialogButtons() const {
-  if (dialog_type_ == DIALOG_TYPE_INFORMATION ||
-      dialog_type_ == DIALOG_TYPE_WARNING)
-    return ui::DIALOG_BUTTON_OK;
-  return ui::DIALOG_BUTTON_OK | ui::DIALOG_BUTTON_CANCEL;
+  if (dialog_type_ == DIALOG_TYPE_QUESTION)
+    return ui::DIALOG_BUTTON_OK | ui::DIALOG_BUTTON_CANCEL;
+  return ui::DIALOG_BUTTON_OK;
 }
 
 string16 SimpleMessageBoxViews::GetDialogButtonLabel(
     ui::DialogButton button) const {
-  return l10n_util::GetStringUTF16((button == ui::DIALOG_BUTTON_OK) ?
-      IDS_OK : IDS_CLOSE);
+  if (dialog_type_ == DIALOG_TYPE_QUESTION) {
+    return l10n_util::GetStringUTF16((button == ui::DIALOG_BUTTON_OK) ?
+        IDS_CONFIRM_MESSAGEBOX_YES_BUTTON_LABEL :

[tfarina, 2012/05/19 14:05:01]

we could have an IDS_YES and IDS_NO.

+        IDS_CONFIRM_MESSAGEBOX_NO_BUTTON_LABEL);
+  }
+  return l10n_util::GetStringUTF16(IDS_OK);
 }
 
 bool SimpleMessageBoxViews::Cancel() {