macOS: Fix a bug in logic to get all memory regions.

components/tracing/common/process_metrics_memory_dump_provider.cc

 // 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 "components/tracing/common/process_metrics_memory_dump_provider.h"
 
 #include <fcntl.h>
 #include <stdint.h>
 
 #include <map>
@@ skipping 412 matching lines @@
 
 // Creates VMRegions from mach_vm_region_recurse. Returns whether the operation
 // succeeded.
 bool GetAllRegions(std::vector<VMRegion>* regions) {
   const int pid = getpid();
   task_t task = mach_task_self();
   mach_vm_size_t size = 0;
   vm_region_submap_info_64 info;
   natural_t depth = 1;
   mach_msg_type_number_t count = sizeof(info);
-  for (mach_vm_address_t address = MACH_VM_MIN_ADDRESS;; address += size) {
+  mach_vm_address_t address = MACH_VM_MIN_ADDRESS;
+  while (true) {
     memset(&info, 0, sizeof(info));
     kern_return_t kr = mach_vm_region_recurse(
         task, &address, &size, &depth,
         reinterpret_cast<vm_region_info_t>(&info), &count);
     if (kr == KERN_INVALID_ADDRESS)  // nothing else left
       break;
     if (kr != KERN_SUCCESS)  // something bad
       return false;
     if (info.is_submap) {
       size = 0;
@@ skipping 260 matching lines @@
 #endif
 }
 
 void ProcessMetricsMemoryDumpProvider::SuspendFastMemoryPolling() {
 #if defined(OS_LINUX) || defined(OS_ANDROID)
   fast_polling_statm_fd_.reset();
 #endif
 }
 
 }  // namespace tracing