[WebView] Remove onFailure from VisualStateCallback.

content/renderer/gpu/frame_swap_message_queue.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/frame_swap_message_queue.h"
 
 #include <limits>
 
 #include "base/containers/hash_tables.h"
 #include "base/logging.h"
@@ skipping 143 matching lines @@
 }
 
 void FrameSwapMessageQueue::DidNotSwap(int source_frame_number,
                                        cc::SwapPromise::DidNotSwapReason reason,
                                        ScopedVector<IPC::Message>* messages) {
   base::AutoLock lock(lock_);
   switch (reason) {
     case cc::SwapPromise::SWAP_FAILS:
     case cc::SwapPromise::COMMIT_NO_UPDATE:
       swap_queue_->DrainMessages(source_frame_number, messages);
-    // fallthrough
+      visual_state_queue_->DrainMessages(source_frame_number, messages);
+      break;
     case cc::SwapPromise::COMMIT_FAILS:
-      visual_state_queue_->DrainMessages(source_frame_number, messages);
+      // Do not queue any responses here.
+      // If COMMIT_FAILS the renderer is shutting down, which will
+      // result in the RenderFrameHostImpl destructor firing the
+      // remaining response callbacks itself.
       break;
     default:
       NOTREACHED();
   }
 }
 
 void FrameSwapMessageQueue::DrainMessages(
     ScopedVector<IPC::Message>* messages) {
   lock_.AssertAcquired();
 
@@ skipping 15 matching lines @@
   for (vector<IPC::Message*>::iterator i = source.begin(); i != source.end();
        ++i) {
     IPC::Message* m(*i);
     dest->push_back(*m);
     delete m;
   }
   source.weak_clear();
 }
 
 }  // namespace content