DescriptionRevert of Add ChannelPosix::ResetSafely() to deal with a lingering crash (patchset #1 id:1 of https://codereview.chromium.org/882543002/)
Reason for revert:
many ipc test failed.
https://build.chromium.org/p/chromium.memory/builders/Linux%20ASan%20LSan%20Tests%20%281%29/builds/10036
Original issue's description:
> Add ChannelPosix::ResetSafely() to deal with a lingering crash
>
> The CL [1] tightened the error check too much and revealed
> an existing inconsistency and resulted a production crash.
> This CL makes a workaround to that crash, turning a PCHECK()
> to DPCHECK().
>
> [1] https://crrev.com/ce44fef5fd60dd2be5c587d4b084bdcd36adcee4
>
> BUG=449233, 448245
> R=agl@chromium.org,jam@chromium.org
>
> Committed: https://crrev.com/cf3eae19b456536c5ff8dacf8800ec09f9af4bca
> Cr-Commit-Position: refs/heads/master@{#313188}
TBR=agl@chromium.org,jam@chromium.org,morrita@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=449233, 448245
Committed: https://crrev.com/b83c03c7b691914a054abaab13fb9131b4e0152b
Cr-Commit-Position: refs/heads/master@{#313226}
Patch Set 1 #
Created: 5 years, 11 months ago
(Patch set is too large to download)
Messages
Total messages: 4 (0 generated)
|