Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(119)

Unified Diff: chrome/browser/ui/views/find_bar_host.cc

Issue 664223002: MacViews: Get views based FindBar to compile on the Mac. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Update guards Created 6 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/ui/views/find_bar_host.h ('k') | chrome/browser/ui/views/find_bar_host_aura.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/views/find_bar_host.cc
diff --git a/chrome/browser/ui/views/find_bar_host.cc b/chrome/browser/ui/views/find_bar_host.cc
index cdec6d827671889443ce4be7aeeb8c39893b1c0d..36604228e7fc3750f0d461d2629c898f623d2907 100644
--- a/chrome/browser/ui/views/find_bar_host.cc
+++ b/chrome/browser/ui/views/find_bar_host.cc
@@ -46,11 +46,6 @@ FindBarHost::~FindBarHost() {
bool FindBarHost::MaybeForwardKeyEventToWebpage(
const ui::KeyEvent& key_event) {
- if (!ShouldForwardKeyEventToWebpageNative(key_event)) {
- // Native implementation says not to forward these events.
- return false;
- }
-
switch (key_event.key_code()) {
case ui::VKEY_DOWN:
case ui::VKEY_UP:
@@ -161,6 +156,12 @@ void FindBarHost::UpdateUIForFindResult(const FindNotificationDetails& result,
ResetFocusTracker();
}
+void FindBarHost::AudibleAlert() {
+#if defined(OS_WIN)
+ MessageBeep(MB_OK);
+#endif
+}
+
bool FindBarHost::IsFindBarVisible() {
return DropdownBarHost::IsVisible();
}
« no previous file with comments | « chrome/browser/ui/views/find_bar_host.h ('k') | chrome/browser/ui/views/find_bar_host_aura.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698