Commit 40c813ce authored by Owen Min's avatar Owen Min Committed by Commit Bot

Disable two compositor layer tests

Disable
LayerWithDelegateTest.Cloning
LayerWithNullDelegateTest.SwitchLayerPreservesCCLayerState
for Linux and UNDEFINED_SANITIZER due to bot UBSanVptr Linux.

Bug: 989936
Change-Id: Ieb732e3ded4efd15277dee6b210336b58201242d
Tbr: flackr@chromium.org
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1731756Reviewed-by: default avatarOwen Min <zmin@chromium.org>
Commit-Queue: Owen Min <zmin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#683327}
parent 9089599f
...@@ -774,7 +774,13 @@ TEST_P(LayerWithRealCompositorTest, HierarchyNoTexture) { ...@@ -774,7 +774,13 @@ TEST_P(LayerWithRealCompositorTest, HierarchyNoTexture) {
EXPECT_TRUE(d3.painted()); EXPECT_TRUE(d3.painted());
} }
TEST_F(LayerWithDelegateTest, Cloning) { // crbug.com/989936
#if defined(OS_LINUX) && defined(UNDEFINED_SANITIZER)
#define MAYBE_Cloning DISABLED_Cloning
#else
#define MAYBE_Cloning Cloning
#endif
TEST_F(LayerWithDelegateTest, MAYBE_Cloning) {
std::unique_ptr<Layer> layer = CreateLayer(LAYER_SOLID_COLOR); std::unique_ptr<Layer> layer = CreateLayer(LAYER_SOLID_COLOR);
gfx::Transform transform; gfx::Transform transform;
...@@ -1039,8 +1045,14 @@ TEST_F(LayerWithNullDelegateTest, EscapedDebugNames) { ...@@ -1039,8 +1045,14 @@ TEST_F(LayerWithNullDelegateTest, EscapedDebugNames) {
ASSERT_TRUE(roundtrip); ASSERT_TRUE(roundtrip);
EXPECT_EQ(name, *roundtrip); EXPECT_EQ(name, *roundtrip);
} }
// crbug.com/989936
TEST_F(LayerWithNullDelegateTest, SwitchLayerPreservesCCLayerState) { #if defined(OS_LINUX) && defined(UNDEFINED_SANITIZER)
#define MAYBE_SwitchLayerPreservesCCLayerState \
DISABLED_SwitchLayerPreservesCCLayerState
#else
#define MAYBE_SwitchLayerPreservesCCLayerState SwitchLayerPreservesCCLayerState
#endif
TEST_F(LayerWithNullDelegateTest, MAYBE_SwitchLayerPreservesCCLayerState) {
std::unique_ptr<Layer> l1 = CreateLayer(LAYER_SOLID_COLOR); std::unique_ptr<Layer> l1 = CreateLayer(LAYER_SOLID_COLOR);
l1->SetFillsBoundsOpaquely(true); l1->SetFillsBoundsOpaquely(true);
l1->SetVisible(false); l1->SetVisible(false);
......
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