1335103 - Remove spurious usage of Actor in lower.py

Status: RESOLVED FIXED

(Core :: IPC, defect)

Description

[Andrew McCreight [:mccr8]]

The Actor class in type.py is used in the code related to checking the relationship between processes. However, it is also used for some reason to convert a protocol+side to a string in lower.py. This patch just does the conversion directly, to ease the way to deleting the Actor class in bug 1316755.

Comment 1

[Andrew McCreight [:mccr8]]

Attachment: Bug 1335103 - Remove spurious usage of Actor in lower.py.

Actor is used in the process checking code, which will get deleted in
a separate bug. Instead, just convert the actor name + side to a
string manually.

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

Comment 2

[Bill McCloskey [inactive unless it's an emergency] (:billm)]

Comment on attachment 8831759 [details]
Bug 1335103 - Remove spurious usage of Actor in lower.py.

https://reviewboard.mozilla.org/r/108302/#review109368

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a15a2c5d3cd5
Remove spurious usage of Actor in lower.py. r=billm

Comment 4

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

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