1466733 - Rename WebExtension parent actor modules

Status: RESOLVED FIXED

(DevTools :: General, enhancement, P2)

Description

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

* WebExtensionParentActor and ProxyChildActor -> WebExtensionTargetActorProxy

Target actor proxy that represents a Web Extension in the parent process.  It launches a target actor in the extension process to do the real work and tunnels the data.

Comment 1

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

(In reply to J. Ryan Stinnett [:jryans] (use ni?) from comment #0)
> * WebExtensionParentActor and ProxyChildActor -> WebExtensionTargetActorProxy

Not going to try to merge these for now... I'll rename them as follows:

* WebExtensionParentActor -> WebExtensionActor
* ProxyChildActor -> WebExtensionTargetActorProxy

Comment 2

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

Attachment: Bug 1466733 - Rename WebExtension parent actors.

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

Comment 3

[Alexandre Poirot [:ochameau]]

Comment on attachment 8985522 [details]
Bug 1466733 - Rename WebExtension parent actors.

https://reviewboard.mozilla.org/r/251138/#review257596

Looks good to me, thanks for the additional doc!

Comment 4

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

https://treeherder.mozilla.org/#/jobs?repo=try&revision=b58d5603646e77cfbdd4c07672f5be6a3fe68b70

Comment 5

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

Comment on attachment 8985522 [details]
Bug 1466733 - Rename WebExtension parent actors.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/251138/diff/1-2/

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/4abef6c30239
Rename WebExtension parent actors. r=ochameau

Comment 7

[Cristina Coroiu [:ccoroiu]]

https://hg.mozilla.org/mozilla-central/rev/4abef6c30239