I can't be the only person who keeps forgetting the prefs to turn on strict JS and the dump() command. We should add UI for these to the Debug preferences panel. Patch coming up - looking for r= and a rubberstamp. Gerv

You probably need someone's approval (possibly Ben's) before checking in... but the code looks good to me - assuming the additional changes you've made are all whitespace changes (can we get a -uw please?). r=hwaara

r=bzbarsky

Gervase Markham wrote: > - <checkbox id="showAboutAsStupidModalWindow" label="&debugShowAboutAsStupidModalWindow.label;" > + <checkbox id="showAboutAsStupidModalWindow" label="&debugShowAboutAsModal.label;" > pref="true" preftype="bool" prefstring="browser.show_about_as_stupid_modal_window" > prefattribute="checked"/> Why did you rename the entity to remove "Stupid" and "Window"?

Why not? :-) Because it was stupidly long and made it harder to format the files. Ben? Blake? Are you cool with this? Gerv

Thank god. Yes, I'm very cool with this, I keep forgetting this pref. Feel like removing the personal toolbar dnd option while you're there (it should always be on now)?

don't pieces of DnD not work on macos and unix?

r=bzbarsky for patch v.3

I'd like to get this in 0.9.3; I've emailed blake for sr=. Gerv

JavaScript is spelled "JavaScript" not "Javascript" !

I'll fix it :-) Still looking for sr=. Gerv

Can we change that "Strict Javascript checking" text to something more clear, like "Show strict Javascript warnings"? sr=blake

Checked in. Gerv

vrfy fixed: changes look good in the General Debug pref panel. linux, 2001.08.15.14-comm winnt, 2001.08.15.06-comm mac OS 9.1 [emul on X], 2001.08.15.08-comm