Move (hopefully) remaining isolate related variables to toolchain.gypi

Move (hopefully) remaining isolate related variables to toolchain.gypi

BUG=chromium:609107
R=machenbach@chromium.org

Committed: https://crrev.com/a68a1eb38dda0a1eba3b7644a172d2f85e9c7abe
Cr-Commit-Position: refs/heads/master@{#36664}

[Michael Achenbach, 2016-06-02 06:46:03]

lgtm - you might also need to move jsfunfuzz variable?

https://codereview.chromium.org/2027213002/diff/1/gypfiles/toolchain.gypi
File gypfiles/toolchain.gypi (right):

https://codereview.chromium.org/2027213002/diff/1/gypfiles/toolchain.gypi#new...
gypfiles/toolchain.gypi:37: 'coverage%': 0,
Did you remove this somewhere in return?

[Michael Achenbach, 2016-06-02 06:47:47]

On 2016/06/02 06:46:03, Michael Achenbach wrote:
> lgtm - you might also need to move jsfunfuzz variable?

Hmm looks like we're setting it:
https://build.chromium.org/p/client.v8/builders/V8%20Linux64%20-%20debug%20bu...

But it seems to not exist anymore, so never mind. The logic was simplified on
the v8 side...

[jochen (gone - plz use gerrit), 2016-06-02 06:48:39]

jsfunfuzz is not referenced from isolate.gypi, so I don't think it's needed

https://codereview.chromium.org/2027213002/diff/1/gypfiles/toolchain.gypi
File gypfiles/toolchain.gypi (right):

https://codereview.chromium.org/2027213002/diff/1/gypfiles/toolchain.gypi#new...
gypfiles/toolchain.gypi:37: 'coverage%': 0,
On 2016/06/02 at 06:46:03, Michael Achenbach wrote:
> Did you remove this somewhere in return?

it's also used in standalone.gypi directly, and that file also duplicates all
other *san variables, etc.. so I didn't delete it there.

[jochen (gone - plz use gerrit), 2016-06-02 06:50:03]

The CQ bit was checked by jochen@chromium.org

[commit-bot: I haz the power, 2016-06-02 06:50:06]

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

[commit-bot: I haz the power, 2016-06-02 06:52:04]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-02 06:53:06]

Description was changed from

==========
Move (hopefully) remaining isolate related variables to toolchain.gypi

BUG=chromium:609107
R=machenbach@chromium.org
==========

to

==========
Move (hopefully) remaining isolate related variables to toolchain.gypi

BUG=chromium:609107
R=machenbach@chromium.org

Committed: https://crrev.com/a68a1eb38dda0a1eba3b7644a172d2f85e9c7abe
Cr-Commit-Position: refs/heads/master@{#36664}
==========

[commit-bot: I haz the power, 2016-06-02 06:53:07]

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

[Michael Achenbach, 2016-06-02 07:10:04]

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

The reason for reverting is: Fails:
https://build.chromium.org/p/client.v8.fyi/builders/V8-Blink%20Linux%2064/bui....