Disable zapping of code space.

Disable zapping of code space.

BUG=

Committed: https://crrev.com/8fdcd13b5248161a40bda8663828791c1c902b06
Cr-Commit-Position: refs/heads/master@{#29386}

[Michael Starzinger, 2015-06-30 15:05:07]

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

[Michael Starzinger, 2015-06-30 15:05:07]

LGTM. It's but a five character change, nothing can go wrong.

[Hannes Payer (out of office), 2015-06-30 15:05:24]

The CQ bit was checked by hpayer@chromium.org

[commit-bot: I haz the power, 2015-06-30 15:05:34]

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

[jochen (gone - plz use gerrit), 2015-06-30 15:15:08]

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

[jochen (gone - plz use gerrit), 2015-06-30 15:15:09]

https://codereview.chromium.org/1214333002/diff/1/src/flag-definitions.h
File src/flag-definitions.h (left):

https://codereview.chromium.org/1214333002/diff/1/src/flag-definitions.h#oldc...
src/flag-definitions.h:688: DEFINE_BOOL(zap_code_space, true,
can you make the default depend on debug vs release?

[Hannes Payer (out of office), 2015-06-30 15:45:05]

The CQ bit was unchecked by hpayer@chromium.org

[Hannes Payer (out of office), 2015-06-30 15:49:33]

https://codereview.chromium.org/1214333002/diff/1/src/flag-definitions.h
File src/flag-definitions.h (left):

https://codereview.chromium.org/1214333002/diff/1/src/flag-definitions.h#oldc...
src/flag-definitions.h:688: DEFINE_BOOL(zap_code_space, true,
On 2015/06/30 15:15:09, jochen wrote:
> can you make the default depend on debug vs release?

Done.

[Hannes Payer (out of office), 2015-06-30 15:56:45]

The CQ bit was checked by hpayer@chromium.org

[Hannes Payer (out of office), 2015-06-30 15:56:46]

The patchset sent to the CQ was uploaded after l-g-t-m from
mstarzinger@chromium.org
Link to the patchset: https://codereview.chromium.org/1214333002/#ps20001
(title: " ")

[commit-bot: I haz the power, 2015-06-30 15:56:54]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1214333002/20001

[jochen (gone - plz use gerrit), 2015-06-30 16:08:56]

lgtm

[commit-bot: I haz the power, 2015-06-30 16:29:03]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-06-30 16:29:14]

Patchset 2 (id:??) landed as
https://crrev.com/8fdcd13b5248161a40bda8663828791c1c902b06
Cr-Commit-Position: refs/heads/master@{#29386}