-
Mark Pilgrim authored
The previous attempt was reverted because it crashed. This attempt features a crash test which reproduces the crash when run against the previous CL, and confirms that this CL does not crash. The cause of the crash was a reference to simple_url_loader after it had been std:move'd. This CL instead gets a reference to the newly moved pointer. previous CL: https://chromium-review.googlesource.com/c/chromium/src/+/1025068 revert CL: https://chromium-review.googlesource.com/c/chromium/src/+/1030590 Bug: 773295 Test: ResetReportUploaderTest.NoCrash Change-Id: I7092ec501872dfa0019d37e8f9e10454458499db Reviewed-on: https://chromium-review.googlesource.com/1042359 Commit-Queue: Mark Pilgrim <pilgrim@chromium.org> Reviewed-by:
Dominic Battré <battre@chromium.org> Reviewed-by:
Matt Menke <mmenke@chromium.org> Cr-Commit-Position: refs/heads/master@{#556447}
19a83d7a