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..0cbf679d48a7617b9c982f3c55dd2b72351e54e4 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");

[dgozman, 2015/09/02 19:02:41]

div -> a?

[pfeldman, 2015/09/02 21:29:13]

Needs to be block.

+            certificateAnchor.textContent = WebInspector.UIString("View certificate");
+            certificateAnchor.href = "";
+            certificateAnchor.addEventListener("click", showCertificaite, false);
+        }
+        /**
+         * @param {!Event} e
+         */
+        function showCertificaite(e)

[lgarron, 2015/09/02 01:43:38]

Nit: s/showCertificaite/showCertificate

(I'd actually also prefer showCertificateViewer(), but I don't have strong
reasons.)

[pfeldman, 2015/09/02 21:29:13]

Done.

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

[pfeldman, 2015/09/02 01:34:14]

We might want to pass the this._target from the security panel here.

[lgarron, 2015/09/02 01:43:38]

I would personally add showCertificate() to WebInspector.SecurityPanel, then
pass `panel` into the WebInspector.SecurityMainView constructor and call:

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

[dgozman, 2015/09/02 19:02:41]

Should not matter actually.

+        }
     },
 
     /**