HAR Extension for Resource Priority • We include _fetchType we can include _priority
Created attachment 364829 [details] [PATCH] Proposed Fix
Created attachment 364862 [details] [PATCH] Proposed Fix
Comment on attachment 364862 [details] [PATCH] Proposed Fix View in context: https://bugs.webkit.org/attachment.cgi?id=364862&action=review r=me > Source/WebInspectorUI/UserInterface/Controllers/HARBuilder.js:332 > + case WI.Resource.NetworkPriority.Low: > + return "low"; > + case WI.Resource.NetworkPriority.Medium: > + return "medium"; > + case WI.Resource.NetworkPriority.High: > + return "high"; Could we use the `NetworkAgent.MetricsPriority` values instead, or are you worried about backward/forward compatibility? > Source/WebInspectorUI/UserInterface/Controllers/HARBuilder.js:335 > + console.assert(false); `console.assert()` works instead of this :P > Source/WebInspectorUI/UserInterface/Controllers/HARBuilder.js:380 > - console.warn("Unknown HAR Protocol _fetchType", fetchType); > + console.warn("Unknown HAR _fetchType value", fetchType); Oops :P > Source/WebInspectorUI/UserInterface/Controllers/HARBuilder.js:392 > + case "low": > + return WI.Resource.NetworkPriority.Low; > + case "medium": > + return WI.Resource.NetworkPriority.Medium; > + case "high": > + return WI.Resource.NetworkPriority.High; Ditto (>327).
Comment on attachment 364862 [details] [PATCH] Proposed Fix View in context: https://bugs.webkit.org/attachment.cgi?id=364862&action=review >> Source/WebInspectorUI/UserInterface/Controllers/HARBuilder.js:332 >> + return "high"; > > Could we use the `NetworkAgent.MetricsPriority` values instead, or are you worried about backward/forward compatibility? Yeah I don't want changing the protocol to break HAR import/export expectations. >> Source/WebInspectorUI/UserInterface/Controllers/HARBuilder.js:392 >> + return WI.Resource.NetworkPriority.High; > > Ditto (>327). These we are converting back from the HAR to WI.Resource so we'd want what we serialized into the HAR.
https://trac.webkit.org/r243031
bump to try and get radar importer to import
<rdar://problem/48986580>