]> source.dussan.org Git - vaadin-framework.git/commitdiff
Enable use of profiler without debug window (#12465)
authorHenri Sara <hesara@vaadin.com>
Wed, 28 Aug 2013 07:43:39 +0000 (10:43 +0300)
committerVaadin Code Review <review@vaadin.com>
Wed, 28 Aug 2013 09:16:48 +0000 (09:16 +0000)
This change also corrects a typo in the name of the internal method
setProfilerResultConsumer()

Change-Id: I877858d992611512ee681022312de7a9bfa4cd39

client/src/com/vaadin/client/Profiler.java
client/src/com/vaadin/client/debug/internal/ProfilerSection.java

index bcbf8279f3f70dc03635d3e4b7d504075c80c20b..083f2559b15458edf1c0a700f3a011b682ce7258 100644 (file)
@@ -273,7 +273,9 @@ public class Profiler {
             }
         });
 
-        getConsumer().addProfilerData(stack.getFirst(), totalList);
+        if (getConsumer() != null) {
+            getConsumer().addProfilerData(stack.getFirst(), totalList);
+        }
     }
 
     /**
@@ -325,7 +327,9 @@ public class Profiler {
                 return;
             }
 
-            getConsumer().addBootstrapData(timings);
+            if (getConsumer() != null) {
+                getConsumer().addBootstrapData(timings);
+            }
         }
     }
 
@@ -386,11 +390,11 @@ public class Profiler {
      * <b>Warning!</b> This is internal API and should not be used by
      * applications or add-ons.
      * 
-     * @since 7.1
+     * @since 7.1.4
      * @param profilerResultConsumer
      *            the consumer that gets profiler data
      */
-    public static void setProfilerResultConsuer(
+    public static void setProfilerResultConsumer(
             ProfilerResultConsumer profilerResultConsumer) {
         if (consumer != null) {
             throw new IllegalStateException("The consumer has already been set");
@@ -399,11 +403,7 @@ public class Profiler {
     }
 
     private static ProfilerResultConsumer getConsumer() {
-        if (consumer == null) {
-            throw new IllegalStateException("No consumer has been registered");
-        } else {
-            return consumer;
-        }
+        return consumer;
     }
 
     private static Logger getLogger() {
index 91f626b9a59be4f8f7c5fd31c151c17aa9b48818..4a2a3a1c38e7e4bc0c0af1399b4b51354dd0e799 100644 (file)
@@ -283,7 +283,7 @@ public class ProfilerSection implements Section {
     private final FlowPanel content = new FlowPanel();
 
     public ProfilerSection() {
-        Profiler.setProfilerResultConsuer(new ProfilerResultConsumer() {
+        Profiler.setProfilerResultConsumer(new ProfilerResultConsumer() {
             @Override
             public void addProfilerData(Node rootNode, List<Node> totals) {
                 double totalTime = 0;