Prune some remnants of DSA client certificate support.

chrome/browser/ui/webui/cookies_tree_model_util.cc

Index: chrome/browser/ui/webui/cookies_tree_model_util.cc
diff --git a/chrome/browser/ui/webui/cookies_tree_model_util.cc b/chrome/browser/ui/webui/cookies_tree_model_util.cc
index ab13500780491fc7ee73fab3ab0a0b85ba64bd21..72bb7a923666163bcd62ea6f19e8f939c7258374 100644
--- a/chrome/browser/ui/webui/cookies_tree_model_util.cc
+++ b/chrome/browser/ui/webui/cookies_tree_model_util.cc
@@ -18,7 +18,6 @@
 #include "content/public/browser/indexed_db_context.h"
 #include "content/public/browser/service_worker_context.h"
 #include "net/cookies/canonical_cookie.h"
-#include "net/ssl/ssl_client_cert_type.h"
 #include "storage/common/fileapi/file_system_types.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/text/bytes_formatting.h"
@@ -68,19 +67,6 @@ const char kKeyScopes[] = "scopes";
 
 const int64 kNegligibleUsage = 1024;  // 1KiB
 
-std::string ClientCertTypeToString(net::SSLClientCertType type) {
-  switch (type) {
-    case net::CLIENT_CERT_RSA_SIGN:
-      return l10n_util::GetStringUTF8(IDS_CLIENT_CERT_RSA_SIGN);
-    case net::CLIENT_CERT_DSS_SIGN:
-      return l10n_util::GetStringUTF8(IDS_CLIENT_CERT_DSS_SIGN);
-    case net::CLIENT_CERT_ECDSA_SIGN:
-      return l10n_util::GetStringUTF8(IDS_CLIENT_CERT_ECDSA_SIGN);
-    default:
-      return base::IntToString(type);
-  }
-}
-
 }  // namespace
 
 CookiesTreeModelUtil::CookiesTreeModelUtil() {
@@ -258,7 +244,7 @@ bool CookiesTreeModelUtil::GetCookieTreeNodeDictionary(
 
       dict->SetString(kKeyServerId, channel_id.server_identifier());
       dict->SetString(kKeyCertType,
-                      ClientCertTypeToString(net::CLIENT_CERT_ECDSA_SIGN));
+                      l10n_util::GetStringUTF8(IDS_CLIENT_CERT_ECDSA_SIGN));

[davidben, 2015/05/12 00:22:04]

Is there any reason to localize this? It's the string "ecdsa_sign".

[Ryan Sleevi, 2015/05/12 00:56:12]

No clue. I suspect maybe they wanted to say "Elliptic Curve" or something fancy,
but never got around to it. git blame & find an Android OWNER, but my gut is
it's safe to not localize.

[davidben, 2015/05/12 21:31:01]

Looks like it's got translateable="false" anyway, but mattm tells me the rule is
that UI strings should end up in grd files even if they're not localized, so
I'll leave it as-is I guess.

       dict->SetString(kKeyCreated, base::UTF16ToUTF8(
           base::TimeFormatFriendlyDateAndTime(
               channel_id.creation_time())));