Fix compilation for the case dcheck_always_on=1.

Fix compilation for the case dcheck_always_on=1.

It is a default option for some try bots.

otherwise compilation fails on
DCHECK(layout_descriptor->IsConsistentWithMap(*map));

BUG=none
LOG=n

Committed: https://crrev.com/2cfebcf92e8518f9c970ce72d2c4fabfae4e1e06
Cr-Commit-Position: refs/heads/master@{#26636}

[loislo, 2015-02-13 08:18:28]

loislo@chromium.org changed reviewers:
+ ishell@chromium.org, svenpanne@chromium.org, verwaest@chromium.org

[loislo, 2015-02-13 08:51:44]

loislo@chromium.org changed reviewers:
+ danno@chromium.org

[Sven Panne, 2015-02-13 09:07:58]

The CQ bit was checked by svenpanne@chromium.org

[Sven Panne, 2015-02-13 09:07:59]

lgtm

LGTM, although I'm not totally convinced that this is enough. Conditional
interfaces are a bad idea, and this #ifdef horror is the price we pay for
this... :-/

[commit-bot: I haz the power, 2015-02-13 09:09:05]

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

[danno, 2015-02-13 09:11:21]

The CQ bit was unchecked by danno@chromium.org

[danno, 2015-02-13 09:12:57]

Actually, I've been chatting with Ilya and I think the conditional interface is
not the way to go. I've asked him to clean up the test to use the style of
#ifdef VERIFY_HEAP + flag checking that is used elsewhere in the runtime for
heap verification.

[loislo, 2015-02-13 09:19:47]

New patchsets have been uploaded after l-g-t-m from svenpanne@chromium.org

[loislo, 2015-02-13 09:21:18]

Patchset #3 (id:40001) has been deleted

[loislo, 2015-02-13 09:22:16]

comments addressed.

PTAL

[danno, 2015-02-13 09:23:55]

lgtm

[danno, 2015-02-13 09:23:59]

The CQ bit was checked by danno@chromium.org

[commit-bot: I haz the power, 2015-02-13 09:24:05]

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

[loislo, 2015-02-13 09:38:37]

The CQ bit was unchecked by loislo@chromium.org

[loislo, 2015-02-13 09:47:01]

New patchsets have been uploaded after l-g-t-m from danno@chromium.org

[loislo, 2015-02-13 09:49:10]

The CQ bit was checked by loislo@chromium.org

[commit-bot: I haz the power, 2015-02-13 09:49:35]

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

[commit-bot: I haz the power, 2015-02-13 10:29:18]

Committed patchset #4 (id:80001)

[commit-bot: I haz the power, 2015-02-13 10:29:36]

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

[Igor Sheludko, 2015-02-16 10:29:10]

On 2015/02/13 10:29:36, I haz the power (commit-bot) wrote:
> Patchset 4 (id:??) landed as
> https://crrev.com/2cfebcf92e8518f9c970ce72d2c4fabfae4e1e06
> Cr-Commit-Position: refs/heads/master@{#26636}

Thank you for taking care of this!