Removed an assert about addTraceEvent not returning empty handles.

There are several codepaths where this is the expected result: * Other threads disabling the category in question. * The TraceBuffer being fill and no new chunks available. * Perfetto being active as the tracing backend. In all cases, the single use of a handle to a TraceEvent is TraceLog::UpdateTraceEventDuration which handles the above cases correctly. BUG=844421 Change-Id: Ieaf3aa5c913cee8c51cfea637907d5bc3b560ceb Reviewed-on: https://chromium-review.googlesource.com/1120841Reviewed-by: Jamie Madill <jmadill@chromium.org> Commit-Queue: Jamie Madill <jmadill@chromium.org>

Removed an assert about addTraceEvent not returning empty handles.
33e72d99 · Oystein Eftevaag · Commit Bot · 1266d20a · 33e72d99
Commit 33e72d99 authored Jun 29, 2018 by Oystein Eftevaag Committed by Commit Bot Jun 29, 2018
Hide whitespace changes
Inline Side-by-side

Showing with 0 additions and 1 deletion

event_tracer.cpp src/common/event_tracer.cpp +0 -1

No files found.
--- a/src/common/event_tracer.cpp
+++ b/src/common/event_tracer.cpp
@@ -45,7 +45,6 @@ angle::TraceEventHandle AddTraceEvent(char phase,
        angle::TraceEventHandle handle =
            platform->addTraceEvent(platform, phase, categoryGroupEnabled, name, id, timestamp,
                                    numArgs, argNames, argTypes, argValues, flags);
-        ASSERT(handle != 0);
        return handle;
    }