From e0e0fbb57433cf821b154e23369b1dfde3e558b8 Mon Sep 17 00:00:00 2001 From: Chris Bracken Date: Thu, 24 Oct 2019 18:29:04 -0700 Subject: [PATCH] Remove TODO on embedder a11y unit tests Removes a TODO implying that the embedder accessibility unittests were disabled due to flakiness. The tests were originally disabled in https://github.com/flutter/engine/pull/9482. The issue tracking re-enabling the tests was https://github.com/flutter/flutter/issues/35218. This issue was resolved, and the tests re-enabled, in https://github.com/flutter/engine/pull/9585. --- shell/platform/embedder/tests/embedder_a11y_unittests.cc | 3 --- 1 file changed, 3 deletions(-) diff --git a/shell/platform/embedder/tests/embedder_a11y_unittests.cc b/shell/platform/embedder/tests/embedder_a11y_unittests.cc index f9255734f6323..091069c8ed1fd 100644 --- a/shell/platform/embedder/tests/embedder_a11y_unittests.cc +++ b/shell/platform/embedder/tests/embedder_a11y_unittests.cc @@ -21,9 +21,6 @@ namespace testing { using Embedder11yTest = testing::EmbedderTest; -// TODO: This test has been disabled as it is flaky (more reproducible in -// profile more). Multiple calls to a11y changed handler in Dart code is -// suspected. https://github.com/flutter/flutter/issues/35218 TEST_F(Embedder11yTest, A11yTreeIsConsistent) { auto& context = GetEmbedderContext();