Bug 195306 – Web Inspector: Reload actions from second level inspector should properly reload the inspected inspector

• Status: RESOLVED
• Resolution: FIXED
• Component: Web Inspector

[Joseph Pecoraro]

Reload actions from second level inspector should properly reload the inspected inspector

Can't use PageAgent.reload, need to InspectorFrontendHost.reopen.

[Joseph Pecoraro]

Created attachment 363583 [details]
[PATCH] Proposed Fix

[Matt Baker]

Comment on attachment 363583 [details]
[PATCH] Proposed Fix

r=me

[WebKit Commit Bot]

Comment on attachment 363583 [details]
[PATCH] Proposed Fix

Clearing flags on attachment: 363583

Committed r242410: <https://trac.webkit.org/changeset/242410>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Radar WebKit Bug Importer]

<rdar://problem/48588528>

[Nikita Vasilyev]

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.

😭

[Joseph Pecoraro]

(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.

[Nikita Vasilyev]

(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.