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

Source/devtools/front_end/security/SecurityPanel.js

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 /**
  * @constructor
  * @extends {WebInspector.PanelWithSidebar}
  * @implements {WebInspector.TargetManager.Observer}
  */
 WebInspector.SecurityPanel = function()
@@ skipping 338 matching lines @@
      */
     _addExplanation: function(explanation)
     {
         var explanationSection = this._securityExplanations.createChild("div", "security-section");
         explanationSection.classList.add("security-explanation");
 
         explanationSection.createChild("div", "lock-icon").classList.add("lock-icon-" + explanation.securityState);
         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));
+        }
     },
 
     /**
      * @param {!SecurityAgent.SecurityState} newSecurityState
      * @param {!Array<!SecurityAgent.SecurityStateExplanation>} explanations
      */
     updateSecurityState: function(newSecurityState, explanations)
     {
         // Remove old state.
         // It's safe to call this even when this._securityState is undefined.
@@ skipping 157 matching lines @@
 
         var valueDiv = row.createChild("div");
         if (value instanceof HTMLDivElement) {
             valueDiv.appendChild(value);
         } else {
             valueDiv.textContent = value;
         }
     }
 }