1247091 - Document WorkerActor in actor-heirarchy.md

Status: RESOLVED FIXED

(DevTools :: Framework, defect)

Description

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

WorkerActor is another TabActor-ish actor that should be documented in the list.

Comment 1

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

Attachment: MozReview Request: Bug 1247091 - Add workers to actor docs. r=ochameau

Review commit: https://reviewboard.mozilla.org/r/34273/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/34273/

Comment 2

[Alexandre Poirot [:ochameau]]

Comment on attachment 8717698 [details]
MozReview Request: Bug 1247091 - Add workers to actor docs. r=ochameau

https://reviewboard.mozilla.org/r/34273/#review31039

Thanks for contributing to docs!

::: devtools/server/docs/actor-hierarchy.md:15
(Diff revision 1)
> -   |    that are not specific to any document/app/addon.
> +   |    that are not specific to any document/tab/app/add-on/worker.

What about adding:
to any +particular context like+ document/tab/..
?

::: devtools/server/docs/actor-hierarchy.md:21
(Diff revision 1)
> +          |    for each are thing you can point a toolbox at.

s/are// ?

Comment 3

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

https://reviewboard.mozilla.org/r/34273/#review31039

Thanks, I've made these fixes.

Comment 4

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/b3acf53ea440

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/b3acf53ea440