DevTools: [security] open certificate viewer from devtools security overview (blink)

Source/devtools/front_end/security/SecurityPanel.js

Index: Source/devtools/front_end/security/SecurityPanel.js
diff --git a/Source/devtools/front_end/security/SecurityPanel.js b/Source/devtools/front_end/security/SecurityPanel.js
index 166d8aa84258b5fcb5123b40355ac0b759f95b4e..3c8ec67f3c58ba49db5baf4495e3b80888b50a9e 100644
--- a/Source/devtools/front_end/security/SecurityPanel.js
+++ b/Source/devtools/front_end/security/SecurityPanel.js
@@ -356,6 +356,20 @@ WebInspector.SecurityMainView.prototype = {
         var text = explanationSection.createChild("div", "security-section-text");
         text.createChild("div", "security-section-title").textContent = explanation.summary;
         text.createChild("div", "security-explanation").textContent = explanation.description;
+        if ("certificateId" in explanation) {
+            var certificateAnchor = text.createChild("div", "security-certificate-id link");
+            certificateAnchor.textContent = WebInspector.UIString("View certificate");
+            certificateAnchor.href = "";
+            certificateAnchor.addEventListener("click", showCertificateViewer, false);
+        }
+        /**
+         * @param {!Event} e
+         */
+        function showCertificateViewer(e)

[lgarron, 2015/09/03 02:15:43]

I think you need:


certificateAnchor.addEventListener("click", this.showCertificateViewer, false);

[pfeldman, 2015/09/03 20:32:55]

No, I don't. the reason is that showCertificateViewer needs to land first.

+        {
+            e.consume();
+            WebInspector.targetManager.mainTarget().networkManager.showCertificateViewer(/** @type {number} */ (explanation.certificateId));
+        }
     },
 
     /**