Refactoring: Remove an argument 'enabled' from onHeapProfilingEnabled

third_party/WebKit/Source/platform/PartitionAllocMemoryDumpProvider.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/PartitionAllocMemoryDumpProvider.h"
 
 #include "base/trace_event/heap_profiler_allocation_context.h"
 #include "base/trace_event/heap_profiler_allocation_context_tracker.h"
 #include "base/trace_event/heap_profiler_allocation_register.h"
 #include "base/trace_event/process_memory_dump.h"
@@ skipping 130 matching lines @@
 PartitionAllocMemoryDumpProvider::PartitionAllocMemoryDumpProvider()
     : m_allocationRegister(nullptr)
     , m_isHeapProfilingEnabled(false)
 {
 }
 
 PartitionAllocMemoryDumpProvider::~PartitionAllocMemoryDumpProvider()
 {
 }
 
-void PartitionAllocMemoryDumpProvider::onHeapProfilingEnabled(bool enabled)
+void PartitionAllocMemoryDumpProvider::onHeapProfilingEnabled()
 {
-    if (enabled) {
-        {
-            MutexLocker locker(m_allocationRegisterMutex);
-            if (!m_allocationRegister)
-                m_allocationRegister = adoptPtr(new base::trace_event::AllocationRegister());
-        }
-        PartitionAllocHooks::setAllocationHook(reportAllocation);
-        PartitionAllocHooks::setFreeHook(reportFree);
-    } else {
-        PartitionAllocHooks::setAllocationHook(nullptr);
-        PartitionAllocHooks::setFreeHook(nullptr);
+    ASSERT(!m_isHeapProfilingEnabled);
+    {
+        MutexLocker locker(m_allocationRegisterMutex);
+        if (!m_allocationRegister)
+            m_allocationRegister = adoptPtr(new base::trace_event::AllocationRegister());
     }
-    m_isHeapProfilingEnabled = enabled;
+    PartitionAllocHooks::setAllocationHook(reportAllocation);
+    PartitionAllocHooks::setFreeHook(reportFree);
+    m_isHeapProfilingEnabled = true;
 }
 
 void PartitionAllocMemoryDumpProvider::insert(void* address, size_t size, const char* typeName)
 {
     base::trace_event::AllocationContext context = base::trace_event::AllocationContextTracker::GetContextSnapshot();
     context.type_name = typeName;
     MutexLocker locker(m_allocationRegisterMutex);
     if (m_allocationRegister)
         m_allocationRegister->Insert(address, size, context);
 }
 
 void PartitionAllocMemoryDumpProvider::remove(void* address)
 {
     MutexLocker locker(m_allocationRegisterMutex);
     if (m_allocationRegister)
         m_allocationRegister->Remove(address);
 }
 
 } // namespace blink