Closed Bug 1233663 Opened 8 years ago Closed 8 years ago

Add a high-level inspector-panel architecture doc to devtools/docs

Categories

(DevTools :: Inspector, defect)

defect
Not set
normal

Tracking

(firefox46 fixed)

RESOLVED FIXED
Firefox 46
Tracking Status
firefox46 --- fixed

People

(Reporter: pbro, Assigned: pbro)

Details

Attachments

(1 file)

      No description provided.
Assignee: nobody → pbrosset
Status: NEW → ASSIGNED
These are essentially the notes from my 5 minutes tech talk about the inspector-panel at one of our devtools meetings, converted to markdown, and added to /devtools/docs (as discussed with jlongster).
We can iterate/clarify/add but that's a start I guess.
Attachment #8699903 - Flags: review?(bgrinstead)
Comment on attachment 8699903 [details] [diff] [review]
Bug_1233663_-_Create_high-level_architecture_inspe.diff

Review of attachment 8699903 [details] [diff] [review]:
-----------------------------------------------------------------

::: devtools/docs/inspector-panel.md
@@ +24,5 @@
> +                                                     +- AnimationInspector
> +
> +## Server dependencies
> +- The inspector panel relies on a series of actors that live on the server.
> +- Some of the most important actors are actually instantiated by the toolbox. Because these actors are useful to other panels too (mostly to preview and/or link DOM nodes, for example the webconsole needs to output DOM nodes and highlight them in the page on mouseover and select them in the inspector on click, this is achieved using some of the same actors that the inspector panel uses).

long parenthesized block here.  What about something like:

Some of the most important actors are actually instantiated by the toolbox, because these actors are needed for other panels to preview and link to DOM nodes.  For example, the webconsole needs to output DOM nodes, highlight them in the page on mouseover, and open them in the inspector on click.  This is achieved using some of the same actors that the inspector panel uses.

@@ +28,5 @@
> +- Some of the most important actors are actually instantiated by the toolbox. Because these actors are useful to other panels too (mostly to preview and/or link DOM nodes, for example the webconsole needs to output DOM nodes and highlight them in the page on mouseover and select them in the inspector on click, this is achieved using some of the same actors that the inspector panel uses).
> +- See Toolbox.prototype.initInspector: This method instantiates the InspectorActor, WalkerActor and HighlighterActor lazily, whenever they're needed by a panel.
> +
> +## Panel loading overview
> +- As for other panels, this starts with Toolbox.prototype.loadTool(id)

'As with' or 'Like'

@@ +56,5 @@
> +
> +__InspectorActor__
> +
> +The (tab-level) actor the inspector panel connects to.
> +Doesn't do much apart from creating and returning the WalkerActor and

Looks like some manual word wrappping here, shouldn't need that as it isn't done elsewhere in the file

@@ +75,5 @@
> +- Emits events when there are DOM mutations. These events are sent to the front-end and used to, for example refresh the markup-view. This uses an instance of MutationObserver (https://developer.mozilla.org/en-US/docs/Web/API/MutationObserver) configured with, in particular, nativeAnonymousChildList set to true, so that mutation events are also sent when pseudo elements are added/removed via css.
> +
> +__NodeActor__
> +
> +- Representation of a single DOM node (tagname, namespace, attributes,

Word wrapping comment again
Attachment #8699903 - Flags: review?(bgrinstead) → review+
I have a couple of things to add (like about walker search) but can do that as a follow up once we get the structure in place
https://hg.mozilla.org/mozilla-central/rev/79e8243e94f7
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 46
[bugday-20160323]

Status: RESOLVED,FIXED -> UNVERIFIED

Comments:
STR: Not clear.
Developer specific testing

Component: 
Name			Firefox
Version			46.0b9
Build ID		20160322075646
Update Channel	beta
User Agent		Mozilla/5.0 (Windows NT 6.1; WOW64; rv:46.0) Gecko/20100101 Firefox/46.0
OS				Windows 7 SP1 x86_64

Expected Results: 
Developer specific testing

Actual Results: 
As expected
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: