diff options
| author | vadimt <vadimt@chromium.org> | 2015-01-16 17:28:06 -0800 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-01-17 01:29:55 +0000 |
| commit | 3549b9a1e5b4698865c91458e6255231fdc4cba9 (patch) | |
| tree | c7fda936c86234fa218b47449bc0a3e75467e2d4 /content/browser/renderer_host/render_widget_host_view_base.cc | |
| parent | 8ce46c7881a3296187c7dd27afb581f84f4b27a7 (diff) | |
| download | chromium_src-3549b9a1e5b4698865c91458e6255231fdc4cba9.zip chromium_src-3549b9a1e5b4698865c91458e6255231fdc4cba9.tar.gz chromium_src-3549b9a1e5b4698865c91458e6255231fdc4cba9.tar.bz2 | |
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}
Diffstat (limited to 'content/browser/renderer_host/render_widget_host_view_base.cc')
| -rw-r--r-- | content/browser/renderer_host/render_widget_host_view_base.cc | 5 |
1 files changed, 5 insertions, 0 deletions
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: |