367601 - add "all" logging support to LOG() in updates.js

Status: RESOLVED FIXED

(Toolkit :: Application Update, enhancement)

Description

[(not reading, please use [e-mail] instead)]

a while back nick suggested we add "app.update.log.all" support, see
 https://bugzilla.mozilla.org/show_bug.cgi?id=349271#c3

it's been useful, and it is on the trunk:  

http://lxr.mozilla.org/seamonkey/source/toolkit/mozapps/update/src/nsUpdateService.js.in#1169

132 function LOG(module, string) {
133   if (module in gLogEnabled || "all" in gLogEnabled) {
134     dump("*** " + module + ": " + string + "\n");
135     gConsole.logStringMessage(string);
136   }

this bug is about added it to updates.js as well, which also uses the "app.update.log." pref branch.

 75 function LOG(module, string) {
 76   if (module in gLogEnabled) {
 77     dump("*** " + module + ":" + string + "\n");
 78     gConsole.logStringMessage(string);
 79   }
 80 }

Comment 1

[Robert Strong (they/them - no direct email)]

Fixed by the checkin of bug 324121