[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
    */
   constructor(target) {
     super(SDK.SecurityOriginManager, target);
 
-    this._securityOriginCounter = new Map();
+    /** @type {!Set<string>} */
+    this._securityOrigins = new Set();
     this._mainSecurityOrigin = '';
   }
 
   /**
    * @param {!SDK.Target} target
    * @return {!SDK.SecurityOriginManager}
    */
   static fromTarget(target) {
     var securityOriginManager = target.model(SDK.SecurityOriginManager);
     if (!securityOriginManager)
       securityOriginManager = new SDK.SecurityOriginManager(target);
     return securityOriginManager;
   }
 
   /**
-   * @param {string} securityOrigin
+   * @param {!Set<string>} securityOrigins
    */
-  addSecurityOrigin(securityOrigin) {
-    var currentCount = this._securityOriginCounter.get(securityOrigin);
-    if (!currentCount) {
-      this._securityOriginCounter.set(securityOrigin, 1);
-      this.dispatchEventToListeners(SDK.SecurityOriginManager.Events.SecurityOriginAdded, securityOrigin);
-      return;
+  updateSecurityOrigins(securityOrigins) {
+    var oldOrigins = this._securityOrigins;
+    this._securityOrigins = new Set(securityOrigins);

[pfeldman, 2017/01/09 20:34:47]

No need to wrap in a new set, it is already a set.

[eostroukhov, 2017/01/09 22:09:49]

I don't trust API users... But I changed the code.

+
+    for (var origin of oldOrigins) {
+      if (!this._securityOrigins.has(origin))
+        this.dispatchEventToListeners(SDK.SecurityOriginManager.Events.SecurityOriginRemoved, origin);
     }
-    this._securityOriginCounter.set(securityOrigin, currentCount + 1);
-  }
 
-  /**
-   * @param {string} securityOrigin
-   */
-  removeSecurityOrigin(securityOrigin) {
-    var currentCount = this._securityOriginCounter.get(securityOrigin);
-    if (currentCount === 1) {
-      this._securityOriginCounter.delete(securityOrigin);
-      this.dispatchEventToListeners(SDK.SecurityOriginManager.Events.SecurityOriginRemoved, securityOrigin);
-      return;
+    for (var origin of this._securityOrigins) {
+      if (!oldOrigins.has(origin))
+        this.dispatchEventToListeners(SDK.SecurityOriginManager.Events.SecurityOriginAdded, origin);
     }
-    this._securityOriginCounter.set(securityOrigin, currentCount - 1);
   }
 
   /**
    * @return {!Array<string>}
    */
   securityOrigins() {
-    return this._securityOriginCounter.keysArray();
+    return this._securityOrigins.valuesArray();
   }
 
   /**
    * @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')
 };