DescriptionReland of Add process uptime metadata event in tracing (patchset #1 id:1 of https://codereview.chromium.org/2768123002/ )
Reason for revert:
The race was caused by https://codereview.chromium.org/2736283003/
Fixed the post tasks.
Original issue's description:
> Revert of Add process uptime metadata event in tracing (patchset #4 id:80001 of https://codereview.chromium.org/2753133002/ )
>
> Reason for revert:
> Looks like this broke base_unittests on Linux TSan: https://uberchromegw.corp.google.com/i/chromium.memory/builders/Linux%20TSan%20Tests/builds/2916
>
> Original issue's description:
> > Add process uptime metadata event in tracing
> >
> > 1. Make CurrentProcessInfo::CreationTime consistent on Linux with Win
> > and Mac implementation: return null value instead of crashing on
> > failure.
> > 2. Use process_info_linux.cc in Android too, it works the same.
> > 3. Add the total process uptime as meadata event in traces and
> > whitelist in filtered mode.
> >
> > BUG=654667
> >
> > Review-Url: https://codereview.chromium.org/2753133002
> > Cr-Commit-Position: refs/heads/master@{#458799}
> > Committed: https://chromium.googlesource.com/chromium/src/+/a714d7dc0f1b8ecdee128db04e24b0da1087df8a
>
> TBR=oysteine@chromium.org,thakis@chromium.org,primiano@chromium.org,ssid@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=654667
>
> Review-Url: https://codereview.chromium.org/2768123002
> Cr-Commit-Position: refs/heads/master@{#458869}
> Committed: https://chromium.googlesource.com/chromium/src/+/958a4bb090a8f57113d094be99b573c036ecb1e0
TBR=oysteine@chromium.org,thakis@chromium.org,dgozman@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
BUG=654667
Review-Url: https://codereview.chromium.org/2768933002
Cr-Commit-Position: refs/heads/master@{#459167}
Committed: https://chromium.googlesource.com/chromium/src/+/dd8debd00f82d49bd0b0cf1a58c6a679540ee87a
Patch Set 1 : rebase original CL. #Patch Set 2 : Fix if condition. #
Messages
Total messages: 16 (9 generated)
|