Revert "[Sheriff] Disable GPU tests on Linux"

This reverts commit a6f84dd7.

Reason for revert: Linux CFI suppression for relocated vulkan-loader dep has been added.

Original change's description:
> [Sheriff] Disable GPU tests on Linux
>
> Bug: 1162117
> Change-Id: I1c5b8f9379150a17b2f35b2681dc2149b2759dec
> TBR: jdarpinian
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2605403
> Commit-Queue: James Darpinian <jdarpinian@chromium.org>
> Auto-Submit: Alan Screen <awscreen@chromium.org>
> Reviewed-by: James Darpinian <jdarpinian@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#839502}

TBR=jdarpinian@chromium.org,awscreen@chromium.org,chromium-scoped@luci-project-accounts.iam.gserviceaccount.com

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: 1162117
Change-Id: Ic2e12eaefafdb361059434d3062001ba15084257
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2607158Reviewed-by: Austin Eng <enga@chromium.org>
Reviewed-by: Alan Screen <awscreen@chromium.org>
Commit-Queue: Austin Eng <enga@chromium.org>
Cr-Commit-Position: refs/heads/master@{#839771}

gpu/command_buffer/tests/webgpu_fence_unittest.cc

@@ -2,19 +2,11 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#include "build/build_config.h"
 #include "gpu/command_buffer/client/webgpu_implementation.h"
 #include "gpu/command_buffer/tests/webgpu_test.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
-// TODO(crbug.com/1162117): gl_tests failing on Linux
-#if defined(OS_LINUX)
-#define MAYBE(test_name) DISABLED_##test_name
-#else
-#define MAYBE(test_name) test_name
-#endif
-
 namespace {
 
 class MockFenceOnCompletionCallback {
@@ -56,7 +48,7 @@ class WebGPUFenceTest : public WebGPUTest {
 };
 
 // Test that getting the value of the fence is the initial value.
-TEST_F(WebGPUFenceTest, MAYBE(InitialValue)) {
+TEST_F(WebGPUFenceTest, InitialValue) {
   if (!WebGPUSupported()) {
     LOG(ERROR) << "Test skipped";
     return;
@@ -79,7 +71,7 @@ TEST_F(WebGPUFenceTest, MAYBE(InitialValue)) {
 }
 
 // Test that after signaling a fence, its completed value gets updated.
-TEST_F(WebGPUFenceTest, MAYBE(GetCompletedValue)) {
+TEST_F(WebGPUFenceTest, GetCompletedValue) {
   if (!WebGPUSupported()) {
     LOG(ERROR) << "Test skipped";
     return;
@@ -98,7 +90,7 @@ TEST_F(WebGPUFenceTest, MAYBE(GetCompletedValue)) {
 
 // Test that a fence's OnCompletion handler is called after the signal value
 // is completed.
-TEST_F(WebGPUFenceTest, MAYBE(OnCompletion)) {
+TEST_F(WebGPUFenceTest, OnCompletion) {
   if (!WebGPUSupported()) {
     LOG(ERROR) << "Test skipped";
     return;
@@ -120,7 +112,7 @@ TEST_F(WebGPUFenceTest, MAYBE(OnCompletion)) {
 }
 
 // Test signaling a fence a million times.
-TEST_F(WebGPUFenceTest, MAYBE(SignalManyTimes)) {
+TEST_F(WebGPUFenceTest, SignalManyTimes) {
   if (!WebGPUSupported()) {
     LOG(ERROR) << "Test skipped";
     return;

gpu/command_buffer/tests/webgpu_mailbox_unittest.cc

@@ -2,7 +2,6 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#include "build/build_config.h"
 #include "components/viz/test/test_gpu_service_holder.h"
 #include "gpu/command_buffer/client/shared_image_interface.h"
 #include "gpu/command_buffer/client/webgpu_implementation.h"
@@ -14,13 +13,6 @@
 #include "testing/gtest/include/gtest/gtest.h"
 #include "ui/gfx/color_space.h"
 
-// TODO(crbug.com/1162117): gl_tests failing on Linux
-#if defined(OS_LINUX)
-#define MAYBE(test_name) DISABLED_##test_name
-#else
-#define MAYBE(test_name) test_name
-#endif
-
 namespace gpu {
 namespace {
 
@@ -96,7 +88,7 @@ class WebGPUMailboxTest : public WebGPUTest {
   }
 };
 
-TEST_F(WebGPUMailboxTest, MAYBE(AssociateMailboxCmd)) {
+TEST_F(WebGPUMailboxTest, AssociateMailboxCmd) {
   if (!WebGPUSupported()) {
     LOG(ERROR) << "Test skipped because WebGPU isn't supported";
     return;
@@ -214,7 +206,7 @@ TEST_F(WebGPUMailboxTest, MAYBE(AssociateMailboxCmd)) {
   GetGpuServiceHolder()->gpu_thread_task_runner()->RunsTasksInCurrentSequence();
 }
 
-TEST_F(WebGPUMailboxTest, MAYBE(DissociateMailboxCmd)) {
+TEST_F(WebGPUMailboxTest, DissociateMailboxCmd) {
   if (!WebGPUSupported()) {
     LOG(ERROR) << "Test skipped because WebGPU isn't supported";
     return;
@@ -282,7 +274,7 @@ TEST_F(WebGPUMailboxTest, MAYBE(DissociateMailboxCmd)) {
 // For simplicity of the test the image is shared between a Dawn device and
 // itself: we render to it using the Dawn device, then re-associate it to a
 // Dawn texture and read back the values that were written.
-TEST_F(WebGPUMailboxTest, MAYBE(WriteToMailboxThenReadFromIt)) {
+TEST_F(WebGPUMailboxTest, WriteToMailboxThenReadFromIt) {
   if (!WebGPUSupported()) {
     LOG(ERROR) << "Test skipped because WebGPU isn't supported";
     return;
@@ -394,7 +386,7 @@ TEST_F(WebGPUMailboxTest, MAYBE(WriteToMailboxThenReadFromIt)) {
 }
 
 // Tests that using a shared image aftr it is dissociated produces an error.
-TEST_F(WebGPUMailboxTest, MAYBE(ErrorWhenUsingTextureAfterDissociate)) {
+TEST_F(WebGPUMailboxTest, ErrorWhenUsingTextureAfterDissociate) {
   if (!WebGPUSupported()) {
     LOG(ERROR) << "Test skipped because WebGPU isn't supported";
     return;
@@ -456,7 +448,7 @@ TEST_F(WebGPUMailboxTest, MAYBE(ErrorWhenUsingTextureAfterDissociate)) {
 // the move-assignment operator to be called. In this case the defaulted
 // move-assignment would first move `representation` then `access`. Causing
 // incorrect member destruction order for the move-to object.
-TEST_F(WebGPUMailboxTest, MAYBE(UseA_UseB_DestroyA_DestroyB)) {
+TEST_F(WebGPUMailboxTest, UseA_UseB_DestroyA_DestroyB) {
   if (!WebGPUSupported()) {
     LOG(ERROR) << "Test skipped because WebGPU isn't supported";
     return;
@@ -509,7 +501,7 @@ TEST_F(WebGPUMailboxTest, MAYBE(UseA_UseB_DestroyA_DestroyB)) {
 // images was stored globally instead of per-device. This meant that of two
 // devices tried to create shared images with the same (id, generation) (which
 // is possible because they can be on different Dawn wires) they would conflict.
-TEST_F(WebGPUMailboxTest, MAYBE(AssociateOnTwoDevicesAtTheSameTime)) {
+TEST_F(WebGPUMailboxTest, AssociateOnTwoDevicesAtTheSameTime) {
   if (!WebGPUSupported()) {
     LOG(ERROR) << "Test skipped because WebGPU isn't supported";
     return;

gpu/command_buffer/tests/webgpu_test.cc

@@ -19,13 +19,6 @@
 #include "gpu/ipc/webgpu_in_process_context.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
-// TODO(crbug.com/1162117): gl_tests failing on Linux
-#if defined(OS_LINUX)
-#define MAYBE(test_name) DISABLED_##test_name
-#else
-#define MAYBE(test_name) test_name
-#endif
-
 namespace gpu {
 
 namespace {
@@ -181,7 +174,7 @@ WebGPUTest::DeviceAndClientID WebGPUTest::GetNewDeviceAndClientID() {
   return result;
 }
 
-TEST_F(WebGPUTest, MAYBE(FlushNoCommands)) {
+TEST_F(WebGPUTest, FlushNoCommands) {
   if (!WebGPUSupported()) {
     LOG(ERROR) << "Test skipped because WebGPU isn't supported";
     return;
@@ -193,7 +186,7 @@ TEST_F(WebGPUTest, MAYBE(FlushNoCommands)) {
 }
 
 // Referred from GLES2ImplementationTest/ReportLoss
-TEST_F(WebGPUTest, MAYBE(ReportLoss)) {
+TEST_F(WebGPUTest, ReportLoss) {
   if (!WebGPUSupported()) {
     LOG(ERROR) << "Test skipped because WebGPU isn't supported";
     return;
@@ -213,7 +206,7 @@ TEST_F(WebGPUTest, MAYBE(ReportLoss)) {
 }
 
 // Referred from GLES2ImplementationTest/ReportLossReentrant
-TEST_F(WebGPUTest, MAYBE(ReportLossReentrant)) {
+TEST_F(WebGPUTest, ReportLossReentrant) {
   if (!WebGPUSupported()) {
     LOG(ERROR) << "Test skipped because WebGPU isn't supported";
     return;
@@ -232,7 +225,7 @@ TEST_F(WebGPUTest, MAYBE(ReportLossReentrant)) {
   EXPECT_EQ(0, lost_count);
 }
 
-TEST_F(WebGPUTest, MAYBE(RequestAdapterAfterContextLost)) {
+TEST_F(WebGPUTest, RequestAdapterAfterContextLost) {
   if (!WebGPUSupported()) {
     LOG(ERROR) << "Test skipped because WebGPU isn't supported";
     return;
@@ -246,7 +239,7 @@ TEST_F(WebGPUTest, MAYBE(RequestAdapterAfterContextLost)) {
                                     base::BindOnce(&OnRequestAdapterCallback)));
 }
 
-TEST_F(WebGPUTest, MAYBE(RequestDeviceAfterContextLost)) {
+TEST_F(WebGPUTest, RequestDeviceAfterContextLost) {
   if (!WebGPUSupported()) {
     LOG(ERROR) << "Test skipped because WebGPU isn't supported";
     return;