[DevTools] Pass the frame payload

third_party/WebKit/Source/devtools/front_end/sdk/SecurityOriginManager.js

 // Copyright 2016 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.
 /**
  * @unrestricted
  */
 SDK.SecurityOriginManager = class extends SDK.SDKModel {
   /**
    * @param {!SDK.Target} target
    */
@@ skipping 10 matching lines @@
    */
   static fromTarget(target) {
     var securityOriginManager = target.model(SDK.SecurityOriginManager);
     if (!securityOriginManager)
       securityOriginManager = new SDK.SecurityOriginManager(target);
     return securityOriginManager;
   }
 
   /**
    * @param {string} securityOrigin
+   * @param {boolean} isMain
    */
-  addSecurityOrigin(securityOrigin) {
-    var currentCount = this._securityOriginCounter.get(securityOrigin);
-    if (!currentCount) {
-      this._securityOriginCounter.set(securityOrigin, 1);
+  addSecurityOrigin(securityOrigin, isMain) {
+    var currentCount = this._securityOriginCounter.get(securityOrigin) || 0;
+    this._securityOriginCounter.set(securityOrigin, currentCount + 1);
+    if (!currentCount)
       this.dispatchEventToListeners(SDK.SecurityOriginManager.Events.SecurityOriginAdded, securityOrigin);
+    if (!isMain)
       return;
-    }
-    this._securityOriginCounter.set(securityOrigin, currentCount + 1);
+    this._mainSecurityOrigin = securityOrigin;
+    this.dispatchEventToListeners(SDK.SecurityOriginManager.Events.MainSecurityOriginChanged, securityOrigin);
   }
 
   /**
    * @param {string} securityOrigin
    */
   removeSecurityOrigin(securityOrigin) {

[pfeldman, 2017/01/07 00:33:18]

I don't think this counter makes sense - _navigate would clean up childFrames,
but not update the securityorigin counters. What you should do instead is
traverse frames every time they change and compare sets of origins.

     var currentCount = this._securityOriginCounter.get(securityOrigin);
     if (currentCount === 1) {
       this._securityOriginCounter.delete(securityOrigin);
       this.dispatchEventToListeners(SDK.SecurityOriginManager.Events.SecurityOriginRemoved, securityOrigin);
       return;
     }
     this._securityOriginCounter.set(securityOrigin, currentCount - 1);
   }
 
   /**
    * @return {!Array<string>}
    */
   securityOrigins() {
     return this._securityOriginCounter.keysArray();
   }
 
   /**
    * @return {string}
    */
   mainSecurityOrigin() {
     return this._mainSecurityOrigin;
   }
-
-  /**
-   * @param {string} securityOrigin
-   */
-  setMainSecurityOrigin(securityOrigin) {
-    this._mainSecurityOrigin = securityOrigin;
-    this.dispatchEventToListeners(SDK.SecurityOriginManager.Events.MainSecurityOriginChanged, securityOrigin);
-  }
 };
 
 /** @enum {symbol} */
 SDK.SecurityOriginManager.Events = {
   SecurityOriginAdded: Symbol('SecurityOriginAdded'),
   SecurityOriginRemoved: Symbol('SecurityOriginRemoved'),
   MainSecurityOriginChanged: Symbol('MainSecurityOriginChanged')
 };