Add hook for developers to play with profiler (unofficially)

base/profiler/scoped_profile.h

Index: base/profiler/scoped_profile.h
===================================================================
--- base/profiler/scoped_profile.h	(revision 113750)
+++ base/profiler/scoped_profile.h	(working copy)
@@ -16,11 +16,31 @@
 #include "base/location.h"
 #include "base/profiler/tracked_time.h"
 
-#define TRACK_RUN_IN_THIS_SCOPED_REGION_FOR_OFFICIAL_BUILDS(variable_name) \
-    ::tracked_objects::ScopedProfile variable_name(FROM_HERE)
+#if defined(GOOGLE_CHROME_BUILD)

[ian fette, 2011/12/13 01:02:16]

There seems to be active discussion around GOOGLE_CHROME_BUILD vs
OFFICIAL_BUILD. I think historically OFFICIAL_BUILD might have been correct here
but
https://groups.google.com/a/chromium.org/group/chromium-dev/browse_thread/thr...
seems to be leaning the other way. I suspect this is probably now correct, but
just pointing that out.

[jar (doing other things), 2011/12/13 06:36:37]

It is hard to tell the outcome of the discussion you cited... but I copied the
flag used in browser_main to decide on UMA metrics setting.

I *think* the bug you cited leans towards using what I have... so I'll stay with
it.

On 2011/12/13 01:02:16, ian fette wrote:

 
-#define TRACK_RUN_IN_IPC_HANDLER(dispatch_function_name) \
-    ::tracked_objects::ScopedProfile some_tracking_variable_name(  \
+// We don't ship these profiled regions.  This is for developer builds only.
+// It allows developers to do some profililng of their code, and see results on

[ramant (doing other things), 2011/12/13 00:02:54]

nit: profililng -> profiling

[jar (doing other things), 2011/12/13 06:36:37]

Done.

+// their about:profiler page.
+#define TRACK_RUN_IN_THIS_SCOPED_REGION_FOR_DEVELOPER_BUILDS(scope_name)       \
+    ((void)0)
+
+#else
+
+#define TRACK_RUN_IN_THIS_SCOPED_REGION_FOR_DEVELOPER_BUILDS(scope_name)       \
+    ::tracked_objects::ScopedProfile  LINE_BASED_VARIABLE_NAME_FOR_PROFILING(  \
+        FROM_HERE_WITH_EXPLICIT_FUNCTION(#scope_name))
+
+#endif
+
+
+
+#define PASTE_LINE_NUMBER_ON_NAME(name, line) name##line
+
+#define LINE_BASED_VARIABLE_NAME_FOR_PROFILING                                               \

[ramant (doing other things), 2011/12/13 00:02:54]

nit: more than 80 chars

[jar (doing other things), 2011/12/13 06:36:37]

Done.

+    PASTE_LINE_NUMBER_ON_NAME(some_profiler_variable_, __LINE__)
+
+#define TRACK_RUN_IN_IPC_HANDLER(dispatch_function_name)                       \
+    ::tracked_objects::ScopedProfile some_tracking_variable_name(              \
         FROM_HERE_WITH_EXPLICIT_FUNCTION(#dispatch_function_name))