diff options
Diffstat (limited to 'chrome')
-rw-r--r-- | chrome/browser/input_window_dialog_win.cc | 11 | ||||
-rw-r--r-- | chrome/browser/views/bookmark_editor_view.h | 5 | ||||
-rw-r--r-- | chrome/browser/views/bookmark_manager_view.cc | 9 | ||||
-rw-r--r-- | chrome/browser/views/bookmark_manager_view.h | 3 | ||||
-rw-r--r-- | chrome/browser/views/bug_report_view.cc | 3 | ||||
-rw-r--r-- | chrome/browser/views/bug_report_view.h | 3 | ||||
-rw-r--r-- | chrome/browser/views/edit_keyword_controller.cc | 8 | ||||
-rw-r--r-- | chrome/browser/views/edit_keyword_controller.h | 5 | ||||
-rw-r--r-- | chrome/browser/views/find_bar_view.cc | 28 | ||||
-rw-r--r-- | chrome/browser/views/find_bar_view.h | 4 | ||||
-rw-r--r-- | chrome/browser/views/frame/browser_view.h | 1 | ||||
-rw-r--r-- | chrome/browser/views/new_profile_dialog.h | 5 | ||||
-rw-r--r-- | chrome/browser/views/options/cookies_view.cc | 17 | ||||
-rw-r--r-- | chrome/browser/views/options/cookies_view.h | 3 | ||||
-rw-r--r-- | chrome/browser/views/options/general_page_view.cc | 3 | ||||
-rw-r--r-- | chrome/browser/views/options/general_page_view.h | 5 | ||||
-rw-r--r-- | chrome/browser/views/shelf_item_dialog.h | 5 |
17 files changed, 53 insertions, 65 deletions
diff --git a/chrome/browser/input_window_dialog_win.cc b/chrome/browser/input_window_dialog_win.cc index 2da360a..3f7b641 100644 --- a/chrome/browser/input_window_dialog_win.cc +++ b/chrome/browser/input_window_dialog_win.cc @@ -78,7 +78,8 @@ class ContentView : public views::View, // views::TextField::Controller overrides: virtual void ContentsChanged(views::TextField* sender, const std::wstring& new_contents); - virtual bool HandleKeystroke(views::TextField*, UINT, TCHAR, UINT, UINT) { + virtual bool HandleKeystroke(views::TextField*, + const views::TextField::Keystroke&) { return false; } @@ -197,10 +198,10 @@ void ContentView::FocusFirstFocusableControl() { } WinInputWindowDialog::WinInputWindowDialog(HWND parent, - const std::wstring& window_title, - const std::wstring& label, - const std::wstring& contents, - Delegate* delegate) + const std::wstring& window_title, + const std::wstring& label, + const std::wstring& contents, + Delegate* delegate) : window_title_(window_title), label_(label), contents_(contents), diff --git a/chrome/browser/views/bookmark_editor_view.h b/chrome/browser/views/bookmark_editor_view.h index 754e5f8..055b1a8 100644 --- a/chrome/browser/views/bookmark_editor_view.h +++ b/chrome/browser/views/bookmark_editor_view.h @@ -84,8 +84,9 @@ class BookmarkEditorView : public BookmarkEditor, virtual void ContentsChanged(views::TextField* sender, const std::wstring& new_contents); virtual bool HandleKeystroke(views::TextField* sender, - UINT message, TCHAR key, UINT repeat_count, - UINT flags) { return false; } + const views::TextField::Keystroke&) { + return false; + } // NativeButton. virtual void ButtonPressed(views::Button* sender); diff --git a/chrome/browser/views/bookmark_manager_view.cc b/chrome/browser/views/bookmark_manager_view.cc index 8a0b019..25cddc0 100644 --- a/chrome/browser/views/bookmark_manager_view.cc +++ b/chrome/browser/views/bookmark_manager_view.cc @@ -455,11 +455,10 @@ void BookmarkManagerView::ContentsChanged(views::TextField* sender, kSearchDelayMS); } -bool BookmarkManagerView::HandleKeystroke(views::TextField* sender, - UINT message, TCHAR key, - UINT repeat_count, - UINT flags) { - if (key == VK_RETURN) { +bool BookmarkManagerView::HandleKeystroke( + views::TextField* sender, + const views::TextField::Keystroke& key) { + if (views::TextField::IsKeystrokeEnter(key)) { PerformSearch(); search_tf_->SelectAll(); } diff --git a/chrome/browser/views/bookmark_manager_view.h b/chrome/browser/views/bookmark_manager_view.h index 9c17571..3c73375 100644 --- a/chrome/browser/views/bookmark_manager_view.h +++ b/chrome/browser/views/bookmark_manager_view.h @@ -137,8 +137,7 @@ class BookmarkManagerView : public views::View, const std::wstring& new_contents); // If return has been pressed this performs an immediate search. virtual bool HandleKeystroke(views::TextField* sender, - UINT message, TCHAR key, UINT repeat_count, - UINT flags); + const views::TextField::Keystroke& key); // ContextMenuController. virtual void ShowContextMenu(views::View* source, diff --git a/chrome/browser/views/bug_report_view.cc b/chrome/browser/views/bug_report_view.cc index 3af66f4..e4bdd7d 100644 --- a/chrome/browser/views/bug_report_view.cc +++ b/chrome/browser/views/bug_report_view.cc @@ -261,8 +261,7 @@ void BugReportView::ContentsChanged(views::TextField* sender, } bool BugReportView::HandleKeystroke(views::TextField* sender, - UINT message, TCHAR key, - UINT repeat_count, UINT flags) { + const views::TextField::Keystroke& key) { return false; } diff --git a/chrome/browser/views/bug_report_view.h b/chrome/browser/views/bug_report_view.h index 0688a1dc..9edf1f0 100644 --- a/chrome/browser/views/bug_report_view.h +++ b/chrome/browser/views/bug_report_view.h @@ -53,8 +53,7 @@ class BugReportView : public views::View, virtual void ContentsChanged(views::TextField* sender, const std::wstring& new_contents); virtual bool HandleKeystroke(views::TextField* sender, - UINT message, TCHAR key, - UINT repeat_count, UINT flags); + const views::TextField::Keystroke& key); // views::ComboBox::Listener implementation: virtual void ItemChanged(views::ComboBox* combo_box, int prev_index, diff --git a/chrome/browser/views/edit_keyword_controller.cc b/chrome/browser/views/edit_keyword_controller.cc index 2774379..78434e5 100644 --- a/chrome/browser/views/edit_keyword_controller.cc +++ b/chrome/browser/views/edit_keyword_controller.cc @@ -158,11 +158,9 @@ void EditKeywordController::ContentsChanged(TextField* sender, UpdateImageViews(); } -bool EditKeywordController::HandleKeystroke(TextField* sender, - UINT message, - TCHAR key, - UINT repeat_count, - UINT flags) { +bool EditKeywordController::HandleKeystroke( + TextField* sender, + const views::TextField::Keystroke& key) { return false; } diff --git a/chrome/browser/views/edit_keyword_controller.h b/chrome/browser/views/edit_keyword_controller.h index 36e9370..baf7efd 100644 --- a/chrome/browser/views/edit_keyword_controller.h +++ b/chrome/browser/views/edit_keyword_controller.h @@ -57,10 +57,7 @@ class EditKeywordController : public views::TextField::Controller, virtual void ContentsChanged(views::TextField* sender, const std::wstring& new_contents); virtual bool HandleKeystroke(views::TextField* sender, - UINT message, - TCHAR key, - UINT repeat_count, - UINT flags); + const views::TextField::Keystroke& key); private: void Init(); diff --git a/chrome/browser/views/find_bar_view.cc b/chrome/browser/views/find_bar_view.cc index ededa28..063ea63 100644 --- a/chrome/browser/views/find_bar_view.cc +++ b/chrome/browser/views/find_bar_view.cc @@ -456,31 +456,29 @@ void FindBarView::ContentsChanged(views::TextField* sender, } } -bool FindBarView::HandleKeystroke(views::TextField* sender, UINT message, - TCHAR key, UINT repeat_count, UINT flags) { +bool FindBarView::HandleKeystroke(views::TextField* sender, + const views::TextField::Keystroke& key) { // If the dialog is not visible, there is no reason to process keyboard input. if (!container_->IsVisible()) return false; // TODO(port): Handle this for other platforms. #if defined(OS_WIN) - if (container_->MaybeForwardKeystrokeToWebpage(message, key, flags)) + if (container_->MaybeForwardKeystrokeToWebpage(key.message, key.key, + key.flags)) return true; // Handled, we are done! - #endif - switch (key) { - case VK_RETURN: { - // Pressing Return/Enter starts the search (unless text box is empty). - std::wstring find_string = find_text_->GetText(); - if (!find_string.empty()) { - // Search forwards for enter, backwards for shift-enter. - container_->GetFindBarController()->tab_contents()->StartFinding( - find_string, - GetKeyState(VK_SHIFT) >= 0); - } - break; + if (views::TextField::IsKeystrokeEnter(key)) { + // Pressing Return/Enter starts the search (unless text box is empty). + std::wstring find_string = find_text_->GetText(); + if (!find_string.empty()) { + // Search forwards for enter, backwards for shift-enter. + container_->GetFindBarController()->tab_contents()->StartFinding( + find_string, + GetKeyState(VK_SHIFT) >= 0); } } + #endif return false; } diff --git a/chrome/browser/views/find_bar_view.h b/chrome/browser/views/find_bar_view.h index e56af0e..00ba8fe1 100644 --- a/chrome/browser/views/find_bar_view.h +++ b/chrome/browser/views/find_bar_view.h @@ -67,8 +67,8 @@ class FindBarView : public views::View, // Overridden from views::TextField::Controller: virtual void ContentsChanged(views::TextField* sender, const std::wstring& new_contents); - virtual bool HandleKeystroke(views::TextField* sender, UINT message, - TCHAR key, UINT repeat_count, UINT flags); + virtual bool HandleKeystroke(views::TextField* sender, + const views::TextField::Keystroke& key); // Set whether or not we're attempting to blend with the toolbar. void set_toolbar_blend(bool toolbar_blend) { toolbar_blend_ = toolbar_blend; } diff --git a/chrome/browser/views/frame/browser_view.h b/chrome/browser/views/frame/browser_view.h index fcb7b4d..e1f71f6 100644 --- a/chrome/browser/views/frame/browser_view.h +++ b/chrome/browser/views/frame/browser_view.h @@ -7,6 +7,7 @@ #include <set> +#include "build/build_config.h" #include "chrome/browser/browser.h" #include "chrome/browser/browser_window.h" #include "chrome/browser/tabs/tab_strip_model.h" diff --git a/chrome/browser/views/new_profile_dialog.h b/chrome/browser/views/new_profile_dialog.h index 863b2ad..a06729d 100644 --- a/chrome/browser/views/new_profile_dialog.h +++ b/chrome/browser/views/new_profile_dialog.h @@ -40,8 +40,9 @@ class NewProfileDialog : public views::DialogDelegate, virtual void ContentsChanged(views::TextField* sender, const std::wstring& new_contents); virtual bool HandleKeystroke(views::TextField* sender, - UINT message, TCHAR key, UINT repeat_count, - UINT flags) { return false; } + const views::TextField::Keystroke& key) { + return false; + } // views::WindowDelegate methods. virtual views::View* GetContentsView(); diff --git a/chrome/browser/views/options/cookies_view.cc b/chrome/browser/views/options/cookies_view.cc index f937c6d..ac5e495 100644 --- a/chrome/browser/views/options/cookies_view.cc +++ b/chrome/browser/views/options/cookies_view.cc @@ -620,16 +620,13 @@ void CookiesView::ContentsChanged(views::TextField* sender, &CookiesView::UpdateSearchResults), kSearchFilterDelayMs); } -bool CookiesView::HandleKeystroke(views::TextField* sender, UINT message, - TCHAR key, UINT repeat_count, UINT flags) { - switch (key) { - case VK_ESCAPE: - ResetSearchQuery(); - break; - case VK_RETURN: - search_update_factory_.RevokeAll(); - UpdateSearchResults(); - break; +bool CookiesView::HandleKeystroke(views::TextField* sender, + const views::TextField::Keystroke& key) { + if (views::TextField::IsKeystrokeEscape(key)) { + ResetSearchQuery(); + } else if (views::TextField::IsKeystrokeEnter(key)) { + search_update_factory_.RevokeAll(); + UpdateSearchResults(); } return false; } diff --git a/chrome/browser/views/options/cookies_view.h b/chrome/browser/views/options/cookies_view.h index f2bc98a..7de565d 100644 --- a/chrome/browser/views/options/cookies_view.h +++ b/chrome/browser/views/options/cookies_view.h @@ -51,8 +51,7 @@ class CookiesView : public views::View, virtual void ContentsChanged(views::TextField* sender, const std::wstring& new_contents); virtual bool HandleKeystroke(views::TextField* sender, - UINT message, TCHAR key, UINT repeat_count, - UINT flags); + const views::TextField::Keystroke& key); // views::WindowDelegate implementation: virtual int GetDialogButtons() const { diff --git a/chrome/browser/views/options/general_page_view.cc b/chrome/browser/views/options/general_page_view.cc index 7c54d5c..35af571 100644 --- a/chrome/browser/views/options/general_page_view.cc +++ b/chrome/browser/views/options/general_page_view.cc @@ -619,8 +619,7 @@ void GeneralPageView::ContentsChanged(views::TextField* sender, } bool GeneralPageView::HandleKeystroke(views::TextField* sender, - UINT message, TCHAR key, - UINT repeat_count, UINT flags) { + const views::TextField::Keystroke&) { return false; } diff --git a/chrome/browser/views/options/general_page_view.h b/chrome/browser/views/options/general_page_view.h index 800cd04..1e2d72f 100644 --- a/chrome/browser/views/options/general_page_view.h +++ b/chrome/browser/views/options/general_page_view.h @@ -50,10 +50,9 @@ class GeneralPageView : public OptionsPageView, // views::TextField::Controller implementation: virtual void ContentsChanged(views::TextField* sender, - const std::wstring& new_contents); + const std::wstring& new_contents); virtual bool HandleKeystroke(views::TextField* sender, - UINT message, TCHAR key, UINT repeat_count, - UINT flags); + const views::TextField::Keystroke& key); // OptionsPageView implementation: virtual void InitControlLayout(); diff --git a/chrome/browser/views/shelf_item_dialog.h b/chrome/browser/views/shelf_item_dialog.h index 5e49625..9ac740d 100644 --- a/chrome/browser/views/shelf_item_dialog.h +++ b/chrome/browser/views/shelf_item_dialog.h @@ -69,8 +69,9 @@ class ShelfItemDialog : public views::View, virtual void ContentsChanged(views::TextField* sender, const std::wstring& new_contents); virtual bool HandleKeystroke(views::TextField* sender, - UINT message, TCHAR key, UINT repeat_count, - UINT flags) { return false; } + const views::TextField::Keystroke& key) { + return false; + } // Overridden from View. virtual gfx::Size GetPreferredSize(); |