Closed
Bug 1247091
Opened 9 years ago
Closed 9 years ago
Document WorkerActor in actor-heirarchy.md
Categories
(DevTools :: Framework, defect)
DevTools
Framework
Tracking
(firefox47 fixed)
RESOLVED
FIXED
Firefox 47
| Tracking | Status | |
|---|---|---|
| firefox47 | --- | fixed |
People
(Reporter: jryans, Assigned: jryans)
Details
Attachments
(1 file)
WorkerActor is another TabActor-ish actor that should be documented in the list.
|
Assignee | |
Comment 1•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/34273/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/34273/
Attachment #8717698 -
Flags: review?(poirot.alex)
|
||
Updated•9 years ago
|
Attachment #8717698 -
Flags: review?(poirot.alex) → review+
|
|
||
Comment 2•9 years ago
|
||
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// ?
|
|
Assignee | |
Comment 3•9 years ago
|
||
https://reviewboard.mozilla.org/r/34273/#review31039
Thanks, I've made these fixes.
|
||
Comment 5•9 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 47
|
||
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•