Racey tests under LayoutTests/external/wpt/webmessaging/... |
||
Issue descriptionTwo tests under LayoutTests/external/wpt/webmessaging/... are racey: - The tests ask 2 subframes to respond to postMessage and collect their responses (also received via postMessage). - The tests expect the responses to come in a particular order (which is not guaranteed - the cross-origin frames can run in parallel) With OOPIFs (e.g. with --isolate-origins) the extra parallelism causes the tests to fail: Bug(none) external/wpt/webmessaging/event.origin.sub.htm [ Failure ] Bug(none) external/wpt/webmessaging/postMessage_asterisk_xorigin.sub.htm [ Failure ]
,
Mar 10 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/ff3e4ac863d143527e1d3209fd4a48c4171a01df commit ff3e4ac863d143527e1d3209fd4a48c4171a01df Author: Lukasz Anforowicz <lukasza@chromium.org> Date: Sat Mar 10 06:27:52 2018 Make 2 tests immune to legitimate races between cross-origin subframes Bug: 820589 Change-Id: I16c02746c61f42fafead5f96ae436b331fba4f85 Reviewed-on: https://chromium-review.googlesource.com/957587 Commit-Queue: Łukasz Anforowicz <lukasza@chromium.org> Reviewed-by: Alex Moshchuk <alexmos@chromium.org> Cr-Commit-Position: refs/heads/master@{#542360} [modify] https://crrev.com/ff3e4ac863d143527e1d3209fd4a48c4171a01df/third_party/WebKit/LayoutTests/FlagExpectations/site-per-process [modify] https://crrev.com/ff3e4ac863d143527e1d3209fd4a48c4171a01df/third_party/WebKit/LayoutTests/external/wpt/webmessaging/event.origin.sub.htm [modify] https://crrev.com/ff3e4ac863d143527e1d3209fd4a48c4171a01df/third_party/WebKit/LayoutTests/external/wpt/webmessaging/postMessage_asterisk_xorigin.sub.htm
,
Mar 12 2018
|
||
►
Sign in to add a comment |
||
Comment 1 by lukasza@chromium.org
, Mar 9 2018