Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(7)

Unified Diff: base/process_util_mac.mm

Issue 10854027: Remove log message for 10.8 CFAllocator patching. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: base/process_util_mac.mm
diff --git a/base/process_util_mac.mm b/base/process_util_mac.mm
index 1ac30fe55812560ced7db63a45e0aab6825af8e1..df539df21a42326dd77f98fc89aaf5d13ed6f614 100644
--- a/base/process_util_mac.mm
+++ b/base/process_util_mac.mm
@@ -770,14 +770,6 @@ void oom_killer_new() {
// === Core Foundation CFAllocators ===
bool CanGetContextForCFAllocator() {
- // TODO(avi): remove at final release; http://crbug.com/117476
- if (base::mac::IsOSMountainLion()) {
- NSLog(@"Unsure about the internals of CFAllocator but going to patch them "
- "anyway. If there is a crash inside of CFAllocatorAllocate, please "
- "report it at http://crbug.com/117476 . If there is a crash and it "
- "is NOT inside of CFAllocatorAllocate, it is NOT RELATED. DO NOT "
- "REPORT IT THERE but rather FILE A NEW BUG.");
- }
return !base::mac::
IsOSDangerouslyLaterThanMountainLionForUseByCFAllocatorReplacement();
Mark Mentovai 2012/08/07 16:43:03 We’re gonna need to remember to do something with
Avi (use Gerrit) 2012/08/07 16:54:54 I plan on doing all of the same work; digging up t
}
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698