Revert "[filesapp] Add missing includes and fail if host crashed"
This reverts commit 4734ecf2. Reason for revert: https://crrev.com/c/2461602 breaks build Linux ChromiumOS MSan Tests in https://ci.chromium.org/p/chromium/builders/ci/Linux%20ChromiumOS%20MSan%20Tests/20900 Failed test is DevToolsListenerBrowserTest.CanCollectCodeCoverage Original change's description: > [filesapp] Add missing includes and fail if host crashed > > Adds #include's that were missing from the initial patch, and FAIL the > test if the DevToolsAgentHost attached to crashes during the test. > > Fix HostString() helper to *actually* take an optional prefix: use C++ > c++17 {} style initializer. > > Test: browser_test --gtest_filter="*DevToolsListener*" > Bug: 1113941 > Change-Id: I5901019add9fea92b6e3a8b5b881b8d3d197b6c0 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2465456 > Commit-Queue: Ben Reich <benreich@chromium.org> > Reviewed-by: Noel Gordon <noel@chromium.org> > Cr-Commit-Position: refs/heads/master@{#816070} TBR=noel@chromium.org,benreich@chromium.org Change-Id: I87ec9b3db970807af8fc75ca137fc83397f775bf No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 1113941 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2466016Reviewed-by:Maggie Cai <mxcai@chromium.org> Commit-Queue: Maggie Cai <mxcai@chromium.org> Cr-Commit-Position: refs/heads/master@{#816400}
Showing
Please register or sign in to comment