Add certificate error handling to devtools.

content/browser/web_contents/web_contents_impl.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 "content/browser/web_contents/web_contents_impl.h"
 
 #include <stddef.h>
 
 #include <cmath>
 #include <utility>
@@ skipping 1447 matching lines @@
 }
 
 void WebContentsImpl::DidChangeVisibleSecurityState() {
   if (delegate_) {
     delegate_->VisibleSecurityStateChanged(this);
     for (auto& observer : observers_)
       observer.DidChangeVisibleSecurityState();
   }
 }
 
+bool WebContentsImpl::NotifyCertificateError(
+    base::Callback<void(content::CertificateRequestResultType)> callback) {
+  bool handled = false;
+  for (auto& observer : observers_)
+    if (observer.NotifyCertificateError(callback))
+      handled = true;

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

I think we have to break the loop in this case - otherwise, we could end up with
two observers that want to handle the error / issue the callback.

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

Done.

+  return handled;
+}
+
 void WebContentsImpl::Stop() {
   for (FrameTreeNode* node : frame_tree_.Nodes())
     node->StopLoading();
   for (auto& observer : observers_)
     observer.NavigationStopped();
 }
 
 WebContents* WebContentsImpl::Clone() {
   // We use our current SiteInstance since the cloned entry will use it anyway.
   // We pass our own opener so that the cloned page can access it if it was set
@@ skipping 3941 matching lines @@
   GetMainFrame()->AddMessageToConsole(
       content::CONSOLE_MESSAGE_LEVEL_WARNING,
       base::StringPrintf("This site does not have a valid SSL "
                          "certificate! Without SSL, your site's and "
                          "visitors' data is vulnerable to theft and "
                          "tampering. Get a valid SSL certificate before"
                          " releasing your website to the public."));
 }
 
 }  // namespace content