On Fri, Feb 2, 2018, at 10:40 AM, Nicholas Nethercote wrote: > [*] One tricky question is what to do with syntax errors. The current > behaviour is here: > https://searchfox.org/mozilla-central/source/modules/libpref/Preferences.cpp#1000-1011. > It writes the error to the browser console, but if that fails, prints it to > stderr instead. And then it also does an NS_WARNING. This means that > errors are easy to overlook, especially in user.js. Do you have suggestions > on a better approach? Do you think the parser should do error recovery? > Bug 107624 has some context for this.
One approach would probably be showing a warning UI to tell people that we fail to parse user.js and ask them to fix it. Not sure how complicated that approach would be. What do we show when we disable legacy addons? We can probably borrow whatever we did there. - Xidorn _______________________________________________ dev-platform mailing list dev-platform@lists.mozilla.org https://lists.mozilla.org/listinfo/dev-platform
