Add tracing AutoOpenCloseEvent.

base/trace_event/persistent_async_event.cc

+// Copyright 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 "base/trace_event/persistent_async_event.h"
+
+#include "base/macros.h"
+#include "base/time/time.h"
+#include "base/trace_event/trace_event.h"
+
+namespace base {
+
+namespace trace_event {
+
+PersistentAsyncEvent::PersistentAsyncEvent() : active_(false) {
+  id_ = static_cast<void*>(this);
+  base::trace_event::TraceLog::GetInstance()->AddEnabledStateObserver(this);
+}
+
+PersistentAsyncEvent::~PersistentAsyncEvent() {
+  base::trace_event::TraceLog::GetInstance()->RemoveEnabledStateObserver(this);
+}
+
+void PersistentAsyncEvent::Start() {
+  RecordStartEvent();
+  active_ = true;
+  start_time_ = base::TimeTicks::Now().ToInternalValue();
+}
+
+void PersistentAsyncEvent::Stop() {
+  RecordStopEvent();
+  active_ = false;
+}
+
+void PersistentAsyncEvent::OnTraceLogEnabled() {
+  if (active_) {

[charliea (OOO until 10-5), 2016/08/31 17:30:38]

Looks like it's okay to leave off the brace for one-line conditionals:

https://cs.chromium.org/search/?q=if.*%5C)+f:base&sq=package:chromium&type=cs

[alexandermont, 2016/08/31 18:31:59]

Done

+    RecordStartEventWithTimestamp(start_time_);
+  }
+}
+
+void PersistentAsyncEvent::OnTraceLogDisabled() {
+  // TODO(alexandermont): How do we call RecordStopEvent before log disabled?
+  if (active_) {
+    RecordStopEvent();
+  }
+}
+
+}  // namespace trace_event
+}  // namespace base