Send certificates to devtools when it's open instead of using certId

content/renderer/devtools/devtools_agent.cc

Index: content/renderer/devtools/devtools_agent.cc
diff --git a/content/renderer/devtools/devtools_agent.cc b/content/renderer/devtools/devtools_agent.cc
index 0016d880d0b49c1bb4df7c51528afb174423c29a..25390e9db16d5006f9a06a7d827645f84ce6e62c 100644
--- a/content/renderer/devtools/devtools_agent.cc
+++ b/content/renderer/devtools/devtools_agent.cc
@@ -35,6 +35,7 @@ using blink::WebDevToolsAgentClient;
 using blink::WebLocalFrame;
 using blink::WebPoint;
 using blink::WebString;
+using blink::WebVector;
 
 using base::trace_event::TraceLog;
 
@@ -198,6 +199,15 @@ void DevToolsAgent::setCPUThrottlingRate(double rate) {
   cpu_throttler_->SetThrottlingRate(rate);
 }
 
+void DevToolsAgent::showCertificateViewer(
+    const WebVector<WebString>& certificate) {
+  std::vector<std::string> cert_vector;
+  for (auto cert : certificate)

[Ryan Sleevi, 2016/09/03 00:02:51]

STYLE nit: const auto& ? Do you really intend to be making a copy each time?

(Per the guidance on https://chromium-cpp.appspot.com/ regarding range-based for
loops)

[jam, 2016/09/03 01:42:39]

That was an oversight, but now that code is gone after Pavel's change

+    cert_vector.push_back(cert.latin1());
+  Send(new DevToolsAgentHostMsg_ShowCertificateViewer(
+      routing_id(), cert_vector));
+}
+
 // static
 DevToolsAgent* DevToolsAgent::FromRoutingId(int routing_id) {
   IdToAgentMap::iterator it = g_agent_for_routing_id.Get().find(routing_id);