Media Galleries API Picasa: Make PicasaDataProvider handle async PMP and INI parsing robustly.

content/common/sandbox_mac.mm

 // Copyright (c) 2012 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 "content/common/sandbox_mac.h"
 
 #import <Cocoa/Cocoa.h>
 
+#include <CoreFoundation/CFTimeZone.h>
 extern "C" {
 #include <sandbox.h>
 }
 #include <signal.h>
 #include <sys/param.h>
 
 #include "base/basictypes.h"
 #include "base/command_line.h"
 #include "base/compiler_specific.h"
 #include "base/file_util.h"
@@ skipping 281 matching lines @@
         CGImageSourceCreateWithData((CFDataRef)data, NULL));
     CGImageSourceGetStatus(img);
   }
 
   {
     // Allow access to /dev/urandom.
     base::GetUrandomFD();
   }
 
   // Process-type dependent warm-up.
+  if (sandbox_type == SANDBOX_TYPE_UTILITY) {
+    // Allow access to local timezones.

[jeremy, 2013/08/30 23:02:51]

Can you provide a much more detailed comment about why we need it/which system
calls fail on which os versions?

The purpose of the comments in this file are to try to provide enough
information that we know when/if it's safe to remove warmup code.

[tommycli, 2013/08/30 23:14:20]

Done.

+    CFTimeZoneCopySystem();
+  }
+
   if (sandbox_type == SANDBOX_TYPE_GPU) {
     // Preload either the desktop GL or the osmesa so, depending on the
     // --use-gl flag.
     gfx::GLSurface::InitializeOneOff();
   }
 }
 
 // static
 NSString* Sandbox::BuildAllowDirectoryAccessSandboxString(
     const base::FilePath& allowed_dir,
@@ skipping 281 matching lines @@
   if (HANDLE_EINTR(fcntl(fd, F_GETPATH, canonical_path)) != 0) {
     DPLOG(FATAL) << "GetCanonicalSandboxPath() failed for: "
                  << path.value();
     return path;
   }
 
   return base::FilePath(canonical_path);
 }
 
 }  // namespace content