On Thu, Mar 16, 2017 at 4:27 PM, Kris Maglione <kmagli...@mozilla.com> wrote:
> On Thu, Mar 16, 2017 at 11:07:19AM -0400, Ehsan Akhgari wrote: > >> This is why a while ago we decided to disallow the addition of new >> synchronous IPC messages by default (bug 1336919) to avoid programmers >> adding more issues of this nature to the code base. >> > > Are there any plans to do something similar for sync message manager > messages from JS? It seems like it would make sense to have a whitelist for > those too, and require review from an IPC peer when making changes to it. > Great question -- I recently found out that navigator.userAgent suffers from a sync IPC issue in JS <https://bugzilla.mozilla.org/show_bug.cgi?id=1347425> that was hidden in our existing telemetry probes. I filed bug 1348116 <https://bugzilla.mozilla.org/show_bug.cgi?id=1348116> for the whitelist you suggested. I also filed bug 1348113 <https://bugzilla.mozilla.org/show_bug.cgi?id=1348113> to get better telemetry for sync IPC caused by JS. -- Ehsan _______________________________________________ dev-platform mailing list dev-platform@lists.mozilla.org https://lists.mozilla.org/listinfo/dev-platform
