GPU Tracer no longer issue device traces when device category is off.

gpu/command_buffer/service/gpu_tracer.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 "gpu/command_buffer/service/gpu_tracer.h"
 
 #include <deque>
 
 #include "base/bind.h"
 #include "base/strings/string_util.h"
@@ skipping 72 matching lines @@
 
 GPUTrace::GPUTrace(scoped_refptr<Outputter> outputter,
                    gfx::GPUTimingClient* gpu_timing_client,
                    const std::string& category,
                    const std::string& name,
                    const bool enabled)
     : category_(category),
       name_(name),
       outputter_(outputter),
       enabled_(enabled) {
-  if (gpu_timing_client->IsAvailable() &&
+  if (enabled &&

[vmiura, 2015/04/10 22:18:50]

Thinking about this, "enabled" here and "tracing_service" in Start() and End()
seem rather inconsistent.

Should we have both 'device_enabled' and 'service_enabled' in the constructor?

[David Yen, 2015/04/10 22:36:09]

Done.

+      gpu_timing_client->IsAvailable() &&
       gpu_timing_client->IsTimerOffsetAvailable()) {
     gpu_timer_ = gpu_timing_client->CreateGPUTimer();
   }
 }
 
 GPUTrace::~GPUTrace() {
 }
 
 void GPUTrace::Destroy(bool have_context) {
   if (gpu_timer_.get()) {
@@ skipping 259 matching lines @@
 void GPUTracer::IssueProcessTask() {
   if (finished_traces_.empty() || process_posted_)
     return;
 
   process_posted_ = true;
   PostTask();
 }
 
 }  // namespace gles2
 }  // namespace gpu