Fix double-delete in BatteryMonitor.

Fix double-delete in BatteryMonitor.

BatterMonitorImpl is uniquely owned by its StrongBinding, so deleting it
directly results in a double-delete.

BUG=653298
Committed: https://crrev.com/4d2bc44e0a5c25025fbad9126003ac9e347bc723
Cr-Commit-Position: refs/heads/master@{#423536}

[dcheng, 2016-10-05 23:16:40]

dcheng@chromium.org changed reviewers:
+ rockot@chromium.org, timvolodine@chromium.org

[dcheng, 2016-10-05 23:16:40]

+timvolodine for OWNERS stamp
+rockot to make sure I'm doing it all correctly

[dcheng, 2016-10-05 23:19:19]

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

[commit-bot: I haz the power, 2016-10-05 23:19:55]

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

[Ken Rockot(use gerrit already), 2016-10-06 00:11:16]

lgtm

[commit-bot: I haz the power, 2016-10-06 00:19:36]

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

[commit-bot: I haz the power, 2016-10-06 00:19:37]

Dry run: This issue passed the CQ dry run.

[timvolodine, 2016-10-06 13:08:41]

On 2016/10/06 00:11:16, Ken Rockot wrote:
> lgtm

rs lgtm.
Is this come kind of anti-pattern? It looks like we have similar situations in
other pieces of code..

[dcheng, 2016-10-06 15:13:15]

On 2016/10/06 13:08:41, timvolodine wrote:
> On 2016/10/06 00:11:16, Ken Rockot wrote:
> > lgtm
> 
> rs lgtm.
> Is this come kind of anti-pattern? It looks like we have similar situations in
> other pieces of code..

|delete this| is a pretty frequent construct; it's just one that happens to be
unsafe with Mojo + StrongBinding. Unfortunately, there's not a good way to
automatically detect this so someone will have to go through and audit...

[dcheng, 2016-10-06 15:13:24]

The CQ bit was checked by dcheng@chromium.org

[commit-bot: I haz the power, 2016-10-06 15:13:42]

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, 2016-10-06 15:17:39]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-06 15:21:10]

Description was changed from

==========
Fix double-delete in BatteryMonitor.

BatterMonitorImpl is uniquely owned by its StrongBinding, so deleting it
directly results in a double-delete.

BUG=653298
==========

to

==========
Fix double-delete in BatteryMonitor.

BatterMonitorImpl is uniquely owned by its StrongBinding, so deleting it
directly results in a double-delete.

BUG=653298

Committed: https://crrev.com/4d2bc44e0a5c25025fbad9126003ac9e347bc723
Cr-Commit-Position: refs/heads/master@{#423536}
==========

[commit-bot: I haz the power, 2016-10-06 15:21:11]

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