Make window message processing tally in the task profiler.

base/win/wrapped_window_proc.h

Index: base/win/wrapped_window_proc.h
diff --git a/base/win/wrapped_window_proc.h b/base/win/wrapped_window_proc.h
index d464a9c279fe4b3618a22a9c3b5625735fc67fdf..02382b11e32c51d41eec8b778a499e113f06e7e4 100644
--- a/base/win/wrapped_window_proc.h
+++ b/base/win/wrapped_window_proc.h
@@ -13,6 +13,7 @@
 #include <windows.h>
 
 #include "base/base_export.h"
+#include "base/profiler/scoped_profile.h"
 #include "base/strings/string16.h"
 
 namespace base {
@@ -69,11 +70,23 @@ BASE_EXPORT void InitializeWindowClass(
 //   CreateWindowW(class_name, window_name, ...
 //
 template <WNDPROC proc>
-LRESULT CALLBACK WrappedWindowProc(HWND hwnd, UINT message,
-                                   WPARAM wparam, LPARAM lparam) {
+LRESULT CALLBACK ProfiledWrappedWindowProc(HWND hwnd,
+                                           UINT message,
+                                           WPARAM wparam,
+                                           LPARAM lparam) {
+  // The template parameter <proc> identifies the wrapped function.
+  tracked_objects::ScopedProfile scoped_profile(

[grt (UTC plus 2), 2017/05/16 07:27:35]

are you doing this in its own function template because stack unwinding won't
run dtors of stack objects within the scope of the __try{} in case of an
exception? if so, please document. if not, what is the benefit to this over
putting scoped_profile within the __try{} block?

i ask because with the two templates defined like this, i (the reader) wonder
why i would use WrappedWindowProc instead of ProfiledWrappedWindowProc. the
latter is the first thing i'll read, and sounds like a reasonable thing to use
("sure, i want profiling!").

[Sigurður Ásgeirsson, 2017/05/16 13:16:58]

This was pure habit - seems the ScopedProfile works just fine inside the __try
block.
The ScopedProfile won't get destructed on SEH, and this will mess up the
cumulatives for this thread, but it don't matter none, cause there's much bigger
problems, and the process isn't long for the world.

+      FROM_HERE, tracked_objects::ScopedProfile::ENABLED);
+
+  return proc(hwnd, message, wparam, lparam);
+}
+
+template <WNDPROC proc>
+LRESULT CALLBACK
+WrappedWindowProc(HWND hwnd, UINT message, WPARAM wparam, LPARAM lparam) {
   LRESULT rv = 0;
   __try {
-    rv = proc(hwnd, message, wparam, lparam);
+    rv = ProfiledWrappedWindowProc<proc>(hwnd, message, wparam, lparam);
   } __except(CallExceptionFilter(GetExceptionInformation())) {
   }
   return rv;