Sandbox warmup - localtime().

chrome/renderer/renderer_main_platform_delegate_mac.mm

 // Copyright (c) 2009 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 "chrome/renderer/renderer_main_platform_delegate.h"
 
 #include "base/debug_util.h"
 
 #import <Foundation/Foundation.h>
 #import <ApplicationServices/ApplicationServices.h>
@@ skipping 29 matching lines @@
                                            rgb_colorspace,
                                            kCGImageAlphaPremultipliedFirst |
                                            kCGBitmapByteOrder32Host);
   CGColorSpaceRelease(rgb_colorspace);
   CGContextRelease(tmp);
 
   // Warm up NSColor conversion which also accesses the FS on it's first call.
   NSColor *color = [NSColor controlTextColor];
   [color colorUsingColorSpaceName:NSCalibratedRGBColorSpace];
 
-  time_t tv = time(NULL);
+  // Warm up localtime().

[brettw, 2009/02/25 18:00:20]

I don't understand why this is here at all, and your comment doesn't help. Can
you elaborate?

+  time_t tv = {0};

[John Grabowski, 2009/02/25 18:29:47]

I think I understand why the warm-up (sandbox so we want locale files), but I
don't see why time(NULL) --> {0} is desired.

   localtime(&tv);
 
 #if 0
 
   // Note: by default, Cocoa is NOT thread safe.  Use of NSThreads
   // tells Cocoa to be MT-aware and create and use locks.  The
   // renderer process only uses Cocoa from the single renderer thread,
   // so we don't need to tell Cocoa we are using threads (even though,
   // oddly, we are using Cocoa from the non-main thread.)
   // The current limit of renderer processes is 20.  Brett states that
@@ skipping 59 matching lines @@
   if (error == -1) {
     LOG(ERROR) << "Failed to Initialize Sandbox: " << error_buff;
   }
   sandbox_free_error(error_buff);
   return success;
 }
 
 void RendererMainPlatformDelegate::RunSandboxTests() {
   // TODO(port): Run sandbox unit test here.
 }