Use same file descriptor for notification and reading
ClosedPublic

Authored by fvogt on Jul 15 2017, 2:32 PM.

Details

Summary

Otherwise we wait for data on a different descriptor than we read from,
which results in missing notifications if data lands in a buffer.

Test Plan

No modules were loaded after reloading the extension from the browser,
this is due to the settings changed message not getting received by the host
as it was in quick succession to the setEnvironment message. With this patch
it now works reliably.

Diff Detail

Repository
R856 Plasma Browser Integration
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
fvogt created this revision.Jul 15 2017, 2:32 PM
Restricted Application added a project: Plasma. · View Herald TranscriptJul 15 2017, 2:32 PM
davidedmundson accepted this revision.Jul 15 2017, 2:37 PM
This revision is now accepted and ready to land.Jul 15 2017, 2:37 PM
This revision was automatically updated to reflect the committed changes.