Refactoring: Remove an argument 'enabled' from onHeapProfilingEnabled

third_party/WebKit/Source/platform/heap/BlinkGCMemoryDumpProvider.cpp

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "platform/heap/BlinkGCMemoryDumpProvider.h"
 
 #include "base/trace_event/heap_profiler_allocation_context_tracker.h"
 #include "base/trace_event/heap_profiler_allocation_register.h"
 #include "base/trace_event/trace_event_memory_overhead.h"
 #include "platform/heap/Handle.h"
@@ skipping 63 matching lines @@
             m_allocationRegister->EstimateTraceMemoryOverhead(&overhead);
         }
         memoryDump->dumpHeapUsage(bytesByContext, overhead, "blink_gc");
     }
 
     // Merge all dumps collected by Heap::collectGarbage.
     memoryDump->takeAllDumpsFrom(m_currentProcessMemoryDump.get());
     return true;
 }
 
-void BlinkGCMemoryDumpProvider::onHeapProfilingEnabled(bool enabled)
+void BlinkGCMemoryDumpProvider::onHeapProfilingEnabled()
 {
-    if (enabled) {
-        {
-            MutexLocker locker(m_allocationRegisterMutex);
-            if (!m_allocationRegister)
-                m_allocationRegister = adoptPtr(new base::trace_event::AllocationRegister());
-        }
-        HeapAllocHooks::setAllocationHook(reportAllocation);
-        HeapAllocHooks::setFreeHook(reportFree);
-    } else {
-        HeapAllocHooks::setAllocationHook(nullptr);
-        HeapAllocHooks::setFreeHook(nullptr);
+    ASSERT(!m_isHeapProfilingEnabled);
+    {
+        MutexLocker locker(m_allocationRegisterMutex);
+        if (!m_allocationRegister)
+            m_allocationRegister = adoptPtr(new base::trace_event::AllocationRegister());
     }
-    m_isHeapProfilingEnabled = enabled;
+    HeapAllocHooks::setAllocationHook(reportAllocation);
+    HeapAllocHooks::setFreeHook(reportFree);
+    m_isHeapProfilingEnabled = true;
 }
 
 WebMemoryAllocatorDump* BlinkGCMemoryDumpProvider::createMemoryAllocatorDumpForCurrentGC(const String& absoluteName)
 {
     return m_currentProcessMemoryDump->createMemoryAllocatorDump(absoluteName);
 }
 
 void BlinkGCMemoryDumpProvider::clearProcessDumpForCurrentGC()
 {
     m_currentProcessMemoryDump->clear();
@@ skipping 16 matching lines @@
 }
 
 void BlinkGCMemoryDumpProvider::remove(Address address)
 {
     MutexLocker locker(m_allocationRegisterMutex);
     if (m_allocationRegister)
         m_allocationRegister->Remove(address);
 }
 
 } // namespace blink