Commit 71fe6221 authored by Andreas Haas's avatar Andreas Haas Committed by Commit Bot

Revert "[Sheriff] Disable PDFAnnotationsTest.AnnotationsFeatureEnabled"

This reverts commit 88941a9c.

Reason for revert: I reverted the CL that caused this issue in
https://crrev.com/c/2078544. We can re-enable this test

Original change's description:
> [Sheriff] Disable PDFAnnotationsTest.AnnotationsFeatureEnabled
>
> on all chrome os.
>
> TBR=thestig@chromium.org
>
> Bug: 1056295, 920684
> Change-Id: Ic5b6e0fc7fff5427e2a7c72ad49077713e1a7937
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2075082
> Commit-Queue: Parastoo Geranmayeh <parastoog@google.com>
> Reviewed-by: Lei Zhang <thestig@chromium.org>
> Reviewed-by: Parastoo Geranmayeh <parastoog@google.com>
> Cr-Commit-Position: refs/heads/master@{#744756}

R=thestig@chromium.org,parastoog@google.com

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

Bug: 1056295, 920684
Change-Id: Ie2c8409015ce85379ca5f0efe6e5e935ef66bd2b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2087215Reviewed-by: default avatarParastoo Geranmayeh <parastoog@google.com>
Reviewed-by: default avatarLei Zhang <thestig@chromium.org>
Reviewed-by: default avatarAndreas Haas <ahaas@chromium.org>
Commit-Queue: Andreas Haas <ahaas@chromium.org>
Cr-Commit-Position: refs/heads/master@{#747605}
parent c0a69caf
...@@ -784,7 +784,13 @@ IN_PROC_BROWSER_TEST_F(PDFExtensionTest, Beep) { ...@@ -784,7 +784,13 @@ IN_PROC_BROWSER_TEST_F(PDFExtensionTest, Beep) {
#if defined(OS_CHROMEOS) #if defined(OS_CHROMEOS)
// TODO(https://crbug.com/920684): Test times out. // TODO(https://crbug.com/920684): Test times out.
IN_PROC_BROWSER_TEST_F(PDFAnnotationsTest, DISABLED_AnnotationsFeatureEnabled) { #if defined(MEMORY_SANITIZER) || defined(LEAK_SANITIZER) || \
defined(ADDRESS_SANITIZER) || defined(_DEBUG)
#define MAYBE_AnnotationsFeatureEnabled DISABLED_AnnotationsFeatureEnabled
#else
#define MAYBE_AnnotationsFeatureEnabled AnnotationsFeatureEnabled
#endif
IN_PROC_BROWSER_TEST_F(PDFAnnotationsTest, MAYBE_AnnotationsFeatureEnabled) {
RunTestsInJsModule("annotations_feature_enabled_test.js", "test.pdf"); RunTestsInJsModule("annotations_feature_enabled_test.js", "test.pdf");
} }
......
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