Switch to using the new Link Doctor API.

chrome/common/localized_error.h

Index: chrome/common/localized_error.h
===================================================================
--- chrome/common/localized_error.h	(revision 248295)
+++ chrome/common/localized_error.h	(working copy)
@@ -8,12 +8,13 @@
 #include <string>
 
 #include "base/basictypes.h"
+#include "base/memory/scoped_ptr.h"
 #include "base/strings/string16.h"
+#include "url/gurl.h"
 
-class GURL;
-
 namespace base {
 class DictionaryValue;
+class ListValue;
 }
 
 namespace extensions {
@@ -26,6 +27,26 @@
 
 class LocalizedError {
  public:
+  // Optional parameters that affect the display of an error page.
+  struct ErrorPageParams {
+    ErrorPageParams();
+    ~ErrorPageParams();
+
+    // Overrides whether reloading is suggested.
+    bool suggest_reload;
+
+    // Overrides default suggestions.  Each entry must contain a header and may
+    // optionally contain a body as well.
+    scoped_ptr<base::ListValue> override_suggestions;
+
+    // Prefix to prepend to search terms.  Search box is only shown if this is

[Deprecated (see juliatuttle), 2014/02/04 17:44:21]

Chrome prompts the user for a search engine at first setup -- can we respect
that here?

[mmenke, 2014/02/04 19:43:56]

No, we can't, without a fair bit of extra work.  We'd have to support POSTs,
GETs, different search parameter names, filling in multiple places in URLs with
the search parameter, weird google params (Like {google:baseURL}), etc.  The
code to to all that is currently based on the profile, and is exclusive to the
browser process.

We could add search information in yet another format, but then custom set
search engines wouldn't work, and we'd need to include that information for all
search engines we suggest (Which vary by locale).

While I agree that we ideally should do this, I don't think it should block
shutting down the current Link Doctor service.  Also not sure it's something the
networking team should really be worrying about.  May be simplest to just remove
it, if the default search engine isn't Google.

+    // a valid url.  Currently only Google searches are supported (Must use
+    // GET with a query parameter named "q").
+    GURL search_url;
+    // Default search terms.  Ignored if |search_prefix| is invalid.
+    std::string search_terms;
+  };
+
   // Fills |error_strings| with values to be used to build an error page used
   // on HTTP errors, like 404 or connection reset.
   static void GetStrings(int error_code,
@@ -34,6 +55,7 @@
                          bool is_post,
                          const std::string& locale,
                          const std::string& accept_languages,
+                         scoped_ptr<ErrorPageParams> params,
                          base::DictionaryValue* strings);
 
   // Returns a description of the encountered error.