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

Unified Diff: ui/keyboard/content/keyboard_ui_content.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 | « ash/virtual_keyboard_controller.cc ('k') | ui/keyboard/keyboard_controller.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/keyboard/content/keyboard_ui_content.cc
diff --git a/ui/keyboard/content/keyboard_ui_content.cc b/ui/keyboard/content/keyboard_ui_content.cc
index c98343d0a89b3801c1406f9f62f65c513ce51437..9f156738e89aaf20c74cf8429b34efdfbf920cde 100644
--- a/ui/keyboard/content/keyboard_ui_content.cc
+++ b/ui/keyboard/content/keyboard_ui_content.cc
@@ -240,6 +240,7 @@ void KeyboardUIContent::ReloadKeyboardIfNeeded() {
// navigate to a keyboard in a different extension. This keeps the UX the
// same as Android. Note we need to explicitly close current page as it
// might try to resize keyboard window in javascript on a resize event.
+ TRACE_EVENT0("vk", "ReloadKeyboardIfNeeded");
GetKeyboardWindow()->SetBounds(gfx::Rect());
keyboard_contents_->ClosePage();
keyboard_controller()->SetKeyboardMode(FULL_WIDTH);
@@ -325,6 +326,7 @@ const aura::Window* KeyboardUIContent::GetKeyboardRootWindow() const {
void KeyboardUIContent::LoadContents(const GURL& url) {
if (keyboard_contents_) {
+ TRACE_EVENT0("vk", "LoadContents");
content::OpenURLParams params(url, content::Referrer(),
WindowOpenDisposition::SINGLETON_TAB,
ui::PAGE_TRANSITION_AUTO_TOPLEVEL, false);
« no previous file with comments | « ash/virtual_keyboard_controller.cc ('k') | ui/keyboard/keyboard_controller.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698