[WebView] Remove onFailure from VisualStateCallback.

content/browser/frame_host/render_frame_host_impl.cc

Index: content/browser/frame_host/render_frame_host_impl.cc
diff --git a/content/browser/frame_host/render_frame_host_impl.cc b/content/browser/frame_host/render_frame_host_impl.cc
index e6cc05197d3a25613468d2938787f3f54da90757..ed63dd08b1830d2c11279307a20b395eb83c7d7d 100644
--- a/content/browser/frame_host/render_frame_host_impl.cc
+++ b/content/browser/frame_host/render_frame_host_impl.cc
@@ -203,7 +203,7 @@ RenderFrameHostImpl::~RenderFrameHostImpl() {
   swapout_event_monitor_timeout_.reset();
 
   for (const auto& iter: visual_state_callbacks_) {
-    iter.second.Run(false);
+    iter.second.Run();

[piman, 2015/02/24 21:51:23]

I think we want to keep the bool here. We would want to differentiate success vs
failure.

   }
 
   if (render_widget_host_)
@@ -1127,7 +1127,7 @@ void RenderFrameHostImpl::OnJavaScriptExecuteResponse(
 void RenderFrameHostImpl::OnVisualStateResponse(uint64 id) {
   auto it = visual_state_callbacks_.find(id);
   if (it != visual_state_callbacks_.end()) {
-    it->second.Run(true);
+    it->second.Run();
     visual_state_callbacks_.erase(it);
   } else {
     NOTREACHED() << "Received script response for unknown request";