There's no reason to have them separate, especially since NetworkManager controls most of SourceMapManager's functionality.
Created attachment 351451 [details] Patch
Comment on attachment 351451 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=351451&action=review r=me > Source/WebInspectorUI/UserInterface/Controllers/NetworkManager.js:41 > + this._sourceMapURLMap = {}; > + this._downloadingSourceMaps = {}; We could modernize this to be a Map and Set. That can be another time. > Source/WebInspectorUI/UserInterface/Controllers/NetworkManager.js:389 > - WI.sourceMapManager.downloadSourceMap(cachedResourcePayload.sourceMapURL, resource.url, resource); > + WI.networkManager.downloadSourceMap(cachedResourcePayload.sourceMapURL, resource.url, resource); Shouldn't these just be `this.` instead of `WI.networkManager.`?
Created attachment 351461 [details] Patch
Comment on attachment 351461 [details] Patch Clearing flags on attachment: 351461 Committed r236783: <https://trac.webkit.org/changeset/236783>
All reviewed patches have been landed. Closing bug.
<rdar://problem/44961679>