Chromium Code Reviews| Index: base/profiler/scoped_profile.h |
| diff --git a/base/profiler/scoped_profile.h b/base/profiler/scoped_profile.h |
| index c435a1dea1cd40513c3ec662c33e366f7f9deaf6..a58c9cb3466adbd544c59e8321018d33b822d254 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( \ |
| +// Profiled region. It allows developers to do profiling of their code, and see |
| +// results on their about:profiler page and in Google-internal server-side |
| +// tools. |
|
Ilya Sherman
2014/09/25 17:28:42
Optional nit: Suggested rephrasing:
// Defines th
vadimt
2014/09/25 17:51:02
Done.
|
| +#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)) |