Ignore fired timer if the object wrapper is no longer alive

Ignore fired timer if the object wrapper is no longer alive

CL [1] has prevented gin from recreating wrappers after GC,
so doesn't fire timer if its wrapper is empty otherwise it will
be crashed.

[1] https://chromium-review.googlesource.com/475077

BUG=707689
Review-Url: https://codereview.chromium.org/2830723003
Cr-Commit-Position: refs/heads/master@{#466314}
Committed: https://chromium.googlesource.com/chromium/src/+/bb89503baf314afded40d6517c8e6e18f1f89b6b

[junwei, 2017-04-20 11:51:18]

junwei.fu@intel.com changed reviewers:
+ rockot@chromium.org

[junwei, 2017-04-20 11:51:18]

PTAL.

[Ken Rockot(use gerrit already), 2017-04-20 14:22:39]

Lgtm but I don't own this code

[junwei, 2017-04-21 00:25:39]

junwei.fu@intel.com changed reviewers:
+ jochen@chromium.org

[junwei, 2017-04-21 00:25:40]

Jochen@ PTAL.

[jochen (gone - plz use gerrit), 2017-04-21 08:41:24]

lgtm

[junwei, 2017-04-21 08:44:32]

The CQ bit was checked by junwei.fu@intel.com to run a CQ dry run

[commit-bot: I haz the power, 2017-04-21 08:44:50]

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-04-21 08:54:08]

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

[commit-bot: I haz the power, 2017-04-21 08:54:09]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[junwei, 2017-04-21 10:32:00]

The CQ bit was checked by junwei.fu@intel.com to run a CQ dry run

[commit-bot: I haz the power, 2017-04-21 10:32:21]

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

[junwei, 2017-04-21 11:06:38]

Patchset #1 (id:1) has been deleted

[commit-bot: I haz the power, 2017-04-21 11:31:03]

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

[commit-bot: I haz the power, 2017-04-21 11:31:04]

Dry run: This issue passed the CQ dry run.

[junwei, 2017-04-21 11:31:57]

The CQ bit was checked by junwei.fu@intel.com

[junwei, 2017-04-21 11:31:58]

The patchset sent to the CQ was uploaded after l-g-t-m from rockot@chromium.org,
jochen@chromium.org
Link to the patchset: https://codereview.chromium.org/2830723003/#ps20001
(title: "Ignore fired timer if the object wrapper is no longer alive")

[commit-bot: I haz the power, 2017-04-21 11:32:08]

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-04-21 11:35:39]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1492774317805070,
"parent_rev": "6844efc9fd050f138609c07837ac93172feb2903", "commit_rev":
"bb89503baf314afded40d6517c8e6e18f1f89b6b"}

[commit-bot: I haz the power, 2017-04-21 11:36:17]

Description was changed from

==========
Ignore fired timer if the object wrapper is no longer alive

CL [1] has prevented gin from recreating wrappers after GC,
so doesn't fire timer if its wrapper is empty otherwise it will
be crashed.

[1] https://chromium-review.googlesource.com/475077

BUG=707689
==========

to

==========
Ignore fired timer if the object wrapper is no longer alive

CL [1] has prevented gin from recreating wrappers after GC,
so doesn't fire timer if its wrapper is empty otherwise it will
be crashed.

[1] https://chromium-review.googlesource.com/475077

BUG=707689

Review-Url: https://codereview.chromium.org/2830723003
Cr-Commit-Position: refs/heads/master@{#466314}
Committed:
https://chromium.googlesource.com/chromium/src/+/bb89503baf314afded40d6517c8e...
==========

[commit-bot: I haz the power, 2017-04-21 11:36:18]

Committed patchset #1 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/bb89503baf314afded40d6517c8e...