diff options
| author | vadimt <vadimt@chromium.org> | 2014-09-25 15:08:59 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2014-09-25 22:09:13 +0000 |
| commit | a19e4877974a6a0cfc53f92c1496a8f995fe2614 (patch) | |
| tree | c04d9239d76317558ae584c6274dd6c0aedf3ec2 /base | |
| parent | ab5a96fa6cec2efe3e096e07b2eef466340de225 (diff) | |
| download | chromium_src-a19e4877974a6a0cfc53f92c1496a8f995fe2614.zip chromium_src-a19e4877974a6a0cfc53f92c1496a8f995fe2614.tar.gz chromium_src-a19e4877974a6a0cfc53f92c1496a8f995fe2614.tar.bz2 | |
Adding tracking to verify the theory that jankiness of ExtensionMessageFilter::OnExtensionAddListener is because of the observer invocation.
BUG=401560, 417106
Review URL: https://codereview.chromium.org/600213002
Cr-Commit-Position: refs/heads/master@{#296804}
Diffstat (limited to 'base')
| -rw-r--r-- | base/profiler/scoped_profile.h | 25 |
1 files changed, 5 insertions, 20 deletions
diff --git a/base/profiler/scoped_profile.h b/base/profiler/scoped_profile.h index c435a1d..7290908 100644 --- a/base/profiler/scoped_profile.h +++ b/base/profiler/scoped_profile.h @@ -17,31 +17,16 @@ #include "base/profiler/tracked_time.h" #include "base/tracked_objects.h" -#if defined(GOOGLE_CHROME_BUILD) - -// We don't ship these profiled regions. This is for developer builds only. -// It allows developers to do some profiling of their code, and see results on -// 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 \ 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( \ +// Defines the containing scope as a profiled region. This allows developers to +// profile their code and see results on their about:profiler page, as well as +// on the UMA dashboard. +#define TRACK_RUN_IN_THIS_SCOPED_REGION(dispatch_function_name) \ + ::tracked_objects::ScopedProfile LINE_BASED_VARIABLE_NAME_FOR_PROFILING( \ FROM_HERE_WITH_EXPLICIT_FUNCTION(#dispatch_function_name)) |