VV8 LUCI CQ[test] Add a unittest platform setup mixin
| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
cppgc: Add owners for cppgc samples Bug: chromium:1056170 Change-Id: I3779ed504235e3aa3fbecb5f14dd2d43d9907c07 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2560201 Reviewed-by: Hannes Payer <hpayer@chromium.org> Commit-Queue: Omer Katz <omerkatz@chromium.org> Cr-Commit-Position: refs/heads/master@{#71420} | 5 年前 | |
[test] Add a unittest platform setup mixin Change the unittest runner to no longer uncondtionally set up a default platform in the "environment", but to instead make platform set-up part of the "mixin" framework for test fixtures. Requires modifying some tests that expect the platform to be available, and all flag implications resolved, before the mixin constructors run. We still keep the environment for setting up the process for cppgc. This process setup can only be done once per process, so it can no longer use the platform -- that's ok though, the page allocator used by cppgc's process initialisation doesn't have to be the same as the platform's so we can just pass in a separate new one. Change-Id: Ic8ccf39722e8212962c5bba87350c4b304388a7c Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3571886 Reviewed-by: Michael Lippautz <mlippautz@chromium.org> Auto-Submit: Leszek Swirski <leszeks@chromium.org> Commit-Queue: Leszek Swirski <leszeks@chromium.org> Cr-Commit-Position: refs/heads/main@{#79820} | 4 年前 |