Closed
Bug 1460656
Opened 6 years ago
Closed 6 years ago
Avoid sync IPC message in listener.js on child process startup
Categories
(Remote Protocol :: Marionette, enhancement)
Remote Protocol
Marionette
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 | |
Updated•6 years ago
|
| Comment hidden (mozreview-request) |
|
||
Comment 2•6 years ago
|
||
| mozreview-review | ||
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
|
||
Comment 4•6 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/52e536beb00b
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox62:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
|
||
Updated•1 year ago
|
Product: Testing → Remote Protocol
You need to log in
before you can comment on or make changes to this bug.
Description
•