Closed Bug 1460656 Opened 3 years ago Closed 3 years ago

Avoid sync IPC message in listener.js on child process startup

Categories

(Testing :: Marionette, enhancement)

enhancement
Not set
normal

Tracking

(firefox62 fixed)

RESOLVED FIXED
mozilla62
Tracking Status
firefox62 --- fixed

People

(Reporter: ato, Assigned: ato)

References

Details

Attachments

(1 file)

As a workaround for https://bugzilla.mozilla.org/show_bug.cgi?id=1411513,
testing/marionette/listener.js makes a synchronous call (sendSyncMessage)
to the main process asking for the Marionette logger repository’s
level.

We can avoid this blocking operation by using initialProcessData,
which is new since the workaround was implemented.
Assignee: nobody → ato
Status: NEW → ASSIGNED
Comment on attachment 8974753 [details]
Bug 1460656 - Fix sync IPC message Marionette:GetLogLevel by using initialProcessData.

https://reviewboard.mozilla.org/r/243144/#review249234
Attachment #8974753 - Flags: review?(mjzffr) → review+
Pushed by atolfsen@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/52e536beb00b
Fix sync IPC message Marionette:GetLogLevel by using initialProcessData. r=maja_zf
https://hg.mozilla.org/mozilla-central/rev/52e536beb00b
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
You need to log in before you can comment on or make changes to this bug.