RESOLVED FIXED 203025
Web Inspector: Network: updates when toggling resource caching should be sent to every connected target 
https://bugs.webkit.org/show_bug.cgi?id=203025
Summary Web Inspector: Network: updates when toggling resource caching should be sent...
Devin Rousso
Reported 2019-10-15 22:20:39 PDT
(In reply to Joseph Pecoraro from comment <https://webkit.org/b/200384#36>) > > Source/WebInspectorUI/UserInterface/Base/Main.js:2759 > > + let target = WI.assumingMainTarget(); > > + target.NetworkAgent.setResourceCachingDisabled(WI.settings.resourceCachingDisabled.value); > > Whoa! NetworkAgent should work across multiple targets. So this is probably an existing bug and should just iterate all targets. (this could be pulled out into its own bug fix) > > for (let target of WI.targets) { > if (target.NetworkAgent) > target.NetworkAgent.setResourceCachingDisabled(WI.settings. resourceCachingDisabled.value); > }
Attachments
Patch (1.40 KB, patch)
2019-10-16 15:21 PDT, Devin Rousso 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Devin Rousso
Comment 1 2019-10-16 15:21:05 PDT
Created attachment 381121 [details] Patch This patch requires <https://webkit.org/b/200384>.
Joseph Pecoraro
Comment 2 2019-10-16 15:23:26 PDT
Comment on attachment 381121 [details] Patch r=me
WebKit Commit Bot
Comment 3 2019-10-17 02:05:51 PDT
Comment on attachment 381121 [details] Patch Clearing flags on attachment: 381121 Committed r251229: <https://trac.webkit.org/changeset/251229>
WebKit Commit Bot
Comment 4 2019-10-17 02:05:53 PDT
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 5 2019-10-17 02:08:57 PDT
<rdar://problem/56363111>
Note You need to log in before you can comment on or make changes to this bug.