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

Unified Diff: ui/keyboard/keyboard_controller.cc

Issue 2923493002: Add TRACE_EVENT markers to various keyboard functions (Closed)
Patch Set: more trace markers Created 3 years, 6 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 | « ui/keyboard/content/keyboard_ui_content.cc ('k') | ui/keyboard/keyboard_layout_manager.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/keyboard/keyboard_controller.cc
diff --git a/ui/keyboard/keyboard_controller.cc b/ui/keyboard/keyboard_controller.cc
index 5e88e8ea7a6af42c90907e8e72d3daae0c00edb4..4b68c515a8cb2f79d52e3f60cf5f95f12284ee16 100644
--- a/ui/keyboard/keyboard_controller.cc
+++ b/ui/keyboard/keyboard_controller.cc
@@ -246,6 +246,8 @@ void KeyboardController::NotifyKeyboardBoundsChanging(
}
void KeyboardController::HideKeyboard(HideReason reason) {
+ TRACE_EVENT0("vk", "HideKeyboard");
+
keyboard_visible_ = false;
ToggleTouchEventLogging(true);
@@ -377,6 +379,8 @@ void KeyboardController::OnTextInputStateChanged(
if (!container_.get())
return;
+ TRACE_EVENT0("vk", "OnTextInputStateChanged");
+
ui::TextInputType type =
client ? client->GetTextInputType() : ui::TEXT_INPUT_TYPE_NONE;
@@ -423,6 +427,8 @@ void KeyboardController::ShowKeyboardInternal(int64_t display_id) {
// |Shell::CreateKeyboard| is called.
DCHECK(container_.get());
+ TRACE_EVENT0("vk", "ShowKeyboardInternal");
+
if (container_->children().empty()) {
keyboard::MarkKeyboardLoadStarted();
aura::Window* keyboard = ui_->GetKeyboardWindow();
« no previous file with comments | « ui/keyboard/content/keyboard_ui_content.cc ('k') | ui/keyboard/keyboard_layout_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698