Disable timing from chrome://profiler on Android.

base/tracked_objects.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/tracked_objects.h"
 
 #include <limits.h>
 #include <stdlib.h>
 
+#include "base/base_switches.h"
+#include "base/command_line.h"
 #include "base/compiler_specific.h"
 #include "base/debug/leak_annotations.h"
 #include "base/logging.h"
 #include "base/process/process_handle.h"
 #include "base/profiler/alternate_timer.h"
 #include "base/strings/stringprintf.h"
 #include "base/third_party/valgrind/memcheck.h"
 #include "base/tracking_info.h"
 
 using base::TimeDelta;
@@ skipping 24 matching lines @@
 
 // Control whether an alternate time source (Now() function) is supported by
 // the ThreadData class.  This compile time flag should be set to true if we
 // want other modules (such as a memory allocator, or a thread-specific CPU time
 // clock) to be able to provide a thread-specific Now() function.  Without this
 // compile-time flag, the code will only support the wall-clock time.  This flag
 // can be flipped to efficiently disable this path (if there is a performance
 // problem with its presence).
 static const bool kAllowAlternateTimeSourceHandling = true;
 
+// Control whether to collect timing information.
+enum ProfilerTimingState {

[Mark Mentovai, 2013/12/09 14:18:32]

You can move this whole enum definition and variable declaration to be in-line
in IsProfilerTimingEnabled.

The enum can also be anonymous, it doesn’t need to have a type name.

[qsr, 2013/12/09 14:48:32]

Done.

+  UNDEFINED_TIMING,
+  ENABLED_TIMING,
+  DISABLED_TIMING,
+} g_timing_enabled = UNDEFINED_TIMING;
+
+inline bool IsProfilerTimingEnabled() {
+  // This initialization is not thread-safe, so the value of |g_timing_enabled|
+  // can be computed multiple time. This is not an issue, as the computed value

[Mark Mentovai, 2013/12/09 14:18:32]

time→times

[qsr, 2013/12/09 14:48:32]

Done.

+  // will always be the same, and is side-effect free.
+  if (g_timing_enabled == UNDEFINED_TIMING) {
+    g_timing_enabled = (CommandLine::ForCurrentProcess()->GetSwitchValueASCII(
+                            switches::kProfilerTiming) ==
+                        switches::kProfilerTimingDisabledValue)
+                           ? DISABLED_TIMING
+                           : ENABLED_TIMING;
+  }
+  return g_timing_enabled == ENABLED_TIMING;
+}
+
 }  // namespace
 
 //------------------------------------------------------------------------------
 // DeathData tallies durations when a death takes place.
 
 DeathData::DeathData() {
   Clear();
 }
 
 DeathData::DeathData(int count) {
@@ skipping 683 matching lines @@
 void ThreadData::SetAlternateTimeSource(NowFunction* now_function) {
   DCHECK(now_function);
   if (kAllowAlternateTimeSourceHandling)
     now_function_ = now_function;
 }
 
 // static
 TrackedTime ThreadData::Now() {
   if (kAllowAlternateTimeSourceHandling && now_function_)
     return TrackedTime::FromMilliseconds((*now_function_)());
-  if (kTrackAllTaskObjects && TrackingStatus())
+  if (kTrackAllTaskObjects && IsProfilerTimingEnabled() && TrackingStatus())
     return TrackedTime::Now();
   return TrackedTime();  // Super fast when disabled, or not compiled.
 }
 
 // static
 void ThreadData::EnsureCleanupWasCalled(int major_threads_shutdown_count) {
   base::AutoLock lock(*list_lock_.Pointer());
   if (worker_thread_data_creation_count_ == 0)
     return;  // We haven't really run much, and couldn't have leaked.
   // Verify that we've at least shutdown/cleanup the major namesd threads.  The
@@ skipping 95 matching lines @@
     : process_id(base::GetCurrentProcId()) {
 #else
     : process_id(0) {
 #endif
 }
 
 ProcessDataSnapshot::~ProcessDataSnapshot() {
 }
 
 }  // namespace tracked_objects