Don't crash debug builds for full allocator.

Don't crash debug builds for full allocator.

The "full" error is caused by things outside of the control of this
code so don't crash because of it.

BUG=675600
Review-Url: https://codereview.chromium.org/2638093002
Cr-Commit-Position: refs/heads/master@{#444101}
Committed: https://chromium.googlesource.com/chromium/src/+/734c246453da8b9a38d7266fe2bbd272037c99e8

[bcwhite, 2017-01-17 16:42:24]

bcwhite@chromium.org changed reviewers:
+ asvitkine@chromium.org

[Alexei Svitkine (slow), 2017-01-17 16:48:01]

LGTM

Can you add a comment above the if to explain the reasoning?

[bcwhite, 2017-01-17 17:01:40]

The CQ bit was checked by bcwhite@chromium.org to run a CQ dry run

[bcwhite, 2017-01-17 17:01:58]

On 2017/01/17 16:48:01, Alexei Svitkine (slow) wrote:
> LGTM
> 
> Can you add a comment above the if to explain the reasoning?

Done.

[commit-bot: I haz the power, 2017-01-17 17:02:10]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-17 17:52:24]

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

[commit-bot: I haz the power, 2017-01-17 17:52:25]

Dry run: This issue passed the CQ dry run.

[bcwhite, 2017-01-17 18:45:29]

The CQ bit was checked by bcwhite@chromium.org

[bcwhite, 2017-01-17 18:45:30]

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

[commit-bot: I haz the power, 2017-01-17 18:45:52]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-17 18:51:49]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1484678729782060,
"parent_rev": "71ee072729403850ee522e08fbd669d3bc2f142f", "commit_rev":
"734c246453da8b9a38d7266fe2bbd272037c99e8"}

[commit-bot: I haz the power, 2017-01-17 18:52:21]

Description was changed from

==========
Don't crash debug builds for full allocator.

The "full" error is caused by things outside of the control of this
code so don't crash because of it.

BUG=675600
==========

to

==========
Don't crash debug builds for full allocator.

The "full" error is caused by things outside of the control of this
code so don't crash because of it.

BUG=675600

Review-Url: https://codereview.chromium.org/2638093002
Cr-Commit-Position: refs/heads/master@{#444101}
Committed:
https://chromium.googlesource.com/chromium/src/+/734c246453da8b9a38d7266fe2bb...
==========

[commit-bot: I haz the power, 2017-01-17 18:52:22]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/734c246453da8b9a38d7266fe2bb...