ResourceLoadTiming: Remove TRACE_EVENT with null timestamp.
The TimeTicks for SendStart seems to be null in some tests, which
crashes when tracing is enabled. It is not trivial to add it in all
tests. It would also be non-obvious why code is setting only SendStart
of all the timing properties.
The trace was originally added in
https://codereview.chromium.org/1471763005 and had the same problem that
the TimeTicks is sometimes null. Apparently the CQ does not run all
tests with tracing enabled so it does not always catch when this
happens.
It's also non-obvious why only SendStart has a TRACE_EVENT in this file.
This CL removes the TRACE_EVENT.
Bug: 1229504
Change-Id: Ib9ca1c9d7cd59527e437e928ff19503138408150
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3061778
Commit-Queue: Matt Falkenhagen <falken@chromium.org>
Reviewed-by: Sami Kyöstilä <skyostil@chromium.org>
Reviewed-by: Eric Seckler <eseckler@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Nicolás Peña Moreno <npm@chromium.org>
Cr-Commit-Position: refs/heads/master@{#908803}
NOKEYCHECK=True
GitOrigin-RevId: f36f84cb1fbaf9b1dd45ef238d18b46b83070e70
1 file changed
