[UseCache] Filter to context bounds

Bug: None
Change-Id: Id481a6bade76630433c856ec652ffc8c57608ecc
Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6379246
Auto-Submit: Adam Raine <[email protected]>
Reviewed-by: Connor Clark <[email protected]>
Commit-Queue: Connor Clark <[email protected]>

Author: Adam Raine

front_end/models/trace/insights/UseCache.ts

@@ -190,11 +190,13 @@ export interface CacheableRequest {
 
 export function generateInsight(
     parsedTrace: Handlers.Types.ParsedTrace, context: InsightSetContext): UseCacheInsightModel {
+  const isWithinContext = (event: Types.Events.Event): boolean => Helpers.Timing.eventIsInBounds(event, context.bounds);
+  const contextRequests = parsedTrace.NetworkRequests.byTime.filter(isWithinContext);
+
   const results: CacheableRequest[] = [];
-  const allRequests = parsedTrace.NetworkRequests.byTime;
   let totalWastedBytes = 0;
   const wastedBytesByRequestId = new Map<string, number>();
-  for (const req of allRequests) {
+  for (const req of contextRequests) {
     if (!isCacheable(req)) {
       continue;
     }