Feature: Add ProfilingData object cache in Profiler

src/Profiler.php

@@ -48,6 +48,11 @@ final class Profiler
      */
     private $requestContext;
 
+    /**
+     * @var ProfilingData|null
+     */
+    private $profilingData;
+
     /**
      * Simple state variable to hold the value of 'Is the profiler running or not?'
      *
@@ -161,7 +166,7 @@ public function disable()
             throw new ProfilerException('Unable to disable profiler: Request context is missing');
         }
 
-        $profile = new ProfilingData($this->config);
+        $profile = $this->getProfilingData();
 
         return $profile->getProfilingData($data, $context);
     }
@@ -328,4 +333,16 @@ private function captureRequestContext()
 
         return $context;
     }
+
+    /**
+     * @return ProfilingData
+     */
+    private function getProfilingData()
+    {
+        if ($this->profilingData === null) {
+            $this->profilingData = new ProfilingData($this->config);
+        }
+
+        return $this->profilingData;
+    }
 }

tests/ProfilerTest.php

@@ -2,12 +2,29 @@
 
 namespace Xhgui\Profiler\Test;
 
+use ReflectionMethod;
+use Xhgui\Profiler\Config;
 use Xhgui\Profiler\Exception\ProfilerException;
+use Xhgui\Profiler\ProfilingData;
 use Xhgui\Profiler\Profiler;
 use Xhgui\Profiler\Test\Resources\TestProfilerStub;
 
 class ProfilerTest extends TestCase
 {
+    public function testGetProfilingDataCachesInstance()
+    {
+        $profiler = new Profiler(new Config(array()));
+        $method = new ReflectionMethod($profiler, 'getProfilingData');
+        $method->setAccessible(true);
+
+        $first = $method->invoke($profiler);
+        $second = $method->invoke($profiler);
+
+        $this->assertInstanceOf(ProfilingData::class, $first);
+        $this->assertSame($first, $second);
+        $this->assertSame($first, $this->getPrivateProperty($profiler, 'profilingData'));
+    }
+
     public function testDisableClearsStateWhenRequestContextIsMissing()
     {
         $profiler = new Profiler(array());