[WebView] Remove onFailure from VisualStateCallback.

content/renderer/gpu/queue_message_swap_promise_unittest.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/renderer/gpu/queue_message_swap_promise.h"
 
 #include <vector>
 
 #include "base/memory/scoped_vector.h"
 #include "cc/base/swap_promise.h"
@@ skipping 276 matching lines @@
 
   EXPECT_TRUE(NextSwapMessages().empty());
   EXPECT_TRUE(frame_swap_message_queue_->Empty());
 }
 
 TEST_F(QueueMessageSwapPromiseTest, VisalStateSwapPromiseDidNotSwapNoUpdate) {
   VisualStateSwapPromiseDidNotSwap(cc::SwapPromise::COMMIT_NO_UPDATE);
 }
 
 TEST_F(QueueMessageSwapPromiseTest,
-       VisalStateSwapPromiseDidNotSwapCommitFails) {
-  VisualStateSwapPromiseDidNotSwap(cc::SwapPromise::COMMIT_FAILS);
+       VisualStateSwapPromiseDidNotSwapCommitFails) {
+  // COMMIT_FAILS is treated differently:
+  //    If we fail to swap with COMMIT_FAILS, then the renderer is
+  //    shutting down, which implies that the RenderFrameHostImpl
+  //    destructor will eventually be called, firing the remaining
+  //    response callbacks (with swap_success = false) itself.
+  QueueMessageData data[] = {
+    /* { policy, source_frame_number } */
+    {MESSAGE_DELIVERY_POLICY_WITH_VISUAL_STATE, 1},
+    {MESSAGE_DELIVERY_POLICY_WITH_VISUAL_STATE, 1},
+    {MESSAGE_DELIVERY_POLICY_WITH_VISUAL_STATE, 2},
+  };
+  QueueMessages(data, arraysize(data));
+
+  promises_[0]->DidNotSwap(cc::SwapPromise::COMMIT_FAILS);
+  ASSERT_FALSE(promises_[1]);
+  EXPECT_TRUE(NextSwapMessages().empty());
+  EXPECT_EQ(0u, DirectSendMessages().size());
+  EXPECT_FALSE(ContainsMessage(DirectSendMessages(), messages_[0]));
+  EXPECT_FALSE(ContainsMessage(DirectSendMessages(), messages_[1]));
+  EXPECT_FALSE(ContainsMessage(DirectSendMessages(), messages_[2]));
+
+  promises_[2]->DidNotSwap(cc::SwapPromise::COMMIT_FAILS);
+  EXPECT_TRUE(NextSwapMessages().empty());
+  EXPECT_FALSE(ContainsMessage(DirectSendMessages(), messages_[2]));
+
+  EXPECT_TRUE(NextSwapMessages().empty());
+  EXPECT_FALSE(frame_swap_message_queue_->Empty());
 }
 
 TEST_F(QueueMessageSwapPromiseTest, VisalStateSwapPromiseDidNotSwapSwapFails) {
   VisualStateSwapPromiseDidNotSwap(cc::SwapPromise::SWAP_FAILS);
 }
 
 }  // namespace content