From 3549b9a1e5b4698865c91458e6255231fdc4cba9 Mon Sep 17 00:00:00 2001 From: vadimt Date: Fri, 16 Jan 2015 17:28:06 -0800 Subject: Instrumenting all wndprocs to find jank. PeekMessage calls are still the top source of jank. The theory is that they invoke some synchronous message processing, and this causes jank. I realized that I didn't instrument all wndprocs, which I'm doing now. BUG=440919 Review URL: https://codereview.chromium.org/850213002 Cr-Commit-Position: refs/heads/master@{#312004} --- content/browser/renderer_host/render_widget_host_view_base.cc | 5 +++++ 1 file changed, 5 insertions(+) (limited to 'content/browser/renderer_host/render_widget_host_view_base.cc') diff --git a/content/browser/renderer_host/render_widget_host_view_base.cc b/content/browser/renderer_host/render_widget_host_view_base.cc index 3aacfdf..d75d89a 100644 --- a/content/browser/renderer_host/render_widget_host_view_base.cc +++ b/content/browser/renderer_host/render_widget_host_view_base.cc @@ -5,6 +5,7 @@ #include "content/browser/renderer_host/render_widget_host_view_base.h" #include "base/logging.h" +#include "base/profiler/scoped_tracker.h" #include "content/browser/accessibility/browser_accessibility_manager.h" #include "content/browser/gpu/gpu_data_manager_impl.h" #include "content/browser/renderer_host/input/synthetic_gesture_target_base.h" @@ -83,6 +84,10 @@ void NotifyPluginProcessHostHelper(HWND window, HWND parent, int tries) { // parent which is typically the RVH window which turns on user gesture. LRESULT CALLBACK PluginWrapperWindowProc(HWND window, unsigned int message, WPARAM wparam, LPARAM lparam) { + // TODO(vadimt): Remove ScopedTracker below once crbug.com/440919 is fixed. + tracked_objects::ScopedTracker tracking_profile( + FROM_HERE_WITH_EXPLICIT_FUNCTION("440919 PluginWrapperWindowProc")); + if (message == WM_PARENTNOTIFY) { switch (LOWORD(wparam)) { case WM_LBUTTONDOWN: -- cgit v1.1