Add debug symbols to all generated .isolate.

Add debug symbols to all generated .isolate.

This will increase the amount of data being uploaded by a ~2x factor or so.
Debug symbols are usually larger than the executable itself on pretty much all
toolsets but generally have the advantage of being highly compressible. The
increase of data not a problem anymore with isolate-go. There will be a small
increase of latency but it should be bearable, especially by the gain of
enabling stack traces to work for isolated tests on Swarming.

R=jam@chromium.org
BUG=98637
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/8125ce3c695e6d8c02f3b51b16d49e17e7934bb4
Cr-Commit-Position: refs/heads/master@{#349231}

[M-A Ruel, 2015-07-20 16:46:32]

Patchset #1 (id:1) has been deleted

[M-A Ruel, 2015-09-09 19:05:53]

Patchset #1 (id:20001) has been deleted

[M-A Ruel, 2015-09-14 16:00:08]

Patchset #2 (id:60001) has been deleted

[M-A Ruel, 2015-09-14 16:00:19]

Patchset #2 (id:80001) has been deleted

[M-A Ruel, 2015-09-14 16:46:52]

Patchset #1 (id:40001) has been deleted

[M-A Ruel, 2015-09-14 16:46:57]

Patchset #1 (id:100001) has been deleted

[M-A Ruel, 2015-09-15 00:47:31]

Patchset #2 (id:140001) has been deleted

[M-A Ruel, 2015-09-15 00:47:39]

Patchset #1 (id:120001) has been deleted

[M-A Ruel, 2015-09-15 01:17:10]

Patchset #2 (id:180001) has been deleted

[M-A Ruel, 2015-09-15 12:27:11]

Patchset #1 (id:160001) has been deleted

[M-A Ruel, 2015-09-15 14:52:58]

Patchset #1 (id:200001) has been deleted

[M-A Ruel, 2015-09-15 16:16:56]

Patchset #3 (id:260001) has been deleted

[M-A Ruel, 2015-09-15 16:17:09]

Patchset #3 (id:280001) has been deleted

[M-A Ruel, 2015-09-15 16:17:14]

Patchset #2 (id:240001) has been deleted

[M-A Ruel, 2015-09-15 17:35:45]

Patchset #1 (id:220001) has been deleted

[M-A Ruel, 2015-09-15 17:35:57]

Patchset #1 (id:300001) has been deleted

[M-A Ruel, 2015-09-15 17:36:03]

Patchset #1 (id:320001) has been deleted

[M-A Ruel, 2015-09-15 17:36:15]

Patchset #1 (id:340001) has been deleted

[M-A Ruel, 2015-09-15 17:36:20]

Patchset #1 (id:360001) has been deleted

[M-A Ruel, 2015-09-15 18:03:06]

Patchset #1 (id:380001) has been deleted

[M-A Ruel, 2015-09-15 19:28:56]

Patchset #1 (id:400001) has been deleted

[M-A Ruel, 2015-09-15 19:30:18]

It works.
It did increase the amount of data uploaded on Windows, from ~2Gb to ~4Gb but
didn't increase the duration of 'isolate tests', for example
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...
took 1m10s which is totally reasonable.

https://isolateserver.appspot.com/browse?namespace=default-gzip&hash=7a1356f6...
confirms chrome.dll.pdb and chrome.exe.pdb and friends are automatically added.

The CL itself is ugly but I still wanted to do it so we would have a baseline
that PDBs are pushed ASAP, so it can safely be enabled in GN right away and we
can assume debug symbols should always be pushed. Hopefully this code will be
deleted within the next year. (I'm also optimistic).

[M-A Ruel, 2015-09-15 22:31:30]

maruel@chromium.org changed reviewers:
- dpranke@chromium.org, thakis@chromium.org

[jam, 2015-09-16 20:06:47]

rubberstamp lgtm

great!

[M-A Ruel, 2015-09-16 20:07:35]

The CQ bit was checked by maruel@chromium.org

[commit-bot: I haz the power, 2015-09-16 20:08:50]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1245703002/420001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1245703002/420001

[commit-bot: I haz the power, 2015-09-16 20:25:01]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-09-16 20:25:03]

Try jobs failed on following builders:
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[M-A Ruel, 2015-09-16 20:30:46]

The CQ bit was checked by maruel@chromium.org

[commit-bot: I haz the power, 2015-09-16 20:31:57]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1245703002/420001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1245703002/420001

[commit-bot: I haz the power, 2015-09-16 21:05:59]

Committed patchset #1 (id:420001)

[commit-bot: I haz the power, 2015-09-16 21:06:36]

Patchset 1 (id:??) landed as
https://crrev.com/8125ce3c695e6d8c02f3b51b16d49e17e7934bb4
Cr-Commit-Position: refs/heads/master@{#349231}

[Ken Russell (switch to Gerrit), 2015-09-16 21:23:42]

kbr@chromium.org changed reviewers:
+ kbr@chromium.org

[Ken Russell (switch to Gerrit), 2015-09-16 21:23:42]

Thanks for pushing this forward!

[Nico, 2015-09-17 03:16:07]

A revert of this CL (patchset #1 id:420001) has been created in
https://codereview.chromium.org/1354523002/ by thakis@chromium.org.

The reason for reverting is: Speculative, possibly caused "dyld: Library not
loaded: @rpath/libchrome_main_dll.dylib" test failures in
http://build.chromium.org/p/chromium.mac/builders/Mac10.9%20Tests%20%28dbg%29....