| commit | 8fd83ce883b96af751b0a2d7919f40b3bbadf1e4 | [log] [tgz] |
|---|---|---|
| author | wez <[email protected]> | Tue Dec 27 23:14:59 2016 |
| committer | Commit bot <[email protected]> | Tue Dec 27 23:17:54 2016 |
| tree | b93ab2b6956fdcc8f3d61d0810f14ac4aad4d050 | |
| parent | 3ec9c0d0b1c26c51d9c2f0478de243298151e4e1 [diff] |
Revert 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 [email protected]. See bug 664430 for detail. > > BUG=664430 > > Committed: https://crrev.com/d8027b2065994c2a5b8cbea2821971dc60b7c6f5 > Cr-Commit-Position: refs/heads/master@{#438333} [email protected],[email protected] # Not skipping CQ checks because original CL landed more than 1 days ago. BUG=664430 Review-Url: https://codereview.chromium.org/2601853002 Cr-Commit-Position: refs/heads/master@{#440801}