Commit 55bc3e46 authored by Miguel Casas's avatar Miguel Casas Committed by Commit Bot

media/gpu/vaapi_unittest: move sandbox init to main()

crrev.com/c/2292813 (in 86.0.4202.0) removed one of the multiple calls
to VaapiWrapper::PreSandboxInitialization(), concretely the one in the
main() function, leaving only one in the test SetUp() method. This
broke the graphics.VAAPIUnittest.* in the CQ eve.

This CL fixes that by leaving a single call to said static method, in
the main() function.

Bug: 1105335, 1040291
Change-Id: I0b1bfa14f8768c6111697f591b0c8ff120c46e83
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2298263
Commit-Queue: Miguel Casas <mcasas@chromium.org>
Reviewed-by: default avatarAndres Calderon Jaramillo <andrescj@chromium.org>
Cr-Commit-Position: refs/heads/master@{#788364}
parent 5ef7ba19
......@@ -86,12 +86,6 @@ class VaapiTest : public testing::Test {
public:
VaapiTest() = default;
~VaapiTest() override = default;
void SetUp() override {
// PreSandboxInitialization() loads and opens the driver, queries its
// capabilities and fills in the VASupportedProfiles.
VaapiWrapper::PreSandboxInitialization();
}
};
std::map<VAProfile, std::vector<VAEntrypoint>> ParseVainfo(
......@@ -234,6 +228,10 @@ TEST_F(VaapiTest, DefaultEntrypointIsSupported) {
int main(int argc, char** argv) {
base::TestSuite test_suite(argc, argv);
// PreSandboxInitialization() loads and opens the driver, queries its
// capabilities and fills in the VASupportedProfiles.
media::VaapiWrapper::PreSandboxInitialization();
return base::LaunchUnitTests(
argc, argv,
base::BindOnce(&base::TestSuite::Run, base::Unretained(&test_suite)));
......
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