http://gwt-code-reviews.appspot.com/651803/diff/3001/4001 File bikeshed/src/com/google/gwt/sample/expenses/gwt/Scaffold.gwt.xml (right):
http://gwt-code-reviews.appspot.com/651803/diff/3001/4001#newcode28 bikeshed/src/com/google/gwt/sample/expenses/gwt/Scaffold.gwt.xml:28: than the system handler, so only that one is enabled for now --> On 2010/07/26 16:49:24, Ray Ryan wrote:
Are all of these things really enabled by default? Not that I'm
against
including them here, they're great documentation, but I'm curious.
simpleRemoteHandler and hasWidgetsHandler (the one ppl use to log to their own panel rather than the popup panel) are not on by default. However console, devMode, firebug and system and popup are on by default. In reality, console and firebug are essentially siblings - one works for firebug and the other works for things like firebug-lite, and I think Safari's JS debugging console. So - by default we log to stderr, DevMode, JsDebuggingConsoles (firebug + friends) and the super annoying popup. I think enough people use the first 3 to justify logging to all of them. For the popup, we've left in on by default so that if someone turns on logging, they definitely know it, and hopefully need to learn some absolute basics about configuring logging in order to turn it off (since not understanding logging configuration at all, or enabling it accidentally, is actually kind of bad) http://gwt-code-reviews.appspot.com/651803/diff/3001/4002 File user/src/com/google/gwt/requestfactory/client/RequestFactoryLogHandler.java (right): http://gwt-code-reviews.appspot.com/651803/diff/3001/4002#newcode59 user/src/com/google/gwt/requestfactory/client/RequestFactoryLogHandler.java:59: setLevel(Level.INFO); On 2010/07/26 16:49:24, Ray Ryan wrote:
How about providing an alternate constructor that accepts a Level
argument, and
make this comment its javadoc?
Done. http://gwt-code-reviews.appspot.com/651803/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors
