Commit 1731b978 authored by rmcilroy's avatar rmcilroy Committed by Commit bot

[tracing] Add a DCHECK to ensure that trace events have a valid timestamp.

Non metadata trace events should not have zero timestamps, so add a DCHECK
that this is the case.

BUG=456940,456940

Review URL: https://codereview.chromium.org/920963002

Cr-Commit-Position: refs/heads/master@{#316226}
parent 8393ee8b
...@@ -1908,6 +1908,7 @@ TraceEventHandle TraceLog::AddTraceEventWithThreadIdAndTimestamp( ...@@ -1908,6 +1908,7 @@ TraceEventHandle TraceLog::AddTraceEventWithThreadIdAndTimestamp(
AutoThreadLocalBoolean thread_is_in_trace_event(&thread_is_in_trace_event_); AutoThreadLocalBoolean thread_is_in_trace_event(&thread_is_in_trace_event_);
DCHECK(name); DCHECK(name);
DCHECK(!timestamp.is_null());
if (flags & TRACE_EVENT_FLAG_MANGLE_ID) if (flags & TRACE_EVENT_FLAG_MANGLE_ID)
id ^= process_id_hash_; id ^= process_id_hash_;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment