Commit d7213a25 authored by Meredith Lane's avatar Meredith Lane Committed by Commit Bot

[Trivial] Explanatory sheriff comment for AddSupervisionBrowserTest.UMATest

Simple follow up CL to add a comment and link to bug that was mistakenly
omitted during sheriffing.

See: https://chromium-review.googlesource.com/c/chromium/src/+/1880408

TBR=tobyhuang@chromium.org

Change-Id: I4d63454c6e9f960907bfa45a05cdc92fe54f5f6f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1891812Reviewed-by: default avatarMeredith Lane <meredithl@chromium.org>
Commit-Queue: Meredith Lane <meredithl@chromium.org>
Cr-Commit-Position: refs/heads/master@{#711623}
parent e46e36a0
...@@ -204,6 +204,8 @@ IN_PROC_BROWSER_TEST_F(AddSupervisionBrowserTest, ...@@ -204,6 +204,8 @@ IN_PROC_BROWSER_TEST_F(AddSupervisionBrowserTest,
ASSERT_TRUE(ConfirmSignoutDialog::IsShowing()); ASSERT_TRUE(ConfirmSignoutDialog::IsShowing());
} }
// Disabled on ASan and LSAn builds, because it's very flaky. See
// crbug.com/1004237
#if defined(ADDRESS_SANITIZER) || defined(LEAK_SANITIZER) #if defined(ADDRESS_SANITIZER) || defined(LEAK_SANITIZER)
#define MAYBE_UMATest DISABLED_UMATest #define MAYBE_UMATest DISABLED_UMATest
#else #else
......
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