Revision: 6830 Author: [email protected] Date: Tue Nov 10 20:40:00 2009 Log: Adding some useful PerfLogger messages.
Review by: bobv http://code.google.com/p/google-web-toolkit/source/detail?r=6830 Modified: /trunk/dev/core/src/com/google/gwt/dev/javac/StandardGeneratorContext.java /trunk/dev/core/src/com/google/gwt/dev/javac/TypeOracleMediator.java /trunk/dev/core/src/com/google/gwt/dev/resource/impl/ResourceOracleImpl.java /trunk/dev/core/src/com/google/gwt/dev/shell/OophmSessionHandler.java ======================================= --- /trunk/dev/core/src/com/google/gwt/dev/javac/StandardGeneratorContext.java Tue Nov 10 08:44:05 2009 +++ /trunk/dev/core/src/com/google/gwt/dev/javac/StandardGeneratorContext.java Tue Nov 10 20:40:00 2009 @@ -31,6 +31,7 @@ import com.google.gwt.dev.javac.impl.Shared; import com.google.gwt.dev.resource.ResourceOracle; import com.google.gwt.dev.util.DiskCache; +import com.google.gwt.dev.util.PerfLogger; import com.google.gwt.dev.util.Util; import com.google.gwt.dev.util.collect.HashSet; import com.google.gwt.dev.util.collect.IdentityHashMap; @@ -451,6 +452,8 @@ setCurrentGenerator(generatorClass); long before = System.currentTimeMillis(); + PerfLogger.start("Generator '" + generator.getClass().getName() + + "' produced '" + typeName + "'"); try { String className = generator.generate(logger, this, typeName); long after = System.currentTimeMillis(); @@ -466,6 +469,8 @@ logger.log(TreeLogger.ERROR, "Generator '" + generatorClass.getName() + "' threw threw an exception while rebinding '" + typeName + "'", e); throw new UnableToCompleteException(); + } finally { + PerfLogger.end(); } } ======================================= --- /trunk/dev/core/src/com/google/gwt/dev/javac/TypeOracleMediator.java Tue Nov 10 08:12:08 2009 +++ /trunk/dev/core/src/com/google/gwt/dev/javac/TypeOracleMediator.java Tue Nov 10 20:40:00 2009 @@ -57,6 +57,7 @@ import com.google.gwt.dev.javac.impl.SourceFileCompilationUnit; import com.google.gwt.dev.resource.Resource; import com.google.gwt.dev.util.Name; +import com.google.gwt.dev.util.PerfLogger; import com.google.gwt.dev.util.Name.InternalName; import java.io.PrintWriter; @@ -275,6 +276,7 @@ * Adds new units to an existing TypeOracle. */ public void addNewUnits(TreeLogger logger, Collection<CompilationUnit> units) { + PerfLogger.start("TypeOracleMediator.addNewUnits"); // First collect all class data and resurrect graveyard types. classMap = new HashMap<String, CollectClassData>(); for (CompilationUnit unit : units) { @@ -367,6 +369,7 @@ // no longer needed classMap = null; classMapType = null; + PerfLogger.end(); } public Map<String, JRealClassType> getBinaryMapper() { ======================================= --- /trunk/dev/core/src/com/google/gwt/dev/resource/impl/ResourceOracleImpl.java Thu Nov 5 10:42:22 2009 +++ /trunk/dev/core/src/com/google/gwt/dev/resource/impl/ResourceOracleImpl.java Tue Nov 10 20:40:00 2009 @@ -18,6 +18,7 @@ import com.google.gwt.core.ext.TreeLogger; import com.google.gwt.dev.resource.Resource; import com.google.gwt.dev.resource.ResourceOracle; +import com.google.gwt.dev.util.PerfLogger; import com.google.gwt.dev.util.msg.Message0; import com.google.gwt.dev.util.msg.Message1String; @@ -320,6 +321,7 @@ * @param logger status and error details are written here */ public void refresh(TreeLogger logger) { + PerfLogger.start("ResourceOracleImpl.refresh"); TreeLogger refreshBranch = Messages.REFRESHING_RESOURCES.branch(logger, null); @@ -386,6 +388,7 @@ if (!didChange) { // Nothing to do, keep the same identities. + PerfLogger.end(); return; } @@ -404,6 +407,7 @@ exposedResources = Collections.unmodifiableSet(externalSet); exposedResourceMap = Collections.unmodifiableMap(externalMap); exposedPathNames = Collections.unmodifiableSet(externalMap.keySet()); + PerfLogger.end(); } public void setPathPrefixes(PathPrefixSet pathPrefixSet) { ======================================= --- /trunk/dev/core/src/com/google/gwt/dev/shell/OophmSessionHandler.java Mon Nov 9 09:33:19 2009 +++ /trunk/dev/core/src/com/google/gwt/dev/shell/OophmSessionHandler.java Tue Nov 10 20:40:00 2009 @@ -20,6 +20,7 @@ import com.google.gwt.dev.shell.BrowserChannel.Value; import com.google.gwt.dev.shell.JsValue.DispatchMethod; import com.google.gwt.dev.shell.JsValue.DispatchObject; +import com.google.gwt.dev.util.PerfLogger; import java.lang.reflect.Member; import java.util.Collections; @@ -159,6 +160,7 @@ String tabKey, String sessionKey, byte[] userAgentIcon) { logger = loadModuleLogger; try { + PerfLogger.start("OophmSessionHandler.loadModule " + moduleName); // Attach a new ModuleSpace to make it programmable. // BrowserChannelServer serverChannel = (BrowserChannelServer) channel; @@ -169,6 +171,7 @@ ModuleSpace moduleSpace = new ModuleSpaceOOPHM(msh, moduleName, serverChannel); moduleMap.put(serverChannel, moduleSpace); + PerfLogger.start("ModuleSpace.onLoad"); moduleSpace.onLoad(logger); } catch (Throwable e) { // We do catch Throwable intentionally because there are a ton of things @@ -178,6 +181,9 @@ this.logger.log(TreeLogger.ERROR, "Failed to load module '" + moduleName + "' from user agent '" + userAgent + "' at " + channel.getRemoteEndpoint(), e); + } finally { + PerfLogger.end(); + PerfLogger.end(); } return this.logger; } --~--~---------~--~----~------------~-------~--~----~ http://groups.google.com/group/Google-Web-Toolkit-Contributors -~----------~----~----~----~------~----~------~--~---
