Reload actions from second level inspector should properly reload the inspected inspector Can't use PageAgent.reload, need to InspectorFrontendHost.reopen.
Created attachment 363583 [details] [PATCH] Proposed Fix
Comment on attachment 363583 [details] [PATCH] Proposed Fix r=me
Comment on attachment 363583 [details] [PATCH] Proposed Fix Clearing flags on attachment: 363583 Committed r242410: <https://trac.webkit.org/changeset/242410>
All reviewed patches have been landed. Closing bug.
<rdar://problem/48588528>
Now there's no way to reload Web Inspector so it picks up changes from the filesystem. With this patch, pressing Command-R in 2nd level inspector doesn't pick up changes from the FS. "Reload Inspector" context menu stopped doing it a few months ago. 😭
(In reply to Nikita Vasilyev from comment #6) > Now there's no way to reload Web Inspector so it picks up changes from the > filesystem. > > With this patch, pressing Command-R in 2nd level inspector doesn't pick up > changes from the FS. > > "Reload Inspector" context menu stopped doing it a few months ago. > > 😭 Reloading from the file system never actually worked properly =(. It may have been a quick hack usable for development but it never re-initialized things properly. If we want to add something like that back we should make sure it works properly.
(In reply to Joseph Pecoraro from comment #7) > (In reply to Nikita Vasilyev from comment #6) > > Now there's no way to reload Web Inspector so it picks up changes from the > > filesystem. > > > > With this patch, pressing Command-R in 2nd level inspector doesn't pick up > > changes from the FS. > > > > "Reload Inspector" context menu stopped doing it a few months ago. > > > > 😭 > > Reloading from the file system never actually worked properly =(. It may > have been a quick hack usable for development but it never re-initialized > things properly. If we want to add something like that back we should make > sure it works properly. What do you mean by properly? I used it for development hundreds of times a day and now I can't. How often do you need to reload 1st level Inspector from the 2nd level Inspector but not load the latest files from the filesystem? This actually reads changes from the FS: WI._reloadInspectedInspector = function() { PageAgent.reload.invoke({ignoreCache: true}, function() { const options = {}; WI.runtimeManager.evaluateInInspectedWindow(`InspectorFrontendHost.reopen()`, options, function(){}); }); }; It reloads Inspector twice, essentially, but it actually works.