Add certificate error handling to devtools.

content/public/browser/web_contents.h

Index: content/public/browser/web_contents.h
diff --git a/content/public/browser/web_contents.h b/content/public/browser/web_contents.h
index a5e25357162ed23c75c10f613567c16cf8d84e25..10b8acf26cf81f0d2384840a17d2f810bbbffa5a 100644
--- a/content/public/browser/web_contents.h
+++ b/content/public/browser/web_contents.h
@@ -17,6 +17,7 @@
 #include "base/supports_user_data.h"
 #include "build/build_config.h"
 #include "content/common/content_export.h"
+#include "content/public/browser/certificate_request_result_type.h"
 #include "content/public/browser/invalidate_type.h"
 #include "content/public/browser/navigation_controller.h"
 #include "content/public/browser/page_navigator.h"
@@ -432,6 +433,11 @@ class WebContents : public PageNavigator,
   // Invoked when visible security state changes.
   virtual void DidChangeVisibleSecurityState() = 0;
 
+  using CertErrorHandler =

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

nit: maybe CertErrorCallback is more descriptive of what this is?

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

Done.

+      base::Callback<void(content::CertificateRequestResultType)>;
+  // Invoked when a certificate error occurs.

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

nit: mention in comment that this is to give observers a chance to handle the
error and that return value indicates whether the certificate error will be
handled by one of the observers.

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

Done.

+  virtual bool NotifyCertificateError(CertErrorHandler handler);

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

nit: how about making this pure-virtual, i.e.
virtual bool NotifyCertificateError(CertErrorHandler handler) = 0;

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

Done.

+
   // Commands ------------------------------------------------------------------
 
   // Stop any pending navigation.