Downgrade lock icon for broken-HTTPS subresources

content/common/frame_messages.h

Index: content/common/frame_messages.h
diff --git a/content/common/frame_messages.h b/content/common/frame_messages.h
index a2766fd2f8b50d5f0cd0b10c3260f7348a53feaf..da98e40e247e71fafed566df8c34b1907c6fbb08 100644
--- a/content/common/frame_messages.h
+++ b/content/common/frame_messages.h
@@ -1212,6 +1212,19 @@ IPC_MESSAGE_ROUTED2(FrameHostMsg_DidRunInsecureContent,
                     std::string /* security_origin */,
                     GURL /* target URL */)
 
+// Sent when the renderer displays content that was loaded with
+// certificate errors.
+IPC_MESSAGE_ROUTED2(FrameHostMsg_DidDisplayContentWithCertificateErrors,
+                    GURL /* resource url */,
+                    std::string /* serialized security info */)
+
+// Sent when the renderer runs content that was loaded with certificate
+// errors.
+IPC_MESSAGE_ROUTED3(FrameHostMsg_DidRunContentWithCertificateErrors,
+                    std::string /* security_origin */,

[jww, 2015/11/20 01:25:08]

While you're digging around in here... :-) Can you convert this std::string for
security_origin to a GURL? Given that it originates in a WebURL, and I believe
you've removed the only other calling location, and ultimately, in it's one use
spot
(https://code.google.com/p/chromium/codesearch#chromium/src/content/browser/ss...)
it's converted *back* to a GURL, I think it would be better to just leave it
as-is. I've found this string security_origin a source of confusion over the
years.

[estark, 2015/11/23 23:40:24]

Done.

+                    GURL /* resource url */,
+                    std::string /* serialized security info */)
+
 // Response to FrameMsg_GetSavableResourceLinks.
 IPC_MESSAGE_ROUTED3(FrameHostMsg_SavableResourceLinksResponse,
                     std::vector<GURL> /* savable resource links */,