compute pressure: Increase timeouts and drop uses of t.step_wait()
-
When we need to wait to verify that an update was not sent, update
the timeout from 1 second to 3 seconds (and mark tests with
timeout=long). 1 second is sometimes too little for busy bots such as
the mac-rel and win-rel ones, which can lead to flakiness or, in this
case, updates not being delivered for the wrong reason (i.e. the
browser process just has not had time to process the
update_virtual_pressure_source() call). -
Introduce a new SyncPressureObserver wrapper class that stores
multiple pressure updates and allows users to synchronously wait for
the next one using Promises. This is useful for tests that need to
send and wait for multiple updates: instead of polling using
t.step_wait(), these tests can just use this class to wait for a
Promise to resolve when an update is delivered.
Bug: 347031400
Change-Id: I7880577757d71678e612d912716f4f12020a5fef
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5912856
Reviewed-by: Reilly Grant reillyg@chromium.org
Commit-Queue: Reilly Grant reillyg@chromium.org
Auto-Submit: Raphael Kubo Da Costa raphael.kubo.da.costa@intel.com
Cr-Commit-Position: refs/heads/main@{#1365092}