1499333 - Rename all "worker client" to "worker front"

Status: RESOLVED FIXED

(DevTools :: Framework, enhancement)

Description

[Alexandre Poirot [:ochameau]]

Bug 1494632 is going to rename WorkerClient class to WorkerTargetFront, but won't rename all the variables where we pass around a worker client.

I'll handle the renaming of these variables in a distinct bug as it is a very conflicting patch that is likely going to be hard to merge.

Comment 1

[Alexandre Poirot [:ochameau]]

This rename should be about doing:
 $ find devtools/ -type f -exec sed -i 's/WorkerClient/workerFront/g' {} \;
 $ find devtools/ -type f -exec sed -i 's/workerClient/workerFront/g' {} \;
And manually rename devtools/shared/webconsole/test/common.js which contains "worker client".

Or should it rather be named "workerTargetFront" rather than "workerFront"?
Note that I'm expecting to eventually merge this WorkerTargetFront class with WorkerTarget/TabTarget.
So it some other places, where we currently use WorkerTarget, the variable is called just "target". And I don't expect any rename in those places.

Also, I'm expecting to simplify/remove most of the code currently managing a "worker client".
I would like the listWorkers to return the worker target front directly (currently it returns an actor ID, which we only translate to a front in attachWorker). And once we merge the Fronts with WorkerTarget/TabTarget we will return the "generic" target object. (which will be the worker target front...) But I'm not sure this explanation makes it any easier to understand?!

Julian, any opinion before I submit a patch for review?

Comment 2

[Julian Descottes [:jdescottes]]

I think I would prefer workerTargetFront (or drop the front? just workerTarget?). Since we have a worker actor, it might be confusing otherwise.

Comment 3

[Alexandre Poirot [:ochameau]]

Attachment: Bug 1499333 - Rename all "worker client" to "worker target front".

MozReview-Commit-ID: 9ev8J4gUYuf

Comment 4

[Alexandre Poirot [:ochameau]]

Attachment: Bug 1499333 - Rename all "worker client" to "worker target front".

MozReview-Commit-ID: 30eOjbkYyvg

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/d893e01e4d37
Rename all "worker client" to "worker target front". r=jdescottes

Comment 6

[Cosmin Sabou [:CosminS]]

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