mac: Periodically shim new malloc zones immediately after startup.

base/allocator/allocator_interception_mac.h

Index: base/allocator/allocator_interception_mac.h
diff --git a/base/allocator/allocator_interception_mac.h b/base/allocator/allocator_interception_mac.h
index f0d945132baec8d837e182f922eaab5e6ad6be1b..0c74f0984d386c5cf3e6e74be5c2134e7c800adc 100644
--- a/base/allocator/allocator_interception_mac.h
+++ b/base/allocator/allocator_interception_mac.h
@@ -35,6 +35,10 @@ bool UncheckedCallocMac(size_t num_items, size_t size, void** result);
 // Has no effect on the default malloc zone if the allocator shim already
 // performs that interception.
 BASE_EXPORT void InterceptAllocationsMac();
+
+// Periodically checks for, and shims new malloc zones.

[Mark Mentovai, 2017/02/23 02:26:17]

This has an expiration date on it, which is kind of important. Mention that
here.

[erikchen, 2017/04/03 06:44:41]

Done.

+BASE_EXPORT void PeriodicallyShimNewMallocZones();

[Mark Mentovai, 2017/02/23 02:26:17]

And this one only does what it says if a certain command-line argument is
present. Its name doesn’t give that away at all, and there’s no comment saying
so either.

Both of these are kind of important for callers to know without having to read
the implementation.

[Primiano Tucci (use gerrit), 2017/02/23 14:41:57]

+1, or alternatively you can move the cmdline check in the caller, which would
make the code in browser_main_loop more reassuring.

[erikchen, 2017/04/03 06:44:41]

I did what primiano suggested.

+
 }  // namespace allocator
 }  // namespace base