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

Unified Diff: chrome/browser/ui/touch/keyboard/keyboard_manager.cc

Issue 7212029: Add a browser-test for the keyboard widget. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 9 years, 5 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
Index: chrome/browser/ui/touch/keyboard/keyboard_manager.cc
diff --git a/chrome/browser/ui/touch/keyboard/keyboard_manager.cc b/chrome/browser/ui/touch/keyboard/keyboard_manager.cc
index c21f20522b46f2e308063ddc7c36b8f4e817b3d9..c5442918d3c06018a2c6797b4ac4e913a1217c3f 100644
--- a/chrome/browser/ui/touch/keyboard/keyboard_manager.cc
+++ b/chrome/browser/ui/touch/keyboard/keyboard_manager.cc
@@ -146,10 +146,22 @@ void KeyboardManager::ShowKeyboardForWidget(views::Widget* widget) {
MoveToTop();
Show();
+
+ bool visible = true;
+ NotificationService::current()->Notify(
+ NotificationType::KEYBOARD_VISIBILITY_CHANGED,
+ Source<KeyboardManager>(this),
+ Details<bool>(&visible));
}
void KeyboardManager::Hide() {
animation_->Hide();
+
+ bool visible = false;
+ NotificationService::current()->Notify(
+ NotificationType::KEYBOARD_VISIBILITY_CHANGED,
+ Source<KeyboardManager>(this),
+ Details<bool>(&visible));
}
bool KeyboardManager::OnKeyEvent(const views::KeyEvent& event) {

Powered by Google App Engine
This is Rietveld 408576698