Remove incorrect isAcceleratedCompositingActive check

content/shell/renderer/test_runner/web_test_proxy.cc

 // Copyright 2014 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 "content/shell/renderer/test_runner/web_test_proxy.h"
 
 #include <cctype>
 
 #include "base/callback_helpers.h"
 #include "base/command_line.h"
@@ skipping 471 matching lines @@
 
   if (notify)
     GetWebView()->acceptLanguagesChanged();
 }
 
 void WebTestProxyBase::CopyImageAtAndCapturePixels(
     int x, int y, const base::Callback<void(const SkBitmap&)>& callback) {
   // It may happen that there is a scheduled animation and
   // no rootGraphicsLayer yet. If so we would run it right now. Otherwise
   // isAcceleratedCompositingActive will return false;
   AnimateNow();

[danakj, 2014/09/24 14:27:38]

Can you also remove all these AnimateNow()s that were added to make the checks
not explode? Maybe once the blink side is fixed?

[enne (OOO), 2014/09/24 16:44:55]

I talked to loislo about this and am going to conservatively wait and remove it
next week.  I think there was a desire to make sure that the AnimateNow did
actually fix the flakiness even if the approach wasn't great, rather than
changing to an untested solution #2 really quickly.

 
-  DCHECK(web_widget_->isAcceleratedCompositingActive());
   DCHECK(!callback.is_null());
   uint64_t sequence_number =  blink::Platform::current()->clipboard()->
       sequenceNumber(blink::WebClipboard::Buffer());
   GetWebView()->copyImageAt(blink::WebPoint(x, y));
   if (sequence_number == blink::Platform::current()->clipboard()->
       sequenceNumber(blink::WebClipboard::Buffer())) {
     SkBitmap emptyBitmap;
     callback.Run(emptyBitmap);
     return;
   }
@@ skipping 64 matching lines @@
 
 void WebTestProxyBase::CapturePixelsAsync(
     const base::Callback<void(const SkBitmap&)>& callback) {
   TRACE_EVENT0("shell", "WebTestProxyBase::CapturePixelsAsync");
 
   // It may happen that there is a scheduled animation and
   // no rootGraphicsLayer yet. If so we would run it right now. Otherwise
   // isAcceleratedCompositingActive will return false;
   AnimateNow();
 
-  DCHECK(web_widget_->isAcceleratedCompositingActive());
   DCHECK(!callback.is_null());
 
   if (test_interfaces_->GetTestRunner()->isPrinting()) {
     base::MessageLoopProxy::current()->PostTask(
         FROM_HERE,
         base::Bind(&WebTestProxyBase::CapturePixelsForPrinting,
                    base::Unretained(this),
                    callback));
     return;
   }
@@ skipping 763 matching lines @@
   if (!push_client_.get())
     push_client_.reset(new MockWebPushClient);
   return push_client_.get();
 }
 
 blink::WebPushClient* WebTestProxyBase::GetWebPushClient() {
   return GetPushClientMock();
 }
 
 }  // namespace content