Bug 144124 – Web Inspector: run a customizable bootstrap function after the UI has fully loaded

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

[Brian Burg]

The point of a "bootstrap script" is to reduce amount of time clicking on the UI to get the inspector to a useful state for debugging.
Instead, you write commands in a function that runs after the inspector has loaded. These commands automate reproduction steps, and can be shared as part of a bug report (if obvious).

Some things you might want to do include opening different sidebars, browsing between web pages, evaluating text in the console.

[Radar WebKit Bug Importer]

<rdar://problem/20674800>

[Brian Burg]

Created attachment 251503 [details]
Patch

[Timothy Hatcher]

Comment on attachment 251503 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=251503&action=review

> Source/WebInspectorUI/UserInterface/Main.html:539
> +    <script src="Base/Bootstrap.js"></script>

Should this file be skipped when combining for Release/Production?

[Brian Burg]

(In reply to comment #3)
> Comment on attachment 251503 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=251503&action=review
> 
> > Source/WebInspectorUI/UserInterface/Main.html:539
> > +    <script src="Base/Bootstrap.js"></script>
> 
> Should this file be skipped when combining for Release/Production?

I suppose that's possible, though I don't think we do that anywhere else? It would require blacklisting in inline-and-minify-stylesheets-and-scripts.py, and maybe other parts to avoid copying the resource.

I wasn't planning to have anything in the function body on trunk; developers can revert or exclude their little scripts from patches. Reusable snippets can live in comments in Bootstrap.js (to be stripped out) or on the wiki.

[Joseph Pecoraro]

Comment on attachment 251503 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=251503&action=review

Seems fine to me. Do you have an example of a bootstrap you might use?

> Source/WebInspectorUI/UserInterface/Base/Main.js:361
> +    if (this.runBootstrapOperations && typeof this.runBootstrapOperations === "function")
> +        this.runBootstrapOperations();

No need to if-check? It is guaranteed right now.

>>> Source/WebInspectorUI/UserInterface/Main.html:539
>>> +    <script src="Base/Bootstrap.js"></script>
>> 
>> Should this file be skipped when combining for Release/Production?
> 
> I suppose that's possible, though I don't think we do that anywhere else? It would require blacklisting in inline-and-minify-stylesheets-and-scripts.py, and maybe other parts to avoid copying the resource.
> 
> I wasn't planning to have anything in the function body on trunk; developers can revert or exclude their little scripts from patches. Reusable snippets can live in comments in Bootstrap.js (to be stripped out) or on the wiki.

This should minify down to:
WebInspector.runBootstrapOperations=function(){};

So it may be fine to leave in.

[Brian Burg]

Committed r183238: <http://trac.webkit.org/changeset/183238>

[Brian Burg]

(In reply to comment #5)
> Comment on attachment 251503 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=251503&action=review
> 
> Seems fine to me. Do you have an example of a bootstrap you might use?

Something I did over and over when doing element tracking feature is to load a page, select the same element in the DOM, and invoke "Start Tracking". This could have been automated by doing querySelector to get the node, then passing it to DOMTracingManager.startTracking(). Further work on specific screens could have been automated by programmatically changing the content browser's represented object.

I'll update the wiki page (https://trac.webkit.org/wiki/WebInspectorDebugging) as I make use of this feature to help share some useful snippets.

[Timothy Hatcher]

Comment on attachment 251503 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=251503&action=review

>>>> Source/WebInspectorUI/UserInterface/Main.html:539
>>>> +    <script src="Base/Bootstrap.js"></script>
>>> 
>>> Should this file be skipped when combining for Release/Production?
>> 
>> I suppose that's possible, though I don't think we do that anywhere else? It would require blacklisting in inline-and-minify-stylesheets-and-scripts.py, and maybe other parts to avoid copying the resource.
>> 
>> I wasn't planning to have anything in the function body on trunk; developers can revert or exclude their little scripts from patches. Reusable snippets can live in comments in Bootstrap.js (to be stripped out) or on the wiki.
> 
> This should minify down to:
> WebInspector.runBootstrapOperations=function(){};
> 
> So it may be fine to leave in.

We skip combining (or combine separately) files in External/*.