[DevTools] Fallback to MainFrameNavigated event url in Security panel

third_party/WebKit/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.
 /**
  * @implements {SDK.TargetManager.Observer}
  * @unrestricted
  */
 Security.SecurityPanel = class extends UI.PanelWithSidebar {
   constructor() {
     super('security');
@@ skipping 329 matching lines @@
 
     this.selectAndSwitchToMainView();
     this._sidebarTree.clearOrigins();
     this._origins.clear();
     this._lastResponseReceivedForLoaderId.clear();
     this._filterRequestCounts.clear();
     // After clearing the filtered request counts, refresh the
     // explanations to reflect the new counts.
     this._mainView.refreshExplanations();
 
-    if (request) {
-      var origin = Common.ParsedURL.extractOrigin(request.url());
-      this._sidebarTree.setMainOrigin(origin);
+    // If we could not find a matching request (as in the case of clicking
+    // through an interstitial, see crbug.com/669309), set the origin based upon
+    // the url data from the MainFrameNavigated event itself.
+    const origin = Common.ParsedURL.extractOrigin(request ? request.url() : frame.url);

[lushnikov, 2017/03/21 18:26:36]

nit: we don't use 'const' since it causes V8 deoptimization

[elawrence, 2017/03/22 01:04:41]

Done.

+    this._sidebarTree.setMainOrigin(origin);
+
+    if (request)
       this._processRequest(request);
-    }
   }
 
   _onInterstitialShown() {
     // The panel might have been displaying the origin view on the
     // previously loaded page. When showing an interstitial, switch
     // back to the Overview view.
     this.selectAndSwitchToMainView();
     this._sidebarTree.toggleOriginsList(true /* hidden */);
   }
 
@@ skipping 677 matching lines @@
     var row = this._element.createChild('div', 'details-table-row');
     row.createChild('div').textContent = key;
 
     var valueDiv = row.createChild('div');
     if (typeof value === 'string')
       valueDiv.textContent = value;
     else
       valueDiv.appendChild(value);
   }
 };