Stop revoking cert exceptions on resources loaded from cache

content/browser/web_contents/web_contents_impl.cc

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 26e6d0a1768a028b7e8fab6b4c62e8a64d574b45..a25b0a0613b78d2c01c718baf5884e56274a90a5 100644
--- a/content/browser/web_contents/web_contents_impl.cc
+++ b/content/browser/web_contents/web_contents_impl.cc
@@ -3460,9 +3460,6 @@ void WebContentsImpl::OnDidLoadResourceFromMemoryCache(
   LoadFromMemoryCacheDetails details(
       url, status.cert_id, status.cert_status, http_method, mime_type,
       resource_type);
-
-  controller_.ssl_manager()->DidLoadFromMemoryCache(details);
-
   FOR_EACH_OBSERVER(WebContentsObserver, observers_,
                     DidLoadResourceFromMemoryCache(details));

[jww, 2016/08/09 02:42:18]

Side question: why isn't SSLManager a WebContentsObserver, given that it seems
to implement several methods that are called identically right before the
WebContentsObserver methods are called? Would it simplify future method changes
if we just made it a WebContentsObserver?

If the answer is 'yes,' definitely not a blocker, but we may want to file a bug.

[felt, 2016/08/10 14:17:18]

That's a great question. I was wondering that too. I don't see any reason why.