Add certificate error handling to devtools.

content/browser/devtools/protocol/security_handler.h

 // Copyright 2015 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_BROWSER_DEVTOOLS_PROTOCOL_SECURITY_HANDLER_H_
 #define CONTENT_BROWSER_DEVTOOLS_PROTOCOL_SECURITY_HANDLER_H_
 
+#include <unordered_map>
+
 #include "base/macros.h"
 #include "content/browser/devtools/protocol/devtools_domain_handler.h"
 #include "content/browser/devtools/protocol/security.h"
+#include "content/public/browser/certificate_request_result_type.h"
 #include "content/public/browser/web_contents_observer.h"
 
 namespace content {
 
 class RenderFrameHostImpl;
 
 namespace protocol {
 
 class SecurityHandler : public DevToolsDomainHandler,
                         public Security::Backend,
                         public WebContentsObserver {
  public:
   SecurityHandler();
   ~SecurityHandler() override;
 
   void Wire(UberDispatcher* dispatcher) override;
   void SetRenderFrameHost(RenderFrameHostImpl* host) override;
 
   Response Enable() override;
   Response Disable() override;
   Response ShowCertificateViewer() override;
+  Response HandleCertificateError(int, const String&) override;

[Eric Seckler, 2017/01/19 11:42:28]

nit: int event_id, const String& action

(I think the style guide regarding parameter names in func def is to only omit
them if they're obvious.)

[irisu, 2017/02/07 23:30:19]

Done.

 
  private:
   void AttachToRenderFrameHost();
 
+  using CertErrorHandlerMap = std::unordered_map<int, CertErrorHandler>;
+
   // WebContentsObserver overrides
   void DidChangeVisibleSecurityState() override;
+  bool NotifyCertificateError(CertErrorHandler) override;
 
   std::unique_ptr<Security::Frontend> frontend_;
   bool enabled_;
   RenderFrameHostImpl* host_;
+  int cert_error_id_;

[Eric Seckler, 2017/01/19 11:42:28]

nit: next_cert_error_id_ 

We should probably also init this in constructor to some sensible value (0?).

[irisu, 2017/02/07 23:30:19]

Done.

+  CertErrorHandlerMap handlers_;
 
   DISALLOW_COPY_AND_ASSIGN(SecurityHandler);
 };
 
 }  // namespace protocol
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_DEVTOOLS_PROTOCOL_SECURITY_HANDLER_H_