We're now starting to treat compiler errors as warnings in necko

As of bug 605180 landing we now have FAIL_ON_WARNINGS turned on in the 
Makefiles for some necko directories (wyciwyg, websockets, 
file|res|data|device|ftp|viewsource protocols).    With Ms2ger's help we 
plan to land this for all the remaining directories.

I'm assuming that benefit of this (catching new warnings that may 
represent code errors/issues) outweighs the negative (your patch may 
compile file on your dev platform then bounce when you check it in--of 
course this is moot if you run your patches through tryserver).

If anyone thinks this is a bad idea, it's probably a good time to pipe up.

Jason