Commit bf93e278 authored by Yuly Novikov's avatar Yuly Novikov Committed by Chromium LUCI CQ

Update pixel wrangling doc

Add instructions for rubber stamping expectations.

Bug: 1159542
Change-Id: Ifc546f0e8915ed2291322d4af38a8066c40ac728
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2611667Reviewed-by: default avatarKenneth Russell <kbr@chromium.org>
Commit-Queue: Yuly Novikov <ynovikov@chromium.org>
Cr-Commit-Position: refs/heads/master@{#840309}
parent 59212f43
...@@ -271,6 +271,10 @@ shift, and a calendar appointment. ...@@ -271,6 +271,10 @@ shift, and a calendar appointment.
how important it is to eliminate flakiness rather than hiding it. how important it is to eliminate flakiness rather than hiding it.
1. For failures of rendering_representative_perf_tests please refer to its 1. For failures of rendering_representative_perf_tests please refer to its
[instructions on updating expectations][rendering_representative_perf_tests]. [instructions on updating expectations][rendering_representative_perf_tests].
1. It's always better to have the CL reviewed properly, but for urgent
suppressions when no reviewer is available, it is possible to rubber
stamp the CL via adding `rubber-stamper@appspot.gserviceaccount.com` as
your reviewer, in addition to the regular reviewer.
1. For the remaining Gtest-style tests, use the [`DISABLED_` 1. For the remaining Gtest-style tests, use the [`DISABLED_`
modifier][gtest-DISABLED] to suppress any failures if necessary. modifier][gtest-DISABLED] to suppress any failures if necessary.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment