[LocalizedError] Add TODO about error suggestions on mobile

components/error_page/common/localized_error.cc

 // Copyright (c) 2012 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 "components/error_page/common/localized_error.h"
 
 #include <stddef.h>
 
 #include "base/command_line.h"
 #include "base/i18n/rtl.h"
@@ skipping 819 matching lines @@
     suggest_network_prediction->SetString("body",
         l10n_util::GetStringUTF16(
             IDS_ERRORPAGES_SUGGESTION_NETWORK_PREDICTION_BODY));
     suggest_network_prediction->SetString(
         "noNetworkPredictionTitle",
         l10n_util::GetStringUTF16(
             IDS_NETWORK_PREDICTION_ENABLED_DESCRIPTION));
     suggestions->Append(suggest_network_prediction);
   }
 
+  // TODO(crbug.com/584615): Does it make sense to show all of these
+  // suggestions on mobile? Several of them seem irrelevant in the mobile
+  // context.
   if (options.suggestions & SUGGEST_FIREWALL_CONFIG) {
     base::DictionaryValue* suggest_firewall_config = new base::DictionaryValue;
     suggest_firewall_config->SetString("header",
         l10n_util::GetStringUTF16(
             IDS_ERRORPAGES_SUGGESTION_FIREWALL_CONFIG_HEADER));
     suggest_firewall_config->SetString("body",
         l10n_util::GetStringUTF16(
             IDS_ERRORPAGES_SUGGESTION_FIREWALL_CONFIG_BODY));
     suggestions->Append(suggest_firewall_config);
   }
@@ skipping 99 matching lines @@
 
 bool LocalizedError::HasStrings(const std::string& error_domain,
                                 int error_code) {
   // Whether or not the there are strings for an error does not depend on
   // whether or not the page was be generated by a POST, so just claim it was
   // not.
   return LookupErrorMap(error_domain, error_code, /*is_post=*/false) != NULL;
 }
 
 }  // namespace error_page