Add error description to the DidFailProvisionalLoad callback.

content/public/renderer/content_renderer_client.h

 // 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.
 
 #ifndef CONTENT_PUBLIC_RENDERER_CONTENT_RENDERER_CLIENT_H_
 #define CONTENT_PUBLIC_RENDERER_CONTENT_RENDERER_CLIENT_H_
 #pragma once
 
 #include <string>
 
@@ skipping 54 matching lines @@
   // Give the embedder the ability to set an error page.
   virtual void ShowErrorPage(RenderView* render_view,
                              WebKit::WebFrame* frame,
                              int http_status_code) = 0;
 
   // Returns the html to display when a navigation error occurs.
   virtual std::string GetNavigationErrorHtml(
       const WebKit::WebURLRequest& failed_request,
       const WebKit::WebURLError& error) = 0;
 
+  // Returns the error message that is shown when a navigation error occurs.
+  virtual string16 GetNavigationErrorDescription(

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

why add another function instead of an extra parameter? we try to keep embedder
functions to a minimum

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

You mean have one function return both/either the error description and the
error html? Is it acceptable if the function takes out* pointers that are filled
with data only if the pointer is not null? 

[jam, 2011/10/05 19:33:57]

sure

+      const WebKit::WebURLRequest& failed_request,
+      const WebKit::WebURLError& error) = 0;
+
   // Returns true if the renderer process should schedule the idle handler when
   // all widgets are hidden.
   virtual bool RunIdleHandlerWhenWidgetsHidden() = 0;
 
   // Returns true if the given url can create popup windows.
   virtual bool AllowPopup(const GURL& creator) = 0;
 
   // Returns true if we should fork a new process for the given navigation.
   virtual bool ShouldFork(WebKit::WebFrame* frame,
                           const GURL& url,
@@ skipping 43 matching lines @@
                                       const std::string& value) = 0;
 
   // True if the protocol implemented to serve |url| supports features required
   // by the media engine.
   virtual bool IsProtocolSupportedForMedia(const GURL& url) = 0;
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_RENDERER_CONTENT_RENDERER_CLIENT_H_