Add error description to the DidFailProvisionalLoad callback.

content/renderer/render_view.cc

 // Copyright (c) 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 "content/renderer/render_view.h"
 
 #include <algorithm>
 #include <cmath>
 #include <string>
 #include <vector>
@@ skipping 2322 matching lines @@
   DCHECK(ds);
 
   const WebURLRequest& failed_request = ds->request();
 
   FOR_EACH_OBSERVER(
       RenderViewObserver, observers_, DidFailProvisionalLoad(frame, error));
 
   bool show_repost_interstitial =
       (error.reason == net::ERR_CACHE_MISS &&
        EqualsASCII(failed_request.httpMethod(), "POST"));
-  Send(new ViewHostMsg_DidFailProvisionalLoadWithError(
-      routing_id_, frame->identifier(), !frame->parent(), error.reason,
-      error.unreachableURL, show_repost_interstitial));
+
+  {

[jam, 2011/10/05 17:12:06]

nit: adding an extra scope here is unnecessary. we avoid tabbing if we can

[mkosiba (inactive), 2011/10/05 17:53:18]

Done.

+    ViewHostMsg_DidFailProvisionalLoadWithError_Params ipc_params;
+    ipc_params.frame_id = frame->identifier();
+    ipc_params.is_main_frame = !frame->parent();
+    ipc_params.error_code = error.reason;
+    ipc_params.error_description = content::GetContentClient()->renderer()->
+        GetNavigationErrorDescription(failed_request, error);
+    ipc_params.url = error.unreachableURL;
+    ipc_params.showing_repost_interstitial = show_repost_interstitial;
+    Send(new ViewHostMsg_DidFailProvisionalLoadWithError(
+        routing_id_, ipc_params));
+  }
 
   // Don't display an error page if this is simply a cancelled load.  Aside
   // from being dumb, WebCore doesn't expect it and it will cause a crash.
   if (error.reason == net::ERR_ABORTED)
     return;
 
   // Make sure we never show errors in view source mode.
   frame->enableViewSourceMode(false);
 
   NavigationState* navigation_state = NavigationState::FromDataSource(ds);
@@ skipping 2045 matching lines @@
   main_frame->enableViewSourceMode(true);
 }
 
 void RenderView::OnLockMouseACK(bool succeeded) {
   pepper_delegate_.OnLockMouseACK(succeeded);
 }
 
 void RenderView::OnMouseLockLost() {
   pepper_delegate_.OnMouseLockLost();
 }
-