summaryrefslogtreecommitdiffstats
path: root/chrome/browser/views/about_chrome_view.cc
diff options
context:
space:
mode:
authortc@google.com <tc@google.com@0039d316-1c4b-4281-b951-d872f2087c98>2009-04-21 22:31:47 +0000
committertc@google.com <tc@google.com@0039d316-1c4b-4281-b951-d872f2087c98>2009-04-21 22:31:47 +0000
commit7f1a907d93ee0f7ac527424a70e245343ce0539a (patch)
treef2676a17b23d23c9c044dd64bde0b6601dc256d1 /chrome/browser/views/about_chrome_view.cc
parent806aeb954d302328cc75bda71331474d8634675b (diff)
downloadchromium_src-7f1a907d93ee0f7ac527424a70e245343ce0539a.zip
chromium_src-7f1a907d93ee0f7ac527424a70e245343ce0539a.tar.gz
chromium_src-7f1a907d93ee0f7ac527424a70e245343ce0539a.tar.bz2
Revert "Refactor DialogDelegate so the DialogButton enum is in cross platform"
This reverts commit r14146. MessageBox is redefined to MessageBoxW by windows. I need to rename the class. Review URL: http://codereview.chromium.org/87064 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14147 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/views/about_chrome_view.cc')
-rw-r--r--chrome/browser/views/about_chrome_view.cc24
1 files changed, 11 insertions, 13 deletions
diff --git a/chrome/browser/views/about_chrome_view.cc b/chrome/browser/views/about_chrome_view.cc
index 88bf4d9..df605ab 100644
--- a/chrome/browser/views/about_chrome_view.cc
+++ b/chrome/browser/views/about_chrome_view.cc
@@ -603,11 +603,15 @@ void AboutChromeView::ViewHierarchyChanged(bool is_add,
////////////////////////////////////////////////////////////////////////////////
// AboutChromeView, views::DialogDelegate implementation:
+int AboutChromeView::GetDialogButtons() const {
+ return DIALOGBUTTON_OK | DIALOGBUTTON_CANCEL;
+}
+
std::wstring AboutChromeView::GetDialogButtonLabel(
- MessageBox::DialogButton button) const {
- if (button == MessageBox::DIALOGBUTTON_OK) {
+ DialogButton button) const {
+ if (button == DIALOGBUTTON_OK) {
return l10n_util::GetString(IDS_ABOUT_CHROME_UPDATE_CHECK);
- } else if (button == MessageBox::DIALOGBUTTON_CANCEL) {
+ } else if (button == DIALOGBUTTON_CANCEL) {
// The OK button (which is the default button) has been re-purposed to be
// 'Check for Updates' so we want the Cancel button should have the label
// OK but act like a Cancel button in all other ways.
@@ -618,22 +622,16 @@ std::wstring AboutChromeView::GetDialogButtonLabel(
return L"";
}
-bool AboutChromeView::IsDialogButtonEnabled(
- MessageBox::DialogButton button) const {
- if (button == MessageBox::DIALOGBUTTON_OK &&
- check_button_status_ != CHECKBUTTON_ENABLED) {
+bool AboutChromeView::IsDialogButtonEnabled(DialogButton button) const {
+ if (button == DIALOGBUTTON_OK && check_button_status_ != CHECKBUTTON_ENABLED)
return false;
- }
return true;
}
-bool AboutChromeView::IsDialogButtonVisible(
- MessageBox::DialogButton button) const {
- if (button == MessageBox::DIALOGBUTTON_OK &&
- check_button_status_ == CHECKBUTTON_HIDDEN) {
+bool AboutChromeView::IsDialogButtonVisible(DialogButton button) const {
+ if (button == DIALOGBUTTON_OK && check_button_status_ == CHECKBUTTON_HIDDEN)
return false;
- }
return true;
}