|
|
DescriptionUse ScopedCrashKey for RendererDidNavigate crash dumps
This will make the crash reports easier to view.
BUG=688425
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
Review-Url: https://codereview.chromium.org/2696193006
Cr-Commit-Position: refs/heads/master@{#451501}
Committed: https://chromium.googlesource.com/chromium/src/+/b1730dabdf125160dc23db993e08f453fc648fc8
Patch Set 1 #Patch Set 2 : apparently I have to register the ScopedCrashKey #Patch Set 3 : apparently I have to register the ScopedCrashKey #Patch Set 4 : revert unrelated 'git cl format' changes #
Messages
Total messages: 35 (23 generated)
Description was changed from ========== Use ScopedCrashKey for RendererDidNavigate crash dumps This will make the crash reports easier to view. BUG=688425 ========== to ========== Use ScopedCrashKey for RendererDidNavigate crash dumps This will make the crash reports easier to view. BUG=688425 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation ==========
The CQ bit was checked by estark@chromium.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
estark@chromium.org changed reviewers: + creis@chromium.org
creis, could you please take a look? (looks like jam and nasko who have been investigating this bug with me are OOO)
The ScopedCrashKey change LGTM. I'm curious about the collected data, though-- what's the theory you're trying to test?
The CQ bit was unchecked by estark@chromium.org
The CQ bit was checked by estark@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
The CQ bit was unchecked by commit-bot@chromium.org
Try jobs failed on following builders: win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)
The CQ bit was checked by estark@chromium.org to run a CQ dry run
estark@chromium.org changed reviewers: + ananta@chromium.org, rsesek@chromium.org
rsesek, can you please review chrome/common/crash_keys.cc? ananta, can you please review chrome/app/chrome_crash_reporter_client_win.cc?
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
The CQ bit was checked by estark@chromium.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
chrome_crash_reporter_client_win.cc lgtm
lgtm
The CQ bit was unchecked by estark@chromium.org
The CQ bit was checked by estark@chromium.org
The patchset sent to the CQ was uploaded after l-g-t-m from creis@chromium.org Link to the patchset: https://codereview.chromium.org/2696193006/#ps60001 (title: "revert unrelated 'git cl format' changes")
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
The CQ bit was unchecked by commit-bot@chromium.org
Try jobs failed on following builders: win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)
The CQ bit was checked by estark@chromium.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: Try jobs failed on following builders: linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
The CQ bit was checked by estark@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
CQ is committing da patch. Bot data: {"patchset_id": 60001, "attempt_start_ts": 1487458860812410, "parent_rev": "bce53455673368a99e755d09ca7871c38a252c0a", "commit_rev": "b1730dabdf125160dc23db993e08f453fc648fc8"}
Message was sent while issue was closed.
Description was changed from ========== Use ScopedCrashKey for RendererDidNavigate crash dumps This will make the crash reports easier to view. BUG=688425 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation ========== to ========== Use ScopedCrashKey for RendererDidNavigate crash dumps This will make the crash reports easier to view. BUG=688425 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation Review-Url: https://codereview.chromium.org/2696193006 Cr-Commit-Position: refs/heads/master@{#451501} Committed: https://chromium.googlesource.com/chromium/src/+/b1730dabdf125160dc23db993e08... ==========
Message was sent while issue was closed.
Committed patchset #4 (id:60001) as https://chromium.googlesource.com/chromium/src/+/b1730dabdf125160dc23db993e08...
Message was sent while issue was closed.
A revert of this CL (patchset #4 id:60001) has been created in https://codereview.chromium.org/2705303002/ by jam@chromium.org. The reason for reverting is: We got the data we wanted.. |