Title: [294716] trunk/Source/WebInspectorUI/UserInterface/Views/NetworkTableContentView.js
Revision
294716
Author
[email protected]
Date
2022-05-23 19:40:27 -0700 (Mon, 23 May 2022)

Log Message

Web Inspector: Network Table: activity is not marked as a previous session when the Network Tab is not selected
https://bugs.webkit.org/show_bug.cgi?id=240846

Reviewed by Patrick Angle.

* Source/WebInspectorUI/UserInterface/Views/NetworkTableContentView.js:
(WI.NetworkTableContentView.prototype._mainResourceDidChange):
(WI.NetworkTableContentView.prototype._entryForResource):
Also mark `WI.Resource` that haven't yet been inserted into the `WI.Table` when navigating the main
resource. Use this when generating an entry for that `WI.Resource` for the first time, instead of
always assuming that the `WI.Resource` is part of the current session.

Canonical link: https://commits.webkit.org/250902@main

Modified Paths

Diff

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/NetworkTableContentView.js (294715 => 294716)


--- trunk/Source/WebInspectorUI/UserInterface/Views/NetworkTableContentView.js	2022-05-24 02:39:23 UTC (rev 294715)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/NetworkTableContentView.js	2022-05-24 02:40:27 UTC (rev 294716)
@@ -1653,6 +1653,9 @@
                 } else {
                     for (let entry of collection.entries)
                         entry.currentSession = false;
+
+                    for (let resource of collection.pendingInsertions)
+                        resource[WI.NetworkTableContentView._currentSessionSymbol] = false;
                 }
             }
 
@@ -1904,7 +1907,7 @@
             remoteAddress: resource.displayRemoteAddress,
             connectionIdentifier: resource.connectionIdentifier,
             startTime: resource.firstTimestamp,
-            currentSession: true,
+            currentSession: resource[WI.NetworkTableContentView._currentSessionSymbol] ?? true,
         };
     }
 
@@ -2533,3 +2536,5 @@
         this._transitioningPageTarget = true;
     }
 };
+
+WI.NetworkTableContentView._currentSessionSymbol = Symbol("current-session");
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to