[Devtools] NetworkLogView now uses NetworkLog as source of truth.

third_party/WebKit/Source/devtools/front_end/network_log/NetworkLog.js

Index: third_party/WebKit/Source/devtools/front_end/network_log/NetworkLog.js
diff --git a/third_party/WebKit/Source/devtools/front_end/network_log/NetworkLog.js b/third_party/WebKit/Source/devtools/front_end/network_log/NetworkLog.js
index 638f68d2d3cdf26fb202cb5a95c8589d33023537..7744863143eda60f3ba41ef2b518daa1016e6265 100644
--- a/third_party/WebKit/Source/devtools/front_end/network_log/NetworkLog.js
+++ b/third_party/WebKit/Source/devtools/front_end/network_log/NetworkLog.js
@@ -31,14 +31,18 @@
 /**
  * @implements {SDK.SDKModelObserver<!SDK.NetworkManager>}
  */
-NetworkLog.NetworkLog = class {
+NetworkLog.NetworkLog = class extends Common.Object {
   constructor() {
+    super();
     /** @type {!Array<!SDK.NetworkRequest>} */
     this._requests = [];
+    /** @type {!Set<!SDK.NetworkRequest>} */
+    this._requestsSet = new Set();
     /** @type {!Map<!SDK.NetworkManager, !Map<string, !SDK.NetworkRequest>>} */
     this._requestsByManagerAndId = new Map();
     /** @type {!Map<!SDK.NetworkManager, !NetworkLog.PageLoad>} */
     this._currentPageLoad = new Map();
+    this._isRecording = true;
     SDK.targetManager.observeModels(SDK.NetworkManager, this);
   }
 
@@ -50,11 +54,17 @@ NetworkLog.NetworkLog = class {
     var eventListeners = [];
     eventListeners.push(
         networkManager.addEventListener(SDK.NetworkManager.Events.RequestStarted, this._onRequestStarted, this));
+    eventListeners.push(
+        networkManager.addEventListener(SDK.NetworkManager.Events.RequestUpdated, this._onRequestUpdated, this));
     eventListeners.push(
         networkManager.addEventListener(SDK.NetworkManager.Events.RequestRedirected, this._onRequestRedirect, this));
+    eventListeners.push(
+        networkManager.addEventListener(SDK.NetworkManager.Events.RequestFinished, this._onRequestUpdated, this));
 
     var resourceTreeModel = networkManager.target().model(SDK.ResourceTreeModel);
     if (resourceTreeModel) {
+      eventListeners.push(
+          resourceTreeModel.addEventListener(SDK.ResourceTreeModel.Events.WillReloadPage, this._willReloadPage, this));
       eventListeners.push(resourceTreeModel.addEventListener(
           SDK.ResourceTreeModel.Events.MainFrameNavigated, this._onMainFrameNavigated, this));
       eventListeners.push(resourceTreeModel.addEventListener(SDK.ResourceTreeModel.Events.Load, this._onLoad, this));
@@ -75,12 +85,23 @@ NetworkLog.NetworkLog = class {
     Common.EventTarget.removeEventListeners(networkManager[NetworkLog.NetworkLog._events]);
   }
 
+  /**
+   * @param {boolean} enabled
+   */
+  setRecording(enabled) {

[dgozman, 2017/06/02 21:13:38]

setIsRecording

[allada, 2017/06/03 00:27:41]

Done.

+    this._isRecording = enabled;
+  }
+
   /**
    * @param {string} url
    * @return {?SDK.NetworkRequest}
    */
   requestForURL(url) {
-    return this._requests.find(request => request.url() === url) || null;
+    for (var request of this._requests) {
+      if (request.url() === url)
+        return request;
+    }
+    return null;
   }
 
   /**
@@ -236,18 +257,37 @@ NetworkLog.NetworkLog = class {
     return request[NetworkLog.NetworkLog._pageLoadForRequestSymbol];
   }
 
+  _willReloadPage() {
+    if (!Common.moduleSetting('network_log.preserve-log').get())
+      this.reset();
+  }
+
   /**
    * @param {!Common.Event} event
    */
   _onMainFrameNavigated(event) {
+    if (!this._isRecording)
+      return;
     var mainFrame = /** @type {!SDK.ResourceTreeFrame} */ (event.data);
     var networkManager = mainFrame.resourceTreeModel().target().model(SDK.NetworkManager);
     if (!networkManager)
       return;
 
-    this._currentPageLoad.delete(networkManager);
     var oldRequests = this.requestsForManager(networkManager);
-    this._requests = this._requests.filter(request => request.networkManager() !== networkManager);
+    if (!Common.moduleSetting('network_log.preserve-log').get()) {
+      var filteredRequests = [];
+      var removedRequests = [];
+      for (var request of this._requests) {
+        if (request.networkManager() !== networkManager)
+          filteredRequests.push(request);
+        else
+          removedRequests.push(request);
+      }
+      this._requests = filteredRequests;

[dgozman, 2017/06/02 21:13:38]

this._requests = this._requests.filter(request => request.networkManager() !==
networkManager);

[allada, 2017/06/03 00:27:41]

Done.

+      this._requestsSet = new Set(filteredRequests);
+      for (var request of removedRequests)
+        this.dispatchEventToListeners(NetworkLog.NetworkLog.Events.RequestRemoved, request);
+    }
     var idMap = new Map();
     this._requestsByManagerAndId.set(networkManager, idMap);
 
@@ -259,6 +299,8 @@ NetworkLog.NetworkLog = class {
         if (!currentPageLoad)
           currentPageLoad = new NetworkLog.PageLoad(request);
         this._requests.push(request);
+        this._requestsSet.add(request);
+        this.dispatchEventToListeners(NetworkLog.NetworkLog.Events.RequestAdded, request);

[dgozman, 2017/06/02 21:13:38]

Let's not dispatch RequestRemoved and then RequestAdded for these requests, and
instead only dispatch removed for actually removed ones.

[allada, 2017/06/03 00:27:41]

Done.

         idMap.set(request.requestId(), request);
         request[NetworkLog.NetworkLog._pageLoadForRequestSymbol] = currentPageLoad;
       }
@@ -271,16 +313,34 @@ NetworkLog.NetworkLog = class {
    * @param {!Common.Event} event
    */
   _onRequestStarted(event) {
+    if (!this._isRecording)
+      return;
     var request = /** @type {!SDK.NetworkRequest} */ (event.data);
     this._requests.push(request);
+    this._requestsSet.add(request);
     this._requestsByManagerAndId.get(request.networkManager()).set(request.requestId(), request);
     request[NetworkLog.NetworkLog._pageLoadForRequestSymbol] = this._currentPageLoad.get(request.networkManager());
+    this.dispatchEventToListeners(NetworkLog.NetworkLog.Events.RequestAdded, request);
+  }
+
+  /**
+   * @param {!Common.Event} event
+   */
+  _onRequestUpdated(event) {
+    if (!this._isRecording)

[dgozman, 2017/06/02 21:13:38]

Instead of checking all the time, should we start/stop listening to events
instead?

[allada, 2017/06/03 00:27:41]

Done.

+      return;
+    var request = /** @type {!SDK.NetworkRequest} */ (event.data);
+    if (!this._requestsSet.has(request))
+      return;
+    this.dispatchEventToListeners(NetworkLog.NetworkLog.Events.RequestUpdated, request);
   }
 
   /**
    * @param {!Common.Event} event
    */
   _onRequestRedirect(event) {
+    if (!this._isRecording)
+      return;
     var request = /** @type {!SDK.NetworkRequest} */ (event.data);
     delete request[NetworkLog.NetworkLog._initiatorDataSymbol];
   }
@@ -290,6 +350,8 @@ NetworkLog.NetworkLog = class {
    * @param {!Common.Event} event
    */
   _onDOMContentLoaded(resourceTreeModel, event) {
+    if (!this._isRecording)
+      return;
     var networkManager = resourceTreeModel.target().model(SDK.NetworkManager);
     var pageLoad = networkManager ? this._currentPageLoad.get(networkManager) : null;
     if (pageLoad)
@@ -300,6 +362,8 @@ NetworkLog.NetworkLog = class {
    * @param {!Common.Event} event
    */
   _onLoad(event) {
+    if (!this._isRecording)
+      return;
     var networkManager = event.data.resourceTreeModel.target().model(SDK.NetworkManager);
     var pageLoad = networkManager ? this._currentPageLoad.get(networkManager) : null;
     if (pageLoad)
@@ -315,6 +379,19 @@ NetworkLog.NetworkLog = class {
     var map = this._requestsByManagerAndId.get(networkManager);
     return map ? (map.get(requestId) || null) : null;
   }
+
+  reset() {
+    this._requests = [];
+    this._requestsSet.clear();
+    this._requestsByManagerAndId.forEach(map => map.clear());
+    var networkManagers = new Set(SDK.targetManager.models(SDK.NetworkManager));
+    for (var networkManager of this._currentPageLoad.keys()) {
+      if (!networkManagers.has(networkManager))
+        this._currentPageLoad.delete(networkManager);
+    }
+
+    this.dispatchEventToListeners(NetworkLog.NetworkLog.Events.Reset);
+  }
 };
 
 NetworkLog.PageLoad = class {
@@ -329,6 +406,7 @@ NetworkLog.PageLoad = class {
     this.loadTime;
     /** @type {number} */
     this.contentLoadTime;
+    this.mainRequest = mainRequest;
   }
 };
 
@@ -337,6 +415,13 @@ NetworkLog.PageLoad._lastIdentifier = 0;
 /** @typedef {!{initiators: !Set<!SDK.NetworkRequest>, initiated: !Set<!SDK.NetworkRequest>}} */
 NetworkLog.NetworkLog.InitiatorGraph;
 
+NetworkLog.NetworkLog.Events = {
+  Reset: Symbol('Reset'),
+  RequestAdded: Symbol('RequestAdded'),
+  RequestRemoved: Symbol('RequestRemoved'),
+  RequestUpdated: Symbol('RequestUpdated')
+};
+
 /** @typedef {!{type: !SDK.NetworkRequest.InitiatorType, url: string, lineNumber: number, columnNumber: number, scriptId: ?string}} */
 NetworkLog.NetworkLog._InitiatorInfo;