Commit b45c86fd authored by miletus's avatar miletus Committed by Commit bot

Remove debug logging in LayerTreeHostTestBreakSwapPromise

The flaky test recovers by itself somehow. Remove the debug logging.

BUG=393995
TBR=danakj@chromium.org

Review URL: https://codereview.chromium.org/574813002

Cr-Commit-Position: refs/heads/master@{#295525}
parent 3c23f4a1
...@@ -4425,19 +4425,11 @@ class TestSwapPromise : public SwapPromise { ...@@ -4425,19 +4425,11 @@ class TestSwapPromise : public SwapPromise {
virtual ~TestSwapPromise() { virtual ~TestSwapPromise() {
base::AutoLock lock(result_->lock); base::AutoLock lock(result_->lock);
LOG(ERROR) << "~TestSwapPromise() "
<< " did_swap_called " << result_->did_swap_called
<< " did_not_swap_called " << result_->did_not_swap_called
<< " result addr " << result_;
result_->dtor_called = true; result_->dtor_called = true;
} }
virtual void DidSwap(CompositorFrameMetadata* metadata) OVERRIDE { virtual void DidSwap(CompositorFrameMetadata* metadata) OVERRIDE {
base::AutoLock lock(result_->lock); base::AutoLock lock(result_->lock);
LOG(ERROR) << "TestSwapPromise::DidSwap "
<< " did_swap_called " << result_->did_swap_called
<< " did_not_swap_called " << result_->did_not_swap_called
<< " result addr " << result_;
EXPECT_FALSE(result_->did_swap_called); EXPECT_FALSE(result_->did_swap_called);
EXPECT_FALSE(result_->did_not_swap_called); EXPECT_FALSE(result_->did_not_swap_called);
result_->did_swap_called = true; result_->did_swap_called = true;
...@@ -4445,11 +4437,6 @@ class TestSwapPromise : public SwapPromise { ...@@ -4445,11 +4437,6 @@ class TestSwapPromise : public SwapPromise {
virtual void DidNotSwap(DidNotSwapReason reason) OVERRIDE { virtual void DidNotSwap(DidNotSwapReason reason) OVERRIDE {
base::AutoLock lock(result_->lock); base::AutoLock lock(result_->lock);
LOG(ERROR) << "TestSwapPromise::DidNotSwap "
<< " reason " << reason
<< " did_swap_called " << result_->did_swap_called
<< " did_not_swap_called " << result_->did_not_swap_called
<< " result addr " << result_;
EXPECT_FALSE(result_->did_swap_called); EXPECT_FALSE(result_->did_swap_called);
EXPECT_FALSE(result_->did_not_swap_called); EXPECT_FALSE(result_->did_not_swap_called);
result_->did_not_swap_called = true; result_->did_not_swap_called = true;
...@@ -4534,8 +4521,6 @@ class LayerTreeHostTestBreakSwapPromise : public LayerTreeHostTest { ...@@ -4534,8 +4521,6 @@ class LayerTreeHostTestBreakSwapPromise : public LayerTreeHostTest {
TestSwapPromiseResult swap_promise_result_[3]; TestSwapPromiseResult swap_promise_result_[3];
}; };
// TODO(miletus): Flaky test: crbug.com/393995
// Enabled with verbose logging information.
MULTI_THREAD_TEST_F(LayerTreeHostTestBreakSwapPromise); MULTI_THREAD_TEST_F(LayerTreeHostTestBreakSwapPromise);
class LayerTreeHostTestBreakSwapPromiseForVisibilityAbortedCommit class LayerTreeHostTestBreakSwapPromiseForVisibilityAbortedCommit
......
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