diff options
Diffstat (limited to 'chrome/browser/views')
47 files changed, 156 insertions, 153 deletions
diff --git a/chrome/browser/views/about_chrome_view.cc b/chrome/browser/views/about_chrome_view.cc index df605ab..131d9e0 100644 --- a/chrome/browser/views/about_chrome_view.cc +++ b/chrome/browser/views/about_chrome_view.cc @@ -603,15 +603,11 @@ void AboutChromeView::ViewHierarchyChanged(bool is_add, //////////////////////////////////////////////////////////////////////////////// // AboutChromeView, views::DialogDelegate implementation: -int AboutChromeView::GetDialogButtons() const { - return DIALOGBUTTON_OK | DIALOGBUTTON_CANCEL; -} - std::wstring AboutChromeView::GetDialogButtonLabel( - DialogButton button) const { - if (button == DIALOGBUTTON_OK) { + MessageBoxFlags::DialogButton button) const { + if (button == MessageBoxFlags::DIALOGBUTTON_OK) { return l10n_util::GetString(IDS_ABOUT_CHROME_UPDATE_CHECK); - } else if (button == DIALOGBUTTON_CANCEL) { + } else if (button == MessageBoxFlags::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. @@ -622,16 +618,22 @@ std::wstring AboutChromeView::GetDialogButtonLabel( return L""; } -bool AboutChromeView::IsDialogButtonEnabled(DialogButton button) const { - if (button == DIALOGBUTTON_OK && check_button_status_ != CHECKBUTTON_ENABLED) +bool AboutChromeView::IsDialogButtonEnabled( + MessageBoxFlags::DialogButton button) const { + if (button == MessageBoxFlags::DIALOGBUTTON_OK && + check_button_status_ != CHECKBUTTON_ENABLED) { return false; + } return true; } -bool AboutChromeView::IsDialogButtonVisible(DialogButton button) const { - if (button == DIALOGBUTTON_OK && check_button_status_ == CHECKBUTTON_HIDDEN) +bool AboutChromeView::IsDialogButtonVisible( + MessageBoxFlags::DialogButton button) const { + if (button == MessageBoxFlags::DIALOGBUTTON_OK && + check_button_status_ == CHECKBUTTON_HIDDEN) { return false; + } return true; } diff --git a/chrome/browser/views/about_chrome_view.h b/chrome/browser/views/about_chrome_view.h index 2118e76..d4cc3fd 100644 --- a/chrome/browser/views/about_chrome_view.h +++ b/chrome/browser/views/about_chrome_view.h @@ -47,10 +47,12 @@ class AboutChromeView : public views::View, views::View* child); // Overridden from views::DialogDelegate: - virtual int GetDialogButtons() const; - virtual std::wstring GetDialogButtonLabel(DialogButton button) const; - virtual bool IsDialogButtonEnabled(DialogButton button) const; - virtual bool IsDialogButtonVisible(DialogButton button) const; + virtual std::wstring GetDialogButtonLabel( + MessageBoxFlags::DialogButton button) const; + virtual bool IsDialogButtonEnabled( + MessageBoxFlags::DialogButton button) const; + virtual bool IsDialogButtonVisible( + MessageBoxFlags::DialogButton button) const; virtual bool CanResize() const; virtual bool CanMaximize() const; virtual bool IsAlwaysOnTop() const; diff --git a/chrome/browser/views/bookmark_editor_view.cc b/chrome/browser/views/bookmark_editor_view.cc index b5a2f6d..eb00198 100644 --- a/chrome/browser/views/bookmark_editor_view.cc +++ b/chrome/browser/views/bookmark_editor_view.cc @@ -80,8 +80,9 @@ void BookmarkEditorView::Show(HWND parent_hwnd, editor->Show(parent_hwnd); } -bool BookmarkEditorView::IsDialogButtonEnabled(DialogButton button) const { - if (button == DIALOGBUTTON_OK) { +bool BookmarkEditorView::IsDialogButtonEnabled( + MessageBoxFlags::DialogButton button) const { + if (button == MessageBoxFlags::DIALOGBUTTON_OK) { const GURL url(GetInputURL()); return bb_model_->IsLoaded() && url.is_valid(); } @@ -96,7 +97,7 @@ std::wstring BookmarkEditorView::GetWindowTitle() const { } bool BookmarkEditorView::Accept() { - if (!IsDialogButtonEnabled(DIALOGBUTTON_OK)) { + if (!IsDialogButtonEnabled(MessageBoxFlags::DIALOGBUTTON_OK)) { // The url is invalid, focus the url field. url_tf_.SelectAll(); url_tf_.RequestFocus(); @@ -107,7 +108,8 @@ bool BookmarkEditorView::Accept() { return true; } -bool BookmarkEditorView::AreAcceleratorsEnabled(DialogButton button) { +bool BookmarkEditorView::AreAcceleratorsEnabled( + MessageBoxFlags::DialogButton button) { return !show_tree_ || !tree_view_->GetEditingNode(); } diff --git a/chrome/browser/views/bookmark_editor_view.h b/chrome/browser/views/bookmark_editor_view.h index a307277..6b472d2 100644 --- a/chrome/browser/views/bookmark_editor_view.h +++ b/chrome/browser/views/bookmark_editor_view.h @@ -86,11 +86,12 @@ class BookmarkEditorView : public views::View, Handler* handler); // DialogDelegate methods: - virtual bool IsDialogButtonEnabled(DialogButton button) const; + virtual bool IsDialogButtonEnabled( + MessageBoxFlags::DialogButton button) const; virtual bool IsModal() const; virtual std::wstring GetWindowTitle() const; virtual bool Accept(); - virtual bool AreAcceleratorsEnabled(DialogButton button); + virtual bool AreAcceleratorsEnabled(MessageBoxFlags::DialogButton button); virtual views::View* GetContentsView(); // View methods. diff --git a/chrome/browser/views/bug_report_view.cc b/chrome/browser/views/bug_report_view.cc index 647dfc1..33dd6d7 100644 --- a/chrome/browser/views/bug_report_view.cc +++ b/chrome/browser/views/bug_report_view.cc @@ -266,8 +266,9 @@ bool BugReportView::HandleKeystroke(views::TextField* sender, return false; } -std::wstring BugReportView::GetDialogButtonLabel(DialogButton button) const { - if (button == DIALOGBUTTON_OK) { +std::wstring BugReportView::GetDialogButtonLabel( + MessageBoxFlags::DialogButton button) const { + if (button == MessageBoxFlags::DIALOGBUTTON_OK) { if (problem_type_ == BugReportComboBoxModel::PHISHING_PAGE) return l10n_util::GetString(IDS_BUGREPORT_SEND_PHISHING_REPORT); else @@ -278,7 +279,7 @@ std::wstring BugReportView::GetDialogButtonLabel(DialogButton button) const { } int BugReportView::GetDefaultDialogButton() const { - return DIALOGBUTTON_NONE; + return MessageBoxFlags::DIALOGBUTTON_NONE; } bool BugReportView::CanResize() const { @@ -306,7 +307,7 @@ std::wstring BugReportView::GetWindowTitle() const { } bool BugReportView::Accept() { - if (IsDialogButtonEnabled(DIALOGBUTTON_OK)) { + if (IsDialogButtonEnabled(MessageBoxFlags::DIALOGBUTTON_OK)) { if (problem_type_ == BugReportComboBoxModel::PHISHING_PAGE) ReportPhishing(); else diff --git a/chrome/browser/views/bug_report_view.h b/chrome/browser/views/bug_report_view.h index 22a2c42..d17e569 100644 --- a/chrome/browser/views/bug_report_view.h +++ b/chrome/browser/views/bug_report_view.h @@ -61,7 +61,8 @@ class BugReportView : public views::View, int new_index); // Overridden from views::DialogDelegate: - virtual std::wstring GetDialogButtonLabel(DialogButton button) const; + virtual std::wstring GetDialogButtonLabel( + MessageBoxFlags::DialogButton button) const; virtual int GetDefaultDialogButton() const; virtual bool CanResize() const; virtual bool CanMaximize() const; diff --git a/chrome/browser/views/clear_browsing_data.cc b/chrome/browser/views/clear_browsing_data.cc index 5a0df99..0b00f6a 100644 --- a/chrome/browser/views/clear_browsing_data.cc +++ b/chrome/browser/views/clear_browsing_data.cc @@ -236,21 +236,22 @@ void ClearBrowsingDataView::ViewHierarchyChanged(bool is_add, // ClearBrowsingDataView, views::DialogDelegate implementation: std::wstring ClearBrowsingDataView::GetDialogButtonLabel( - DialogButton button) const { - if (button == DIALOGBUTTON_OK) { + MessageBoxFlags::DialogButton button) const { + if (button == MessageBoxFlags::DIALOGBUTTON_OK) { return l10n_util::GetString(IDS_CLEAR_BROWSING_DATA_COMMIT); - } else if (button == DIALOGBUTTON_CANCEL) { + } else if (button == MessageBoxFlags::DIALOGBUTTON_CANCEL) { return l10n_util::GetString(IDS_CLOSE); } else { return std::wstring(); } } -bool ClearBrowsingDataView::IsDialogButtonEnabled(DialogButton button) const { +bool ClearBrowsingDataView::IsDialogButtonEnabled( + MessageBoxFlags::DialogButton button) const { if (delete_in_progress_) return false; - if (button == DIALOGBUTTON_OK) { + if (button == MessageBoxFlags::DIALOGBUTTON_OK) { return del_history_checkbox_->checked() || del_downloads_checkbox_->checked() || del_cache_checkbox_->checked() || @@ -287,7 +288,7 @@ std::wstring ClearBrowsingDataView::GetWindowTitle() const { } bool ClearBrowsingDataView::Accept() { - if (!IsDialogButtonEnabled(DIALOGBUTTON_OK)) { + if (!IsDialogButtonEnabled(MessageBoxFlags::DIALOGBUTTON_OK)) { return false; } diff --git a/chrome/browser/views/clear_browsing_data.h b/chrome/browser/views/clear_browsing_data.h index 18912fc..de4ea31 100644 --- a/chrome/browser/views/clear_browsing_data.h +++ b/chrome/browser/views/clear_browsing_data.h @@ -50,8 +50,9 @@ class ClearBrowsingDataView : public views::View, views::View* child); // Overridden from views::DialogDelegate: - virtual std::wstring GetDialogButtonLabel(DialogButton button) const; - virtual bool IsDialogButtonEnabled(DialogButton button) const; + virtual std::wstring GetDialogButtonLabel( + MessageBoxFlags::DialogButton button) const; + virtual bool IsDialogButtonEnabled(MessageBoxFlags::DialogButton button) const; virtual bool CanResize() const; virtual bool CanMaximize() const; virtual bool IsAlwaysOnTop() const; diff --git a/chrome/browser/views/edit_keyword_controller.cc b/chrome/browser/views/edit_keyword_controller.cc index 1463777d..8173849 100644 --- a/chrome/browser/views/edit_keyword_controller.cc +++ b/chrome/browser/views/edit_keyword_controller.cc @@ -77,12 +77,9 @@ std::wstring EditKeywordController::GetWindowTitle() const { IDS_SEARCH_ENGINES_EDITOR_NEW_WINDOW_TITLE); } -int EditKeywordController::GetDialogButtons() const { - return DIALOGBUTTON_OK | DIALOGBUTTON_CANCEL; -} - -bool EditKeywordController::IsDialogButtonEnabled(DialogButton button) const { - if (button == DIALOGBUTTON_OK) { +bool EditKeywordController::IsDialogButtonEnabled( + MessageBoxFlags::DialogButton button) const { + if (button == MessageBoxFlags::DIALOGBUTTON_OK) { return (IsKeywordValid() && !title_tf_->GetText().empty() && IsURLValid()); } return true; diff --git a/chrome/browser/views/edit_keyword_controller.h b/chrome/browser/views/edit_keyword_controller.h index 00a76fe..e9a8f36 100644 --- a/chrome/browser/views/edit_keyword_controller.h +++ b/chrome/browser/views/edit_keyword_controller.h @@ -44,8 +44,8 @@ class EditKeywordController : public views::TextField::Controller, // DialogDelegate overrides. virtual bool IsModal() const; virtual std::wstring GetWindowTitle() const; - virtual int GetDialogButtons() const; - virtual bool IsDialogButtonEnabled(DialogButton button) const; + virtual bool IsDialogButtonEnabled( + MessageBoxFlags::DialogButton button) const; virtual void DeleteDelegate(); virtual bool Cancel(); virtual bool Accept(); diff --git a/chrome/browser/views/external_protocol_dialog.cc b/chrome/browser/views/external_protocol_dialog.cc index 50cdce3..2852b60 100644 --- a/chrome/browser/views/external_protocol_dialog.cc +++ b/chrome/browser/views/external_protocol_dialog.cc @@ -43,17 +43,13 @@ ExternalProtocolDialog::~ExternalProtocolDialog() { ////////////////////////////////////////////////////////////////////////////// // ExternalProtocolDialog, views::DialogDelegate implementation: -int ExternalProtocolDialog::GetDialogButtons() const { - return DIALOGBUTTON_OK | DIALOGBUTTON_CANCEL; -} - int ExternalProtocolDialog::GetDefaultDialogButton() const { - return DIALOGBUTTON_CANCEL; + return MessageBoxFlags::DIALOGBUTTON_CANCEL; } std::wstring ExternalProtocolDialog::GetDialogButtonLabel( - DialogButton button) const { - if (button == DIALOGBUTTON_OK) + MessageBoxFlags::DialogButton button) const { + if (button == MessageBoxFlags::DIALOGBUTTON_OK) return l10n_util::GetString(IDS_EXTERNAL_PROTOCOL_OK_BUTTON_TEXT); // Set the button to have a default name. @@ -104,7 +100,7 @@ ExternalProtocolDialog::ExternalProtocolDialog(TabContents* tab_contents, message_text += l10n_util::GetString(IDS_EXTERNAL_PROTOCOL_WARNING); - message_box_view_ = new MessageBoxView(MessageBox::kIsConfirmMessageBox, + message_box_view_ = new MessageBoxView(MessageBoxFlags::kIsConfirmMessageBox, message_text, L"", kMessageWidth); diff --git a/chrome/browser/views/external_protocol_dialog.h b/chrome/browser/views/external_protocol_dialog.h index 3936556..7392cbb 100644 --- a/chrome/browser/views/external_protocol_dialog.h +++ b/chrome/browser/views/external_protocol_dialog.h @@ -35,9 +35,9 @@ class ExternalProtocolDialog : public views::DialogDelegate { virtual ~ExternalProtocolDialog(); // views::DialogDelegate Methods: - virtual int GetDialogButtons() const; virtual int GetDefaultDialogButton() const; - virtual std::wstring GetDialogButtonLabel(DialogButton button) const; + virtual std::wstring GetDialogButtonLabel( + MessageBoxFlags::DialogButton button) const; virtual std::wstring GetWindowTitle() const; virtual void DeleteDelegate(); virtual bool Accept(); diff --git a/chrome/browser/views/first_run_customize_view.cc b/chrome/browser/views/first_run_customize_view.cc index 1d0b469..a4dbe5d 100644 --- a/chrome/browser/views/first_run_customize_view.cc +++ b/chrome/browser/views/first_run_customize_view.cc @@ -178,7 +178,7 @@ views::View* FirstRunCustomizeView::GetContentsView() { } bool FirstRunCustomizeView::Accept() { - if (!IsDialogButtonEnabled(DIALOGBUTTON_OK)) + if (!IsDialogButtonEnabled(MessageBoxFlags::DIALOGBUTTON_OK)) return false; DisableButtons(); diff --git a/chrome/browser/views/first_run_view.cc b/chrome/browser/views/first_run_view.cc index 24f5e11..0c94020 100644 --- a/chrome/browser/views/first_run_view.cc +++ b/chrome/browser/views/first_run_view.cc @@ -171,7 +171,7 @@ views::View* FirstRunView::GetContentsView() { } bool FirstRunView::Accept() { - if (!IsDialogButtonEnabled(DIALOGBUTTON_OK)) + if (!IsDialogButtonEnabled(MessageBoxFlags::DIALOGBUTTON_OK)) return false; DisableButtons(); diff --git a/chrome/browser/views/first_run_view_base.cc b/chrome/browser/views/first_run_view_base.cc index c3df0e2..08f30d3 100644 --- a/chrome/browser/views/first_run_view_base.cc +++ b/chrome/browser/views/first_run_view_base.cc @@ -151,8 +151,9 @@ bool FirstRunViewBase::HasAlwaysOnTopMenu() const { return false; } -std::wstring FirstRunViewBase::GetDialogButtonLabel(DialogButton button) const { - if (DIALOGBUTTON_OK == button) +std::wstring FirstRunViewBase::GetDialogButtonLabel( + MessageBoxFlags::DialogButton button) const { + if (MessageBoxFlags::DIALOGBUTTON_OK == button) return l10n_util::GetString(IDS_FIRSTRUN_DLG_OK); // The other buttons get the default text. return std::wstring(); diff --git a/chrome/browser/views/first_run_view_base.h b/chrome/browser/views/first_run_view_base.h index d4a08f7..365f6dd 100644 --- a/chrome/browser/views/first_run_view_base.h +++ b/chrome/browser/views/first_run_view_base.h @@ -39,7 +39,7 @@ class FirstRunViewBase : public views::View, virtual bool HasAlwaysOnTopMenu() const; // Overridden from views::DialogDelegate. - std::wstring GetDialogButtonLabel(DialogButton button) const; + std::wstring GetDialogButtonLabel(MessageBoxFlags::DialogButton button) const; protected: // Returns the items that the first run process is required to import diff --git a/chrome/browser/views/frame/browser_view.cc b/chrome/browser/views/frame/browser_view.cc index 6cee57d..8652d7f 100644 --- a/chrome/browser/views/frame/browser_view.cc +++ b/chrome/browser/views/frame/browser_view.cc @@ -221,14 +221,15 @@ class DownloadInProgressConfirmDialogDelegate : public views::DialogDelegate, // DialogDelegate implementation: virtual int GetDefaultDialogButton() const { - return DIALOGBUTTON_CANCEL; + return MessageBoxFlags::DIALOGBUTTON_CANCEL; } - virtual std::wstring GetDialogButtonLabel(DialogButton button) const { - if (button == DIALOGBUTTON_OK) + virtual std::wstring GetDialogButtonLabel( + MessageBoxFlags::DialogButton button) const { + if (button == MessageBoxFlags::DIALOGBUTTON_OK) return l10n_util::GetString(IDS_DOWNLOAD_REMOVE_CONFIRM_OK_BUTTON_LABEL); - DCHECK_EQ(DIALOGBUTTON_CANCEL, button); + DCHECK_EQ(MessageBoxFlags::DIALOGBUTTON_CANCEL, button); return l10n_util::GetString( IDS_DOWNLOAD_REMOVE_CONFIRM_CANCEL_BUTTON_LABEL); } diff --git a/chrome/browser/views/hung_renderer_view.cc b/chrome/browser/views/hung_renderer_view.cc index 9d239e4..9b7b8ec8 100644 --- a/chrome/browser/views/hung_renderer_view.cc +++ b/chrome/browser/views/hung_renderer_view.cc @@ -126,7 +126,7 @@ class HungRendererWarningView : public views::View, virtual void WindowClosing(); virtual int GetDialogButtons() const; virtual std::wstring GetDialogButtonLabel( - views::DialogDelegate::DialogButton button) const; + MessageBoxFlags::DialogButton button) const; virtual views::View* GetExtraView(); virtual bool Accept(bool window_closing); virtual views::View* GetContentsView(); @@ -272,12 +272,12 @@ int HungRendererWarningView::GetDialogButtons() const { // the OK button to wait for responsiveness (and close the dialog) and our // additional button (which we create) to kill the process (which will result // in the dialog being destroyed). - return DIALOGBUTTON_OK; + return MessageBoxFlags::DIALOGBUTTON_OK; } std::wstring HungRendererWarningView::GetDialogButtonLabel( - views::DialogDelegate::DialogButton button) const { - if (button == DIALOGBUTTON_OK) + MessageBoxFlags::DialogButton button) const { + if (button == MessageBoxFlags::DIALOGBUTTON_OK) return l10n_util::GetString(IDS_BROWSER_HANGMONITOR_RENDERER_WAIT); return std::wstring(); } diff --git a/chrome/browser/views/importer_lock_view.cc b/chrome/browser/views/importer_lock_view.cc index 7c6b809..06aef04 100644 --- a/chrome/browser/views/importer_lock_view.cc +++ b/chrome/browser/views/importer_lock_view.cc @@ -46,10 +46,10 @@ void ImporterLockView::Layout() { } std::wstring ImporterLockView::GetDialogButtonLabel( - DialogButton button) const { - if (button == DIALOGBUTTON_OK) { + MessageBoxFlags::DialogButton button) const { + if (button == MessageBoxFlags::DIALOGBUTTON_OK) { return l10n_util::GetString(IDS_IMPORTER_LOCK_OK); - } else if (button == DIALOGBUTTON_CANCEL) { + } else if (button == MessageBoxFlags::DIALOGBUTTON_CANCEL) { return l10n_util::GetString(IDS_IMPORTER_LOCK_CANCEL); } return std::wstring(); diff --git a/chrome/browser/views/importer_lock_view.h b/chrome/browser/views/importer_lock_view.h index e513bd4..7b17e78 100644 --- a/chrome/browser/views/importer_lock_view.h +++ b/chrome/browser/views/importer_lock_view.h @@ -28,7 +28,8 @@ class ImporterLockView : public views::View, virtual void Layout(); // Overridden from views::DialogDelegate: - virtual std::wstring GetDialogButtonLabel(DialogButton button) const; + virtual std::wstring GetDialogButtonLabel( + MessageBoxFlags::DialogButton button) const; virtual bool IsModal() const; virtual std::wstring GetWindowTitle() const; virtual bool Accept(); diff --git a/chrome/browser/views/importer_view.cc b/chrome/browser/views/importer_view.cc index c3dbbf0..f3a2d11 100644 --- a/chrome/browser/views/importer_view.cc +++ b/chrome/browser/views/importer_view.cc @@ -99,8 +99,8 @@ void ImporterView::Layout() { } std::wstring ImporterView::GetDialogButtonLabel( - DialogButton button) const { - if (button == DIALOGBUTTON_OK) { + MessageBoxFlags::DialogButton button) const { + if (button == MessageBoxFlags::DIALOGBUTTON_OK) { return l10n_util::GetString(IDS_IMPORT_COMMIT); } else { return std::wstring(); @@ -116,7 +116,7 @@ std::wstring ImporterView::GetWindowTitle() const { } bool ImporterView::Accept() { - if (!IsDialogButtonEnabled(DIALOGBUTTON_OK)) { + if (!IsDialogButtonEnabled(MessageBoxFlags::DIALOGBUTTON_OK)) { return false; } diff --git a/chrome/browser/views/importer_view.h b/chrome/browser/views/importer_view.h index 9c6e7a8..903764f 100644 --- a/chrome/browser/views/importer_view.h +++ b/chrome/browser/views/importer_view.h @@ -37,7 +37,8 @@ class ImporterView : public views::View, virtual void Layout(); // Overridden from views::DialogDelegate: - virtual std::wstring GetDialogButtonLabel(DialogButton button) const; + virtual std::wstring GetDialogButtonLabel( + MessageBoxFlags::DialogButton button) const; virtual bool IsModal() const; virtual std::wstring GetWindowTitle() const; virtual bool Accept(); diff --git a/chrome/browser/views/importing_progress_view.cc b/chrome/browser/views/importing_progress_view.cc index d7dcf81..cac8043 100644 --- a/chrome/browser/views/importing_progress_view.cc +++ b/chrome/browser/views/importing_progress_view.cc @@ -182,12 +182,12 @@ void ImportingProgressView::ViewHierarchyChanged(bool is_add, // ImportingProgressView, views::DialogDelegate implementation: int ImportingProgressView::GetDialogButtons() const { - return DIALOGBUTTON_CANCEL; + return MessageBoxFlags::DIALOGBUTTON_CANCEL; } std::wstring ImportingProgressView::GetDialogButtonLabel( - DialogButton button) const { - DCHECK(button == DIALOGBUTTON_CANCEL); + MessageBoxFlags::DialogButton button) const { + DCHECK(button == MessageBoxFlags::DIALOGBUTTON_CANCEL); return l10n_util::GetString(IDS_IMPORT_PROGRESS_STATUS_CANCEL); } diff --git a/chrome/browser/views/importing_progress_view.h b/chrome/browser/views/importing_progress_view.h index 3ff77a5..3de6cfc 100644 --- a/chrome/browser/views/importing_progress_view.h +++ b/chrome/browser/views/importing_progress_view.h @@ -39,7 +39,8 @@ class ImportingProgressView : public views::View, // Overridden from views::DialogDelegate: virtual int GetDialogButtons() const; - virtual std::wstring GetDialogButtonLabel(DialogButton button) const; + virtual std::wstring GetDialogButtonLabel( + MessageBoxFlags::DialogButton button) const; virtual bool IsModal() const; virtual std::wstring GetWindowTitle() const; virtual bool Cancel(); diff --git a/chrome/browser/views/input_window.cc b/chrome/browser/views/input_window.cc index 8d8c24f..2bc2363 100644 --- a/chrome/browser/views/input_window.cc +++ b/chrome/browser/views/input_window.cc @@ -33,7 +33,8 @@ class ContentView : public views::View, } // views::DialogDelegate overrides: - virtual bool IsDialogButtonEnabled(DialogButton button) const; + virtual bool IsDialogButtonEnabled( + MessageBoxFlags::DialogButton button) const; virtual bool Accept(); virtual bool Cancel(); virtual void WindowClosing(); @@ -77,9 +78,12 @@ class ContentView : public views::View, /////////////////////////////////////////////////////////////////////////////// // ContentView, views::DialogDelegate implementation: -bool ContentView::IsDialogButtonEnabled(DialogButton button) const { - if (button == DIALOGBUTTON_OK && !delegate_->IsValid(text_field_->GetText())) +bool ContentView::IsDialogButtonEnabled( + MessageBoxFlags::DialogButton button) const { + if (button == MessageBoxFlags::DIALOGBUTTON_OK && + !delegate_->IsValid(text_field_->GetText())) { return false; + } return true; } diff --git a/chrome/browser/views/jsmessage_box_dialog.cc b/chrome/browser/views/jsmessage_box_dialog.cc index bd0c7f2..b42c2d4 100644 --- a/chrome/browser/views/jsmessage_box_dialog.cc +++ b/chrome/browser/views/jsmessage_box_dialog.cc @@ -20,7 +20,7 @@ JavascriptMessageBoxDialog::JavascriptMessageBoxDialog( : parent_(parent), dialog_(NULL), message_box_view_(new MessageBoxView( - parent->dialog_flags() | MessageBox::kAutoDetectAlignment, + parent->dialog_flags() | MessageBoxFlags::kAutoDetectAlignment, message_text, default_prompt_text)) { DCHECK(message_box_view_); @@ -59,11 +59,11 @@ void JavascriptMessageBoxDialog::CloseModalDialog() { int JavascriptMessageBoxDialog::GetDialogButtons() const { int dialog_buttons = 0; - if (parent_->dialog_flags() & MessageBox::kFlagHasOKButton) - dialog_buttons = DIALOGBUTTON_OK; + if (parent_->dialog_flags() & MessageBoxFlags::kFlagHasOKButton) + dialog_buttons = MessageBoxFlags::DIALOGBUTTON_OK; - if (parent_->dialog_flags() & MessageBox::kFlagHasCancelButton) - dialog_buttons |= DIALOGBUTTON_CANCEL; + if (parent_->dialog_flags() & MessageBoxFlags::kFlagHasCancelButton) + dialog_buttons |= MessageBoxFlags::DIALOGBUTTON_CANCEL; return dialog_buttons; } @@ -96,11 +96,11 @@ bool JavascriptMessageBoxDialog::Accept() { } std::wstring JavascriptMessageBoxDialog::GetDialogButtonLabel( - DialogButton button) const { + MessageBoxFlags::DialogButton button) const { if (parent_->is_before_unload_dialog()) { - if (button == DialogDelegate::DIALOGBUTTON_OK) { + if (button == MessageBoxFlags::DIALOGBUTTON_OK) { return l10n_util::GetString(IDS_BEFOREUNLOAD_MESSAGEBOX_OK_BUTTON_LABEL); - } else if (button == DialogDelegate::DIALOGBUTTON_CANCEL) { + } else if (button == MessageBoxFlags::DIALOGBUTTON_CANCEL) { return l10n_util::GetString( IDS_BEFOREUNLOAD_MESSAGEBOX_CANCEL_BUTTON_LABEL); } diff --git a/chrome/browser/views/jsmessage_box_dialog.h b/chrome/browser/views/jsmessage_box_dialog.h index 7195904..e86e8ac 100644 --- a/chrome/browser/views/jsmessage_box_dialog.h +++ b/chrome/browser/views/jsmessage_box_dialog.h @@ -37,7 +37,8 @@ class JavascriptMessageBoxDialog : public views::DialogDelegate { virtual void DeleteDelegate(); virtual bool Cancel(); virtual bool Accept(); - virtual std::wstring GetDialogButtonLabel(DialogButton button) const; + virtual std::wstring GetDialogButtonLabel( + MessageBoxFlags::DialogButton button) const; // views::WindowDelegate Methods: virtual bool IsModal() const { return true; } diff --git a/chrome/browser/views/keyword_editor_view.cc b/chrome/browser/views/keyword_editor_view.cc index 4a4f064..b972144 100644 --- a/chrome/browser/views/keyword_editor_view.cc +++ b/chrome/browser/views/keyword_editor_view.cc @@ -444,7 +444,7 @@ std::wstring KeywordEditorView::GetWindowTitle() const { } int KeywordEditorView::GetDialogButtons() const { - return DIALOGBUTTON_CANCEL; + return MessageBoxFlags::DIALOGBUTTON_CANCEL; } bool KeywordEditorView::Accept() { diff --git a/chrome/browser/views/new_profile_dialog.cc b/chrome/browser/views/new_profile_dialog.cc index fbb9c8e..cf9551d1 100644 --- a/chrome/browser/views/new_profile_dialog.cc +++ b/chrome/browser/views/new_profile_dialog.cc @@ -30,9 +30,9 @@ NewProfileDialog::NewProfileDialog() { IDS_NEW_PROFILE_DIALOG_LABEL_TEXT); const int kDialogWidth = views::Window::GetLocalizedContentsWidth( IDS_NEW_PROFILE_DIALOG_WIDTH_CHARS); - const int kMessageBoxFlags = MessageBox::kFlagHasOKButton | - MessageBox::kFlagHasCancelButton | - MessageBox::kFlagHasPromptField; + const int kMessageBoxFlags = MessageBoxFlags::kFlagHasOKButton | + MessageBoxFlags::kFlagHasCancelButton | + MessageBoxFlags::kFlagHasPromptField; message_box_view_ = new MessageBoxView(kMessageBoxFlags, message_text.c_str(), std::wstring(), @@ -46,10 +46,6 @@ NewProfileDialog::NewProfileDialog() { NewProfileDialog::~NewProfileDialog() { } -int NewProfileDialog::GetDialogButtons() const { - return DIALOGBUTTON_OK | DIALOGBUTTON_CANCEL; -} - views::View* NewProfileDialog::GetInitiallyFocusedView() { views::TextField* text_box = message_box_view_->text_box(); DCHECK(text_box); @@ -57,8 +53,8 @@ views::View* NewProfileDialog::GetInitiallyFocusedView() { } bool NewProfileDialog::IsDialogButtonEnabled( - DialogButton button) const { - if (button == DIALOGBUTTON_OK) { + MessageBoxFlags::DialogButton button) const { + if (button == MessageBoxFlags::DIALOGBUTTON_OK) { std::wstring profile_name = message_box_view_->GetInputText(); // TODO(munjal): Refactor the function ReplaceIllegalCharacters in // file_util to something that just checks if there are illegal chars diff --git a/chrome/browser/views/new_profile_dialog.h b/chrome/browser/views/new_profile_dialog.h index 868480e..4564ec2 100644 --- a/chrome/browser/views/new_profile_dialog.h +++ b/chrome/browser/views/new_profile_dialog.h @@ -30,9 +30,9 @@ class NewProfileDialog : public views::DialogDelegate, // views::DialogDelegate methods. virtual bool Accept(); - virtual int GetDialogButtons() const; virtual views::View* GetInitiallyFocusedView(); - virtual bool IsDialogButtonEnabled(DialogButton button) const; + virtual bool IsDialogButtonEnabled( + MessageBoxFlags::DialogButton button) const; virtual std::wstring GetWindowTitle() const; virtual void DeleteDelegate(); diff --git a/chrome/browser/views/options/advanced_page_view.cc b/chrome/browser/views/options/advanced_page_view.cc index fcf083b..20d8616 100644 --- a/chrome/browser/views/options/advanced_page_view.cc +++ b/chrome/browser/views/options/advanced_page_view.cc @@ -36,14 +36,12 @@ class ResetDefaultsConfirmBox : public views::DialogDelegate { protected: // views::DialogDelegate - virtual int GetDialogButtons() const { - return DIALOGBUTTON_OK | DIALOGBUTTON_CANCEL; - } - virtual std::wstring GetDialogButtonLabel(DialogButton button) const { + virtual std::wstring GetDialogButtonLabel( + MessageBoxFlags::DialogButton button) const { switch (button) { - case DIALOGBUTTON_OK: + case MessageBoxFlags::DIALOGBUTTON_OK: return l10n_util::GetString(IDS_OPTIONS_RESET_OKLABEL); - case DIALOGBUTTON_CANCEL: + case MessageBoxFlags::DIALOGBUTTON_CANCEL: return l10n_util::GetString(IDS_OPTIONS_RESET_CANCELLABEL); default: break; @@ -69,7 +67,7 @@ class ResetDefaultsConfirmBox : public views::DialogDelegate { const int kDialogWidth = 400; // Also deleted when the window closes. message_box_view_ = new MessageBoxView( - MessageBox::kFlagHasMessage | MessageBox::kFlagHasOKButton, + MessageBoxFlags::kFlagHasMessage | MessageBoxFlags::kFlagHasOKButton, l10n_util::GetString(IDS_OPTIONS_RESET_MESSAGE).c_str(), std::wstring(), kDialogWidth); diff --git a/chrome/browser/views/options/cookies_view.h b/chrome/browser/views/options/cookies_view.h index 2baed44..c05bce7 100644 --- a/chrome/browser/views/options/cookies_view.h +++ b/chrome/browser/views/options/cookies_view.h @@ -55,9 +55,11 @@ class CookiesView : public views::View, UINT flags); // views::WindowDelegate implementation: - virtual int GetDialogButtons() const { return DIALOGBUTTON_CANCEL; } + virtual int GetDialogButtons() const { + return MessageBoxFlags::DIALOGBUTTON_CANCEL; + } virtual views::View* GetInitiallyFocusedView() { - return search_field_; + return search_field_; } virtual bool CanResize() const { return true; } virtual std::wstring GetWindowTitle() const; diff --git a/chrome/browser/views/options/options_window_view.cc b/chrome/browser/views/options/options_window_view.cc index f7f492f..d163640 100644 --- a/chrome/browser/views/options/options_window_view.cc +++ b/chrome/browser/views/options/options_window_view.cc @@ -42,7 +42,9 @@ class OptionsWindowView : public views::View, void ShowOptionsPage(OptionsPage page, OptionsGroup highlight_group); // views::DialogDelegate implementation: - virtual int GetDialogButtons() const { return DIALOGBUTTON_CANCEL; } + virtual int GetDialogButtons() const { + return MessageBoxFlags::DIALOGBUTTON_CANCEL; + } virtual std::wstring GetWindowTitle() const; virtual void WindowClosing(); virtual bool Cancel(); diff --git a/chrome/browser/views/options/passwords_exceptions_window_view.cc b/chrome/browser/views/options/passwords_exceptions_window_view.cc index 9bf44b4..4ea100f 100644 --- a/chrome/browser/views/options/passwords_exceptions_window_view.cc +++ b/chrome/browser/views/options/passwords_exceptions_window_view.cc @@ -65,7 +65,7 @@ void PasswordsExceptionsWindowView::ViewHierarchyChanged( // PasswordsExceptionsWindowView, views::DisloagDelegate implementations int PasswordsExceptionsWindowView::GetDialogButtons() const { - return DIALOGBUTTON_CANCEL; + return MessageBoxFlags::DIALOGBUTTON_CANCEL; } std::wstring PasswordsExceptionsWindowView::GetWindowTitle() const { diff --git a/chrome/browser/views/page_info_window.cc b/chrome/browser/views/page_info_window.cc index ecc3ef8..5fa345e 100644 --- a/chrome/browser/views/page_info_window.cc +++ b/chrome/browser/views/page_info_window.cc @@ -597,7 +597,7 @@ void PageInfoWindow::Show() { } int PageInfoWindow::GetDialogButtons() const { - return DIALOGBUTTON_CANCEL; + return MessageBoxFlags::DIALOGBUTTON_CANCEL; } std::wstring PageInfoWindow::GetWindowTitle() const { diff --git a/chrome/browser/views/repost_form_warning_view.cc b/chrome/browser/views/repost_form_warning_view.cc index b8e9153..97e33fd 100644 --- a/chrome/browser/views/repost_form_warning_view.cc +++ b/chrome/browser/views/repost_form_warning_view.cc @@ -25,7 +25,7 @@ RepostFormWarningView::RepostFormWarningView( : navigation_controller_(navigation_controller), message_box_view_(NULL) { message_box_view_ = new MessageBoxView( - MessageBox::kIsConfirmMessageBox, + MessageBoxFlags::kIsConfirmMessageBox, l10n_util::GetString(IDS_HTTP_POST_WARNING), L""); // TODO(beng): fix this - this dialog box should be shown by a method on the @@ -57,10 +57,10 @@ std::wstring RepostFormWarningView::GetWindowTitle() const { } std::wstring RepostFormWarningView::GetDialogButtonLabel( - DialogButton button) const { - if (button == DialogDelegate::DIALOGBUTTON_OK) + MessageBoxFlags::DialogButton button) const { + if (button == MessageBoxFlags::DIALOGBUTTON_OK) return l10n_util::GetString(IDS_HTTP_POST_WARNING_RESEND); - if (button == DialogDelegate::DIALOGBUTTON_CANCEL) + if (button == MessageBoxFlags::DIALOGBUTTON_CANCEL) return l10n_util::GetString(IDS_HTTP_POST_WARNING_CANCEL); return L""; } diff --git a/chrome/browser/views/repost_form_warning_view.h b/chrome/browser/views/repost_form_warning_view.h index a653922..809b7b5 100644 --- a/chrome/browser/views/repost_form_warning_view.h +++ b/chrome/browser/views/repost_form_warning_view.h @@ -23,7 +23,8 @@ class RepostFormWarningView : public views::DialogDelegate, // views::DialogDelegate Methods: virtual std::wstring GetWindowTitle() const; - virtual std::wstring GetDialogButtonLabel(DialogButton button) const; + virtual std::wstring GetDialogButtonLabel( + MessageBoxFlags::DialogButton button) const; virtual void DeleteDelegate(); virtual bool Cancel(); virtual bool Accept(); diff --git a/chrome/browser/views/restart_message_box.cc b/chrome/browser/views/restart_message_box.cc index e61e23f..e9728cb 100644 --- a/chrome/browser/views/restart_message_box.cc +++ b/chrome/browser/views/restart_message_box.cc @@ -21,12 +21,12 @@ void RestartMessageBox::ShowMessageBox(HWND parent_hwnd) { } int RestartMessageBox::GetDialogButtons() const { - return DialogDelegate::DIALOGBUTTON_OK; + return MessageBoxFlags::DIALOGBUTTON_OK; } -std::wstring RestartMessageBox::GetDialogButtonLabel(DialogButton button) - const { - DCHECK(button == DIALOGBUTTON_OK); +std::wstring RestartMessageBox::GetDialogButtonLabel( + MessageBoxFlags::DialogButton button) const { + DCHECK(button == MessageBoxFlags::DIALOGBUTTON_OK); return l10n_util::GetString(IDS_OK); } @@ -53,7 +53,7 @@ RestartMessageBox::RestartMessageBox(HWND parent_hwnd) { const int kDialogWidth = 400; // Also deleted when the window closes. message_box_view_ = new MessageBoxView( - MessageBox::kFlagHasMessage | MessageBox::kFlagHasOKButton, + MessageBoxFlags::kFlagHasMessage | MessageBoxFlags::kFlagHasOKButton, l10n_util::GetString(IDS_OPTIONS_RESTART_REQUIRED).c_str(), std::wstring(), kDialogWidth); diff --git a/chrome/browser/views/restart_message_box.h b/chrome/browser/views/restart_message_box.h index 893bf43..8180de4 100644 --- a/chrome/browser/views/restart_message_box.h +++ b/chrome/browser/views/restart_message_box.h @@ -20,7 +20,8 @@ class RestartMessageBox : public views::DialogDelegate { protected: // views::DialogDelegate: virtual int GetDialogButtons() const; - virtual std::wstring GetDialogButtonLabel(DialogButton button) const; + virtual std::wstring GetDialogButtonLabel( + MessageBoxFlags::DialogButton button) const; virtual std::wstring GetWindowTitle() const; // views::WindowDelegate: diff --git a/chrome/browser/views/select_profile_dialog.cc b/chrome/browser/views/select_profile_dialog.cc index b10c262..3a5c6d2 100644 --- a/chrome/browser/views/select_profile_dialog.cc +++ b/chrome/browser/views/select_profile_dialog.cc @@ -66,10 +66,6 @@ void SelectProfileDialog::Layout() { GetLayoutManager()->Layout(this); } -int SelectProfileDialog::GetDialogButtons() const { - return DIALOGBUTTON_OK | DIALOGBUTTON_CANCEL; -} - views::View* SelectProfileDialog::GetInitiallyFocusedView() { return profile_combobox_; } diff --git a/chrome/browser/views/select_profile_dialog.h b/chrome/browser/views/select_profile_dialog.h index a761623..723efb2 100644 --- a/chrome/browser/views/select_profile_dialog.h +++ b/chrome/browser/views/select_profile_dialog.h @@ -50,7 +50,6 @@ class SelectProfileDialog virtual bool Accept(); virtual bool Cancel(); virtual views::View* GetContentsView(); - virtual int GetDialogButtons() const; virtual views::View* GetInitiallyFocusedView(); virtual std::wstring GetWindowTitle() const; virtual bool IsModal() const { return false; } diff --git a/chrome/browser/views/shelf_item_dialog.cc b/chrome/browser/views/shelf_item_dialog.cc index 68050b1..bb05c44 100644 --- a/chrome/browser/views/shelf_item_dialog.cc +++ b/chrome/browser/views/shelf_item_dialog.cc @@ -375,8 +375,9 @@ bool ShelfItemDialog::IsModal() const { return true; } -std::wstring ShelfItemDialog::GetDialogButtonLabel(DialogButton button) const { - if (button == DialogDelegate::DIALOGBUTTON_OK) +std::wstring ShelfItemDialog::GetDialogButtonLabel( + MessageBoxFlags::DialogButton button) const { + if (button == MessageBoxFlags::DIALOGBUTTON_OK) return l10n_util::GetString(IDS_ASI_ADD); return std::wstring(); } @@ -426,7 +427,7 @@ void ShelfItemDialog::ContentsChanged(views::TextField* sender, } bool ShelfItemDialog::Accept() { - if (!IsDialogButtonEnabled(DIALOGBUTTON_OK)) { + if (!IsDialogButtonEnabled(MessageBoxFlags::DIALOGBUTTON_OK)) { if (!GetInputURL().is_valid()) url_field_->RequestFocus(); else if (title_field_) @@ -437,8 +438,9 @@ bool ShelfItemDialog::Accept() { return true; } -bool ShelfItemDialog::IsDialogButtonEnabled(DialogButton button) const { - if (button == DIALOGBUTTON_OK) +bool ShelfItemDialog::IsDialogButtonEnabled( + MessageBoxFlags::DialogButton button) const { + if (button == MessageBoxFlags::DIALOGBUTTON_OK) return GetInputURL().is_valid(); return true; } diff --git a/chrome/browser/views/shelf_item_dialog.h b/chrome/browser/views/shelf_item_dialog.h index 9e5520d..e1c932e 100644 --- a/chrome/browser/views/shelf_item_dialog.h +++ b/chrome/browser/views/shelf_item_dialog.h @@ -59,9 +59,10 @@ class ShelfItemDialog : public views::View, // DialogDelegate. virtual std::wstring GetWindowTitle() const; virtual bool IsModal() const; - virtual std::wstring GetDialogButtonLabel(DialogButton button) const; + virtual std::wstring GetDialogButtonLabel( + MessageBoxFlags::DialogButton button) const; virtual bool Accept(); - virtual bool IsDialogButtonEnabled(DialogButton button) const; + virtual bool IsDialogButtonEnabled(MessageBoxFlags::DialogButton button) const; virtual views::View* GetContentsView(); // TextField::Controller. diff --git a/chrome/browser/views/uninstall_dialog.cc b/chrome/browser/views/uninstall_dialog.cc index da54ca8..51e7023 100644 --- a/chrome/browser/views/uninstall_dialog.cc +++ b/chrome/browser/views/uninstall_dialog.cc @@ -30,10 +30,6 @@ bool UninstallDialog::Cancel() { return true; } -int UninstallDialog::GetDialogButtons() const { - return DIALOGBUTTON_OK | DIALOGBUTTON_CANCEL; -} - std::wstring UninstallDialog::GetWindowTitle() const { return l10n_util::GetString(IDS_UNINSTALL_CHROME); } @@ -50,7 +46,8 @@ UninstallDialog::UninstallDialog(int& user_selection) : user_selection_(user_selection) { // Also deleted when the window closes. message_box_view_ = new MessageBoxView( - MessageBox::kIsConfirmMessageBox | MessageBox::kAutoDetectAlignment, + MessageBoxFlags::kIsConfirmMessageBox | + MessageBoxFlags::kAutoDetectAlignment, l10n_util::GetString(IDS_UNINSTALL_VERIFY).c_str(), std::wstring()); message_box_view_->SetCheckBoxLabel( diff --git a/chrome/browser/views/uninstall_dialog.h b/chrome/browser/views/uninstall_dialog.h index 0874db6..bf9a6c9 100644 --- a/chrome/browser/views/uninstall_dialog.h +++ b/chrome/browser/views/uninstall_dialog.h @@ -20,7 +20,6 @@ class UninstallDialog : public views::DialogDelegate { // Overridden from views::DialogDelegate: virtual bool Accept(); virtual bool Cancel(); - virtual int GetDialogButtons() const; virtual std::wstring GetWindowTitle() const; // Overridden from views::WindowDelegate: diff --git a/chrome/browser/views/user_data_dir_dialog.cc b/chrome/browser/views/user_data_dir_dialog.cc index 3598950..79bcd4e 100644 --- a/chrome/browser/views/user_data_dir_dialog.cc +++ b/chrome/browser/views/user_data_dir_dialog.cc @@ -27,7 +27,7 @@ UserDataDirDialog::UserDataDirDialog(const std::wstring& user_data_dir) std::wstring message_text = l10n_util::GetStringF( IDS_CANT_WRITE_USER_DIRECTORY_SUMMARY, user_data_dir); const int kDialogWidth = 400; - message_box_view_ = new MessageBoxView(MessageBox::kIsConfirmMessageBox, + message_box_view_ = new MessageBoxView(MessageBoxFlags::kIsConfirmMessageBox, message_text.c_str(), std::wstring(), kDialogWidth); views::Window::CreateChromeWindow(NULL, gfx::Rect(), this)->Show(); @@ -37,19 +37,14 @@ UserDataDirDialog::~UserDataDirDialog() { select_file_dialog_->ListenerDestroyed(); } -int UserDataDirDialog::GetDialogButtons() const { - return DIALOGBUTTON_OK | DIALOGBUTTON_CANCEL; - -} - std::wstring UserDataDirDialog::GetDialogButtonLabel( - DialogButton button) const { + MessageBoxFlags::DialogButton button) const { switch (button) { - case DIALOGBUTTON_OK: + case MessageBoxFlags::DIALOGBUTTON_OK: return l10n_util::GetString( IDS_CANT_WRITE_USER_DIRECTORY_CHOOSE_DIRECTORY_BUTTON); - case DIALOGBUTTON_CANCEL: + case MessageBoxFlags::DIALOGBUTTON_CANCEL: return l10n_util::GetString(IDS_CANT_WRITE_USER_DIRECTORY_EXIT_BUTTON); default: NOTREACHED(); diff --git a/chrome/browser/views/user_data_dir_dialog.h b/chrome/browser/views/user_data_dir_dialog.h index 78192ce..7408132 100644 --- a/chrome/browser/views/user_data_dir_dialog.h +++ b/chrome/browser/views/user_data_dir_dialog.h @@ -32,8 +32,8 @@ class UserDataDirDialog : public views::DialogDelegate, std::wstring user_data_dir() { return user_data_dir_; } // views::DialogDelegate Methods: - virtual int GetDialogButtons() const; - virtual std::wstring GetDialogButtonLabel(DialogButton button) const; + virtual std::wstring GetDialogButtonLabel( + MessageBoxFlags::DialogButton button) const; virtual std::wstring GetWindowTitle() const; virtual void DeleteDelegate(); virtual bool Accept(); |