Instrument ContextCacheController::OnIdle for postmortem reports

cc/output/context_cache_controller.cc

 // Copyright (c) 2016 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 "cc/output/context_cache_controller.h"
 
 #include "base/bind.h"
+#include "base/debug/activity_tracker.h"
 #include "base/logging.h"
 #include "base/memory/ptr_util.h"
 #include "base/synchronization/lock.h"
 #include "gpu/command_buffer/client/context_support.h"
 #include "third_party/skia/include/gpu/GrContext.h"
 
 namespace cc {
 namespace {
 static const int kIdleCleanupDelaySeconds = 1;
 }  // namespace
@@ skipping 133 matching lines @@
 
   // Try to acquire the context lock - if we can't acquire it then we've become
   // busy since checking |current_idle_generation_| above. In this case, just
   // re-post our idle callback and return.
   if (context_lock_ && !context_lock_->Try()) {
     base::AutoLock hold(current_idle_generation_lock_);
     PostIdleCallback(current_idle_generation_);
     return;
   }
 
-  if (gr_context_)
+  if (gr_context_) {
+    // TODO(manzagop): remove ScopedActivity once crbug.com/703342 is fixed.
+    base::debug::ScopedActivity activity_free(0, 0, 0);

[bcwhite, 2017/03/21 12:34:38]

There is a default ctor that is (0,0,0) if you don't want to specify them...
though it's recommended that you do if you want to be able to identify them
during analysis.

[manzagop (departed), 2017/03/21 14:00:23]

Done. (Another option is to rely on the activity's address.)

     gr_context_->freeGpuResources();
+  }
 
   // Toggle SetAggressivelyFreeResources to drop command buffer data.
   context_support_->SetAggressivelyFreeResources(true);
   context_support_->SetAggressivelyFreeResources(false);
 
   callback_pending_ = false;
 
   if (context_lock_)
     context_lock_->Release();
 }
 
 }  // namespace cc