LUCI Bisection could not automatically submit this revert because LUCI Bisection has not yet support auto-commit of revert CL for test failure.
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Bot-Commit | +1 |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Owners-Override | +1 |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Code-Review | +1 |
| Commit-Queue | +2 |
I suppose https://chromium.googlesource.com/chromium/src/+/HEAD/docs/testing/web_test_expectations.md#how-to-rebaseline should be used when re-landing to update the expectations.
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Revert "Remove insignificant trailing spaces from the output of testRunner.log()"
This reverts commit 65c2187d64cb6fadc17087fa9cff4e2e945835c8.
Reason for revert:
LUCI Bisection has identified this change as the cause of a test failure. See the analysis: https://ci.chromium.org/ui/p/chromium/bisection/test-analysis/b/6421186574548992
Sample build with failed test: https://ci.chromium.org/b/8695694665875663633
Affected test(s):
[://\:blink_web_tests!webtest::http/tests/inspector-protocol/fetch-later#activate-and-redirect-cross-site-when-document-alive.https.js](https://ci.chromium.org/ui/test/chromium/:%2F%2F%5C:blink_web_tests%21webtest::http%2Ftests%2Finspector-protocol%2Ffetch-later%23activate-and-redirect-cross-site-when-document-alive.https.js?q=VHash%3A9b30675a77baf900)
[://\:blink_web_tests!webtest::http/tests/inspector-protocol/fetch-later#activate-and-redirect-when-document-alive.https.js](https://ci.chromium.org/ui/test/chromium/:%2F%2F%5C:blink_web_tests%21webtest::http%2Ftests%2Finspector-protocol%2Ffetch-later%23activate-and-redirect-when-document-alive.https.js?q=VHash%3A9b30675a77baf900)
[://\:blink_web_tests!webtest::http/tests/inspector-protocol/fetch-later#activate-post-when-document-alive.https.js](https://ci.chromium.org/ui/test/chromium/:%2F%2F%5C:blink_web_tests%21webtest::http%2Ftests%2Finspector-protocol%2Ffetch-later%23activate-post-when-document-alive.https.js?q=VHash%3A9b30675a77baf900)
[://\:blink_web_tests!webtest::http/tests/inspector-protocol/fetch-later#activate-when-document-alive.https.js](https://ci.chromium.org/ui/test/chromium/:%2F%2F%5C:blink_web_tests%21webtest::http%2Ftests%2Finspector-protocol%2Ffetch-later%23activate-when-document-alive.https.js?q=VHash%3A9b30675a77baf900)
If this is a false positive, please report it at http://b.corp.google.com/createIssue?component=1199205&description=Analysis%3A+https%3A%2F%2Fci.chromium.org%2Fui%2Fp%2Fchromium%2Fbisection%2Ftest-analysis%2Fb%2F6421186574548992&format=PLAIN&priority=P3&title=Wrongly+blamed+https%3A%2F%2Fchromium-review.googlesource.com%2Fc%2Fchromium%2Fsrc%2F%2B%2F7248977&type=BUG
Original change's description:
> Remove insignificant trailing spaces from the output of testRunner.log()
>
> ... to prevent linter from tripping on it.
> NOPRESUBMIT=true
> (because win-presubmit can't handle so many files)
>
> Change-Id: Ia072a0766dee97ed688e82b0703a5c78d677be87
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7248977
> Reviewed-by: Alex N. Jose <ale...@chromium.org>
> Commit-Queue: Andrey Kosyakov <ca...@chromium.org>
> Cr-Commit-Position: refs/heads/main@{#1557684}
>
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |