Temporarily CHECK use after free in Value

Temporarily CHECK use after free in Value

COMMIT=false

This CL was abandoned in favour of https://codereview.chromium.org/2743363004, because the minor change would spread into too many depending switch-case statements.

One of the hypotheses for https://crbug.com/697817 is that a Value is used
after free (see #5 there). To verify/refute that hypothesis, this CL
temporarily adds CHECKS and a new Value::Type (DELETED). This change will be
reverted after a couple of days in Canary, and only serves investigation
purposes.

BUG=697817

[vabr (Chromium), 2017-03-14 10:39:09]

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

[commit-bot: I haz the power, 2017-03-14 10:39:30]

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-03-14 10:46:34]

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

[commit-bot: I haz the power, 2017-03-14 10:46:35]

Dry run: Try jobs failed on following builders:
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)

[vabr (Chromium), 2017-03-14 10:59:11]

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

[commit-bot: I haz the power, 2017-03-14 10:59:22]

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-03-14 11:06:09]

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

[commit-bot: I haz the power, 2017-03-14 11:06:09]

Dry run: Try jobs failed on following builders:
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)

[vabr (Chromium), 2017-03-14 11:16:50]

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

[commit-bot: I haz the power, 2017-03-14 11:17:06]

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

[vabr (Chromium), 2017-03-14 11:29:34]

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

[commit-bot: I haz the power, 2017-03-14 11:29:53]

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

[vabr (Chromium), 2017-03-14 11:38:30]

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

[commit-bot: I haz the power, 2017-03-14 11:38:45]

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-03-14 11:45:57]

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

[commit-bot: I haz the power, 2017-03-14 11:45:58]

Dry run: Try jobs failed on following builders:
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  win_clang on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_clang/builds/...)

[vabr (Chromium), 2017-03-14 12:26:39]

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

[commit-bot: I haz the power, 2017-03-14 12:26:55]

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-03-14 12:56:22]

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

[commit-bot: I haz the power, 2017-03-14 12:56:23]

Dry run: Try jobs failed on following builders:
  win_clang on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_clang/builds/...)

[vabr (Chromium), 2017-03-14 13:01:51]

Description was changed from

==========
Temporarily CHECK use after free in Value

One of the hypotheses for https://crbug.com/697817 is that a Value is used
after free (see #5 there). To verify/refute that hypothesis, this CL
temporarily adds CHECKS and a new Value::Type (DELETED). This change will be
reverted after a couple of days in Canary, and only serves investigation
purposes.

BUG=697817
==========

to

==========
Temporarily CHECK use after free in Value

COMMIT=false

This CL was abandoned, because the minor change would spread into too many
depending switch-case statements.

One of the hypotheses for https://crbug.com/697817 is that a Value is used
after free (see #5 there). To verify/refute that hypothesis, this CL
temporarily adds CHECKS and a new Value::Type (DELETED). This change will be
reverted after a couple of days in Canary, and only serves investigation
purposes.

BUG=697817
==========

[vabr (Chromium), 2017-03-14 14:44:05]

Description was changed from

==========
Temporarily CHECK use after free in Value

COMMIT=false

This CL was abandoned, because the minor change would spread into too many
depending switch-case statements.

One of the hypotheses for https://crbug.com/697817 is that a Value is used
after free (see #5 there). To verify/refute that hypothesis, this CL
temporarily adds CHECKS and a new Value::Type (DELETED). This change will be
reverted after a couple of days in Canary, and only serves investigation
purposes.

BUG=697817
==========

to

==========
Temporarily CHECK use after free in Value

COMMIT=false

This CL was abandoned in favour of https://codereview.chromium.org/2743363004,
because the minor change would spread into too many depending switch-case
statements.

One of the hypotheses for https://crbug.com/697817 is that a Value is used
after free (see #5 there). To verify/refute that hypothesis, this CL
temporarily adds CHECKS and a new Value::Type (DELETED). This change will be
reverted after a couple of days in Canary, and only serves investigation
purposes.

BUG=697817
==========

[vabr (Chromium), 2017-03-14 14:44:23]

This was closed in favour of https://codereview.chromium.org/2743363004