Closed Bug 1573779 Opened 5 years ago Closed 4 years ago

Introduce WorkerDescriptors

Tracking

(Fission Milestone:M6c, firefox83 fixed)

Status:

RESOLVED FIXED

Milestone:

83 Branch

Project Flags:

Fission Milestone

M6c

Tracking Flags:

Tracking

Status

firefox83

---

fixed

People

(Reporter: yulia, Assigned: nchevobbe)

References

(Blocks 2 open bugs)

Details

(Whiteboard: dt-fission-m2-mvp)

Attachments

(4 files, 3 obsolete files)

Bug 1573779 - Rename WorkerTarget to WorkerDescriptor 5 years ago Yulia Startsev [:yulia] \| Back in Oct. (deleted), text/x-phabricator-request		Details
Bug 1573779 - move worker target to own file and make it a proper actor 5 years ago Yulia Startsev [:yulia] \| Back in Oct. (deleted), text/x-phabricator-request		Details
Bug 1573779 - migrate target to a target actor 5 years ago Yulia Startsev [:yulia] \| Back in Oct. (deleted), text/x-phabricator-request		Details
Bug 1573779 - split responsibilities between target and descriptor 5 years ago Yulia Startsev [:yulia] \| Back in Oct. (deleted), text/x-phabricator-request		Details
Bug 1573779 - rename workerTargetFront to workerDescriptorFront where necessary 5 years ago Yulia Startsev [:yulia] \| Back in Oct. (deleted), text/x-phabricator-request		Details
Bug 1573779 - [devtools] Rename WorkerTarget to WorkerDescriptor. r=ochameau,jdescottes. 4 years ago Nicolas Chevobbe [:nchevobbe] (deleted), text/x-phabricator-request		Details
Bug 1573779 - [devtools] Rename WorkerTargetActorList to WorkerDescriptorActorList. r=jdescottes. 4 years ago Nicolas Chevobbe [:nchevobbe] (deleted), text/x-phabricator-request		Details

Yulia Startsev [:yulia] | Back in Oct.

Reporter

Description

•

5 years ago

This bug introduces a descriptor classes for workers, similar to the processDescriptor

Yulia Startsev [:yulia] | Back in Oct.

Reporter

Updated

•

5 years ago

Blocks: dt-fission-framework

QA Whiteboard: dt-fission

Priority: -- → P2

Yulia Startsev [:yulia] | Back in Oct.

Reporter

Updated

•

5 years ago

QA Whiteboard: dt-fission

Whiteboard: dt-fission

Yulia Startsev [:yulia] | Back in Oct.

Reporter

Updated

•

5 years ago

Blocks: 1471754
No longer blocks: dt-fission-framework

Yulia Startsev [:yulia] | Back in Oct.

Reporter

Updated

•

5 years ago

Blocks: dt-fission-framework
No longer blocks: 1471754

Yulia Startsev [:yulia] | Back in Oct.

Reporter

Updated

•

5 years ago

Blocks: 1471754

Yulia Startsev [:yulia] | Back in Oct.

Reporter

Updated

•

5 years ago

No longer blocks: dt-fission-framework

Marco Mucci [:MarcoM]

Updated

•

5 years ago

Whiteboard: dt-fission

Yulia Startsev [:yulia] | Back in Oct.

Reporter

Updated

•

5 years ago

Assignee: nobody → ystartsev

Priority: P2 → P1

Yulia Startsev [:yulia] | Back in Oct.

Reporter

Comment 1

•

5 years ago

Attached file Bug 1573779 - Rename WorkerTarget to WorkerDescriptor (deleted) — Details

Yulia Startsev [:yulia] | Back in Oct.

Reporter

Comment 2

•

5 years ago

Attached file Bug 1573779 - move worker target to own file and make it a proper actor (deleted) — Details

Depends on D48037

Yulia Startsev [:yulia] | Back in Oct.

Reporter

Comment 3

•

5 years ago

Attached file Bug 1573779 - migrate target to a target actor (obsolete) (deleted) — Details

Depends on D48039

Yulia Startsev [:yulia] | Back in Oct.

Reporter

Comment 4

•

5 years ago

Attached file Bug 1573779 - split responsibilities between target and descriptor (obsolete) (deleted) — Details

Depends on D48041

Yulia Startsev [:yulia] | Back in Oct.

Reporter

Comment 5

•

5 years ago

Attached file Bug 1573779 - rename workerTargetFront to workerDescriptorFront where necessary (obsolete) (deleted) — Details

Depends on D49152

Phabricator Automation

Updated

•

5 years ago

Attachment #9098466 - Attachment description: Bug 1573779 - move worker target to own file → Bug 1573779 - move worker target to own file and make it a proper actor

Phabricator Automation

Updated

•

5 years ago

Attachment #9098466 - Attachment description: Bug 1573779 - move worker target to own file and make it a proper actor → Bug 1573779 - split responsibilities between target and descriptor

Phabricator Automation

Updated

•

5 years ago

Attachment #9098466 - Attachment description: Bug 1573779 - split responsibilities between target and descriptor → Bug 1573779 - move worker target to own file and make it a proper actor

Phabricator Automation

Updated

•

5 years ago

Attachment #9098468 - Attachment is obsolete: true

Alexandre Poirot [:ochameau]

Updated

•

5 years ago

Blocks: 1574348

Nicolas Chevobbe [:nchevobbe]

Assignee

Comment 6

•

4 years ago

Attached file Bug 1573779 - [devtools] Rename WorkerTarget to WorkerDescriptor. r=ochameau,jdescottes. (deleted) — Details

Phabricator Automation

Updated

•

4 years ago

Attachment #9177308 - Attachment description: Bug 1573779 - Rename WorkerTarget to WorkerDescriptor. r=ochameau,jdescottes. → Bug 1573779 - [devtools] Rename WorkerTarget to WorkerDescriptor. r=ochameau,jdescottes.

Nicolas Chevobbe [:nchevobbe]

Assignee

Updated

•

4 years ago

Blocks: 1662734

Nicolas Chevobbe [:nchevobbe]

Assignee

Updated

•

4 years ago

Whiteboard: dt-fission-m2-mvp

Marco Mucci [:MarcoM]

Updated

•

4 years ago

Assignee: ystartsev → nchevobbe

Status: NEW → ASSIGNED

Chris Peterson [:cpeterson]

Comment 7

•

4 years ago

Tracking dt-fission-m2-mvp bugs for Fission Nightly milestone (M6c).

Fission Milestone: --- → M6c

Nicolas Chevobbe [:nchevobbe]

Assignee

Updated

•

4 years ago

Blocks: 1667404

Nicolas Chevobbe [:nchevobbe]

Assignee

Comment 8

•

4 years ago

Attached file Bug 1573779 - [devtools] Rename WorkerTargetActorList to WorkerDescriptorActorList. r=jdescottes. (deleted) — Details

This class is creating (and handling) WorkerDescriptorActor now, so we rename
it to make it explicit.

Depends on D91100

Pulsebot

Comment 9

•

4 years ago

Pushed by nchevobbe@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/0c605800aa6b [devtools] Rename WorkerTarget to WorkerDescriptor. r=ochameau,jdescottes,devtools-backward-compat-reviewers. https://hg.mozilla.org/integration/autoland/rev/c6bb84f2a032 [devtools] Rename WorkerTargetActorList to WorkerDescriptorActorList. r=jdescottes.

Dorel Luca [:dluca]

Comment 10

•

4 years ago

Backed out 3 changesets (bug 1573779, bug 1662734) for Geckoview failures. CLOSED TREE

Log:
https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=317017244&repo=autoland&lineNumber=1637

Push with failures:
https://treeherder.mozilla.org/#/jobs?repo=autoland&group_state=expanded&revision=0adff4f08b28cc76a7faf844be9f4ba84168f6b6

Backout:
https://hg.mozilla.org/integration/autoland/rev/66d1f77a6b87122c6a0dee35c02b6c6aeb3d98d5

Flags: needinfo?(nchevobbe)

Nicolas Chevobbe [:nchevobbe]

Assignee

Comment 11

•

4 years ago

forgot to rename a path properly. Fixed it and pushed again to check geckoview tests: https://treeherder.mozilla.org/#/jobs?repo=try&revision=2f39007b51118a486852eb264de24cfc55864823

Flags: needinfo?(nchevobbe)

Pulsebot

Comment 12

•

4 years ago

Pushed by nchevobbe@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/c1d81faf8aaf [devtools] Rename WorkerTarget to WorkerDescriptor. r=ochameau,jdescottes,devtools-backward-compat-reviewers. https://hg.mozilla.org/integration/autoland/rev/6c3e94a0bc90 [devtools] Rename WorkerTargetActorList to WorkerDescriptorActorList. r=jdescottes.

Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)

Comment 13

•

4 years ago

bugherder

https://hg.mozilla.org/mozilla-central/rev/c1d81faf8aaf
https://hg.mozilla.org/mozilla-central/rev/6c3e94a0bc90

Status: ASSIGNED → RESOLVED

Closed: 4 years ago

status-firefox83: --- → fixed

Resolution: --- → FIXED

Target Milestone: --- → 83 Branch

Phabricator Automation

Updated

•

3 years ago

Attachment #9100861 - Attachment is obsolete: true

Phabricator Automation

Updated

•

3 years ago

Attachment #9100860 - Attachment is obsolete: true

You need to log in before you can comment on or make changes to this bug.