commit f71656d592bfbe44d8081e29cff99d64df2b7ada
author Dominic Farolino <dom@chromium.org> Tue Dec 01 17:57:44 2020
committer Chromium LUCI CQ <chromium-scoped@luci-project-accounts.iam.gserviceaccount.com> Tue Dec 01 17:57:44 2020
tree 136724bc6fb07822bca0d57aa3c6b23d3aaa80de
parent 6204cae016902419ebd8050c01b1be6472ce8276

Revert "Reland "[Prefetch]: DCHECK if recursive prefetch token is set incorrectly""

This reverts commit 422bc672fc650d77b6acc20aed8b484e89168862.

Reason for revert: We've collected enough info

Original change's description:
> Reland "[Prefetch]: DCHECK if recursive prefetch token is set incorrectly"
>
> This reverts commit 2638a6e6b1314ce5b4dab1060cc9f1a030659e33.
>
> Reason for revert: We thought we had collected enough crash data, however crbug.com/1122182 was not actually fixed yet, so the minidumps did not actually contain the information we needed. agrieve@ has informed me that the issue should be fixed now, and we'll confirm whether or not that is the case by:
>  1.) Reverting this change
>  2.) Collecting minidumps
>  3.) Examining the minidumps to confirm that the debugging information exists
>
> Original change's description:
> > Revert "[Prefetch]: DCHECK if recursive prefetch token is set incorrectly"
> >
> > This reverts commit b344652bc792459f87e1e29312e6e2c4a403bd14.
> > Reason for revert: We have collected enough crashdumps now, and this is
> > causing crash metrics to be skewed as it's getting counted as a real
> > crash.
> >
> > Bug: 1132770
> > Change-Id: Ia2e7a92bea2ebeca01c62ca9af73d3fe014685f6
> > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2505985
> > Auto-Submit: Mugdha Lakhani <nator@chromium.org>
> > Commit-Queue: Charlie Harrison <csharrison@chromium.org>
> > Reviewed-by: Charlie Harrison <csharrison@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#822189}
>
> TBR=csharrison@chromium.org,nator@chromium.org
>
> # Not skipping CQ checks because original CL landed > 1 day ago.
>
> TBR: charrison@chromium.org
> Bug: 1132770
> Change-Id: I31ac96c2c5f4f0738fd2d8c4e60236b2bc0c35a3
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2513109
> Reviewed-by: Charlie Harrison <csharrison@chromium.org>
> Reviewed-by: Mugdha Lakhani <nator@chromium.org>
> Reviewed-by: Dominic Farolino <dom@chromium.org>
> Commit-Queue: Dominic Farolino <dom@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#829091}

TBR=csharrison@chromium.org,dom@chromium.org,nator@chromium.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: 1132770
Change-Id: Ie8515e382ba8d4610aaeb3026909bec63a1e9405
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2568110
Commit-Queue: Dominic Farolino <dom@chromium.org>
Reviewed-by: Dominic Farolino <dom@chromium.org>
Cr-Commit-Position: refs/heads/master@{#832416}