Disable the pending task if the memory reducer is torn down.

Disable the pending task if the memory reducer is torn down.

BUG=chromium:508584
LOG=NO

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

[ulan, 2015-07-10 09:08:46]

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

[ulan, 2015-07-10 09:08:47]

PTAL

[jochen (gone - plz use gerrit), 2015-07-10 09:10:54]

lgtm

https://codereview.chromium.org/1230163002/diff/20001/src/heap/memory-reducer.cc
File src/heap/memory-reducer.cc (right):

https://codereview.chromium.org/1230163002/diff/20001/src/heap/memory-reducer...
src/heap/memory-reducer.cc:183: pending_task_ = NULL;
nullptr here and everywhere

[ulan, 2015-07-10 09:18:54]

Thanks, landing

https://codereview.chromium.org/1230163002/diff/20001/src/heap/memory-reducer.cc
File src/heap/memory-reducer.cc (right):

https://codereview.chromium.org/1230163002/diff/20001/src/heap/memory-reducer...
src/heap/memory-reducer.cc:183: pending_task_ = NULL;
On 2015/07/10 09:10:54, jochen wrote:
> nullptr here and everywhere

Done.

[ulan, 2015-07-10 09:18:59]

The CQ bit was checked by ulan@chromium.org

[ulan, 2015-07-10 09:19:00]

The patchset sent to the CQ was uploaded after l-g-t-m from jochen@chromium.org
Link to the patchset: https://codereview.chromium.org/1230163002/#ps60001
(title: "more nullptr")

[commit-bot: I haz the power, 2015-07-10 09:19:08]

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

[commit-bot: I haz the power, 2015-07-10 09:21:50]

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

[commit-bot: I haz the power, 2015-07-10 09:21:51]

Try jobs failed on following builders:
  v8_mac_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_mac_rel/builds/7579)

(exceeded global retry quota)

[ulan, 2015-07-10 09:25:52]

The CQ bit was checked by ulan@chromium.org

[ulan, 2015-07-10 09:25:52]

The patchset sent to the CQ was uploaded after l-g-t-m from jochen@chromium.org
Link to the patchset: https://codereview.chromium.org/1230163002/#ps80001
(title: "Fix check")

[commit-bot: I haz the power, 2015-07-10 09:25:59]

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

[commit-bot: I haz the power, 2015-07-10 09:30:31]

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

[commit-bot: I haz the power, 2015-07-10 09:30:32]

Try jobs failed on following builders:
  v8_linux_chromium_gn_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_chromium_gn_rel/bu...)

(exceeded global retry quota)

[ulan, 2015-07-10 09:32:32]

The CQ bit was checked by ulan@chromium.org

[commit-bot: I haz the power, 2015-07-10 09:32:37]

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

[commit-bot: I haz the power, 2015-07-10 10:07:39]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2015-07-10 10:08:02]

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