DescriptionRevert of Fixed a bug where chrome crash omits crash time stamp after it's sent. (patchset #1 id:1 of https://codereview.chromium.org/2568953003/ )
Reason for revert:
See crbug.com/674249: This change broke things in two ways:
- The string format incorrectly uses the crash-time parameter twice, rather than the second parameter coming from upload-time.
- The string format assumes that all crashes have a valid crash-time, which is not the case for Breakpad reports.
Original issue's description:
> Fixed a bug where chrome crash omits crash time stamp after it's sent.
>
> This cl patches in the solution devised by phistuck@gmail.com. See bug 664430 for detail.
>
> BUG=664430
>
> Committed: https://crrev.com/d8027b2065994c2a5b8cbea2821971dc60b7c6f5
> Cr-Commit-Position: refs/heads/master@{#438333}
TBR=rsesek@chromium.org,jiameng@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=664430
Committed: https://crrev.com/8fd83ce883b96af751b0a2d7919f40b3bbadf1e4
Cr-Commit-Position: refs/heads/master@{#440801}
Patch Set 1 #
Messages
Total messages: 14 (6 generated)
|