Enable test LayerTreeHostTestBreakSwapPromise with verbose logging information

cc/trees/layer_tree_host_unittest.cc

 // Copyright 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "cc/trees/layer_tree_host.h"
 
 #include <algorithm>
 
 #include "base/auto_reset.h"
 #include "base/synchronization/lock.h"
@@ skipping 4450 matching lines @@
   SwapPromise::DidNotSwapReason reason;
   base::Lock lock;
 };
 
 class TestSwapPromise : public SwapPromise {
  public:
   explicit TestSwapPromise(TestSwapPromiseResult* result) : result_(result) {}
 
   virtual ~TestSwapPromise() {
     base::AutoLock lock(result_->lock);
+    LOG(INFO) << "~TestSwapPromise() "

[danakj, 2014/08/13 23:09:07]

i suggest LOG(ERROR), maybe this works too but i've always used ERROR.

[Yufeng Shen (Slow to review), 2014/08/13 23:25:58]

Done.

+              << " did_swap_called " << result_->did_swap_called
+              << " did_not_swap_called " << result_->did_not_swap_called
+              << " result addr " << result_;
     result_->dtor_called = true;
   }
 
   virtual void DidSwap(CompositorFrameMetadata* metadata) OVERRIDE {
     base::AutoLock lock(result_->lock);
+    LOG(INFO) << "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_not_swap_called);
     result_->did_swap_called = true;
   }
 
   virtual void DidNotSwap(DidNotSwapReason reason) OVERRIDE {
     base::AutoLock lock(result_->lock);
+    LOG(INFO) << "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_not_swap_called);
     result_->did_not_swap_called = true;
     result_->reason = reason;
   }
 
  private:
   // Not owned.
   TestSwapPromiseResult* result_;
 };
@@ skipping 63 matching lines @@
       EXPECT_TRUE(swap_promise_result_[2].dtor_called);
     }
   }
 
   int commit_count_;
   int commit_complete_count_;
   TestSwapPromiseResult swap_promise_result_[3];
 };
 
 // TODO(miletus): Flaky test: crbug.com/393995
-// MULTI_THREAD_TEST_F(LayerTreeHostTestBreakSwapPromise);
+// Enabled with verbose logging information.
+MULTI_THREAD_TEST_F(LayerTreeHostTestBreakSwapPromise);
 
 class LayerTreeHostTestBreakSwapPromiseForVisibilityAbortedCommit
     : public LayerTreeHostTest {
  protected:
   LayerTreeHostTestBreakSwapPromiseForVisibilityAbortedCommit()
       : commit_count_(0) {}
 
   virtual void BeginTest() OVERRIDE { PostSetNeedsCommitToMainThread(); }
 
   virtual void WillBeginMainFrame() OVERRIDE {
@@ skipping 464 matching lines @@
   const gfx::Size bounds_;
   FakeContentLayerClient client_;
   scoped_refptr<ContentLayerWithUpdateTracking> content_layer_;
   scoped_refptr<FakePictureLayer> picture_layer_;
   Layer* child_layer_;
 };
 
 MULTI_THREAD_TEST_F(LayerTreeHostTestContinuousPainting);
 
 }  // namespace cc