[go: nahoru, domu]

[M123] Gardener: Disable flaky test in WebContentsVideoCaptureDeviceBrowserTest

Disabled:
WebContentsVideoCaptureDeviceBrowserTest.ChangesTargettedRenderView
WebContentsVideoCaptureDeviceBrowserTest.DeliversRefreshFramesUponRequest
WebContentsVideoCaptureDeviceBrowserTest.RecoversAfterRendererCrash

(cherry picked from commit 40c7672a2f45b1afc26e6c9b46c6a8b2e61c8e92)

Bug: 328658521
Change-Id: I7255df195fd0e5207769055a0c41b73eade1b829
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5353536
Reviewed-by: Alvin Ji <alvinji@chromium.org>
Reviewed-by: Adithya Srinivasan <adithyas@chromium.org>
Commit-Queue: Alvin Ji <alvinji@chromium.org>
Owners-Override: Adithya Srinivasan <adithyas@chromium.org>
Cr-Original-Commit-Position: refs/heads/main@{#1269867}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5368696
Commit-Queue: Phillis Tang <phillis@chromium.org>
Auto-Submit: Elaine Chien <elainechien@chromium.org>
Owners-Override: Elaine Chien <elainechien@google.com>
Reviewed-by: Phillis Tang <phillis@chromium.org>
Cr-Commit-Position: refs/branch-heads/6312@{#566}
Cr-Branched-From: 6711dcdae48edaf98cbc6964f90fac85b7d9986e-refs/heads/main@{#1262506}
diff --git a/content/browser/media/capture/web_contents_video_capture_device_browsertest.cc b/content/browser/media/capture/web_contents_video_capture_device_browsertest.cc
index a567bd9a..46bfaa71 100644
--- a/content/browser/media/capture/web_contents_video_capture_device_browsertest.cc
+++ b/content/browser/media/capture/web_contents_video_capture_device_browsertest.cc
@@ -342,7 +342,8 @@
 // changes.
 // TODO(crbug.com/40947039): Fails with MSAN. Determine if enabling the test for
 // MSAN is feasible or not
-#if defined(MEMORY_SANITIZER)
+// TODO(crbug.com/328658521): It is also flaky on macOS.
+#if defined(MEMORY_SANITIZER) || BUILDFLAG(IS_MAC)
 #define MAYBE_ChangesTargettedRenderView DISABLED_ChangesTargettedRenderView
 #else
 #define MAYBE_ChangesTargettedRenderView ChangesTargettedRenderView
@@ -422,7 +423,8 @@
 // reload. Regression test for http://crbug.com/916332.
 // TODO(crbug.com/40947039): Fails with MSAN. Determine if enabling the test for
 // MSAN is feasible or not
-#if defined(MEMORY_SANITIZER)
+// TODO(crbug.com/328658521): It is also flaky on macOS.
+#if defined(MEMORY_SANITIZER) || BUILDFLAG(IS_MAC)
 #define MAYBE_RecoversAfterRendererCrash DISABLED_RecoversAfterRendererCrash
 #else
 #define MAYBE_RecoversAfterRendererCrash RecoversAfterRendererCrash
@@ -501,7 +503,8 @@
 // content is not changing.
 // TODO(crbug.com/40947039): Fails with MSAN. Determine if enabling the test for
 // MSAN is feasible or not
-#if defined(MEMORY_SANITIZER)
+// TODO(crbug.com/328658521): It is also flaky on macOS.
+#if defined(MEMORY_SANITIZER) || BUILDFLAG(IS_MAC)
 #define MAYBE_DeliversRefreshFramesUponRequest \
   DISABLED_DeliversRefreshFramesUponRequest
 #else