Fix crash-on-launch on Snow Leopard

Fix crash-on-launch on Snow Leopard

The signal-self syntax needed to fix bug 20370 doesn't work on Leopard and
causes the Sandbox to barf so renderers don't star tup.

We introduce a ;10.5_ONLY keyword to the sandbox that is substituted at runtime (the line with the rule is effectively uncommented).  This allows us to use leopard or snow-leapard-only sandbox syntax

BUG=20516
TEST=Chrome should no longer crash on launch under SL.

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=24777

[John Grabowski, 2009-08-28 19:41:33]

LGTM

http://codereview.chromium.org/183003/diff/1/2
File chrome/renderer/renderer.sb (right):

http://codereview.chromium.org/183003/diff/1/2#newcode10
Line 10: ; ;10.5_ONLY is substitued at runtime.
:;10.5_ONLY" is replaced with "" at runtime as needed.

[Mark Mentovai, 2009-08-28 21:50:11]

Are processes allowed to send themselves signals under 10.6?  Does yesterday's
bug not occur there for other reasons?

[jeremy, 2009-08-28 21:57:19]

This was a quick fix for the dev release.
I've got a CL ready to be submitted that uses syntax that works on 10.5 &
10.6 and reverts this patch.

On Fri, Aug 28, 2009 at 2:50 PM, <mark@chromium.org> wrote:

> Are processes allowed to send themselves signals under 10.6?  Does
> yesterday's bug not occur there for other reasons?
>
>
> http://codereview.chromium.org/183003
>