Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(402)

Unified Diff: base/profiler/scoped_profile.h

Issue 600213002: Instrumenting OnExtensionAddListener observer invocation (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: brettw@ comment Created 6 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | extensions/browser/event_router.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: base/profiler/scoped_profile.h
diff --git a/base/profiler/scoped_profile.h b/base/profiler/scoped_profile.h
index c435a1dea1cd40513c3ec662c33e366f7f9deaf6..7622dc0d925a8be57e5a774fb731193c08acf66d 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))
« no previous file with comments | « no previous file | extensions/browser/event_router.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698