Build mac and linux release builds with symbols and don't strip them by default

Build mac and linux release builds with symbols and don't strip them by default

On Mac, we already build with symbols, but strip them.

Building with symbols (a) matches what we do for official builds and (b)
makes crashes on release builders more useful.

BUG=315598
R=mark@chromium.org, thakis@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=234257

[Mark Mentovai, 2013-11-08 16:36:51]

Did you test this out with a buildtype=Official branding=Chrome build on the
Mac?

[jochen (gone - plz use gerrit), 2013-11-08 19:02:04]

On 2013/11/08 16:36:51, Mark Mentovai wrote:
> Did you test this out with a buildtype=Official branding=Chrome build on the
> Mac?

No, I didn't. I'll need to get an internal checkout next week to verify.

It's, however, a copy of the same rule linux already had:
https://code.google.com/p/chromium/codesearch#chromium/src/build/common.gypi&...

[jochen (gone - plz use gerrit), 2013-11-11 12:18:59]

On 2013/11/08 16:36:51, Mark Mentovai wrote:
> Did you test this out with a buildtype=Official branding=Chrome build on the
> Mac?

Here are the sizes with and without the patch:

Without:
$ stat -f %z Google\ Chrome\ Framework.framework/Versions/Current/Google\
Chrome\ Framework
80130500
$ stat -f %z Google\ Chrome.app/Contents/MacOS/Google\ Chrome
8524

With:
$ stat -f %z Google\ Chrome\ Framework.framework/Versions/Current/Google\
Chrome\ Framework
80130512
$ stat -f %z Google\ Chrome.app/Contents/MacOS/Google\ Chrome
8524

no idea where the 12 byte increase comes from but it seems benign

[Mark Mentovai, 2013-11-11 12:20:12]

LGTM

[commit-bot: I haz the power, 2013-11-11 12:21:23]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jochen@chromium.org/66673005/1

[jochen (gone - plz use gerrit), 2013-11-11 19:25:10]

Committed patchset #1 manually as r234257 (presubmit successful).