Open Bug 1538325 Opened 5 years ago Updated 2 years ago

Document anonymous and non-anonymous walkers in inspector actors

Categories

(DevTools :: Inspector, enhancement, P3)

Product:
DevTools
Component:
Inspector
Type:
enhancement

Tracking

(Not tracked)

Status:
ASSIGNED

People

(Reporter: jdescottes, Assigned: jdescottes)

References

(Blocks 1 open bug)

Details

From the review of Bug 1537877

In D24414#717090, @pbro wrote:

I think this looks good. It's getting increasingly hard to understand what
anonymous means in the walker. We have methods to walk the anonymous tree,
and others to walk normally, but then these also depend on a pref. And then
there's anonymous nodes and shadow dom nodes.
I think it might be time for a big documentation comment at the top that
explains it all. Not in this patch, but I think we do need some
documentation.

See Also: → 1537877
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.