Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(612)

Unified Diff: content/browser/web_contents/web_contents_impl.cc

Issue 2639203003: Add certificate error handling to devtools. (Closed)
Patch Set: format Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/web_contents/web_contents_impl.cc
diff --git a/content/browser/web_contents/web_contents_impl.cc b/content/browser/web_contents/web_contents_impl.cc
index 9bbc2e84720ce30fa6b8a34a3f7cf8adc22c6a92..3b20abc22a8eafaf13123a1836b11bd7c8a506ab 100644
--- a/content/browser/web_contents/web_contents_impl.cc
+++ b/content/browser/web_contents/web_contents_impl.cc
@@ -1465,6 +1465,15 @@ void WebContentsImpl::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 - o
irisu 2017/02/07 23:30:19 Done.
+ return handled;
+}
+
void WebContentsImpl::Stop() {
for (FrameTreeNode* node : frame_tree_.Nodes())
node->StopLoading();

Powered by Google App Engine
This is Rietveld 408576698