Remove performanceMonitor()->unsubscribeAll() from Performance's destructor

Remove performanceMonitor()->unsubscribeAll() from Performance's destructor

Performance object is on Oilpan's heap, so its destructor is not allowed to touch
other on-heap objects.

Also we don't need to call performanceMonitor()->unsubscribeAll() because it is
just unregistering a back pointer Member to the Performance object from the map
in PerformanceMonitor.

BUG=

Committed: https://crrev.com/18502d9c2a5ec6c54e8f7cd2a11bb2e45e78e331
Cr-Commit-Position: refs/heads/master@{#439081}

[haraken, 2016-12-16 02:57:46]

haraken@chromium.org changed reviewers:
+ sigbjornf@opera.com

[haraken, 2016-12-16 02:57:46]

PTAL

[sof, 2016-12-16 07:36:49]

lgtm

[haraken, 2016-12-16 07:40:26]

The CQ bit was checked by haraken@chromium.org

[commit-bot: I haz the power, 2016-12-16 07:40:43]

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-12-16 10:00:40]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1481874026956190, "parent_rev":
"36fd8d0f46d153d865b76b78b767b6047a70bafe", "commit_rev":
"106f261472880f5fd4c95d735e469ac477a2e1de"}

[commit-bot: I haz the power, 2016-12-16 10:01:07]

Description was changed from

==========
Remove performanceMonitor()->unsubscribeAll() from Performance's destructor

Performance object is on Oilpan's heap, so its destructor is not allowed to
touch
other on-heap objects.

Also we don't need to call performanceMonitor()->unsubscribeAll() because it is
just unregistering a back pointer Member to the Performance object from the map
in PerformanceMonitor.

BUG=
==========

to

==========
Remove performanceMonitor()->unsubscribeAll() from Performance's destructor

Performance object is on Oilpan's heap, so its destructor is not allowed to
touch
other on-heap objects.

Also we don't need to call performanceMonitor()->unsubscribeAll() because it is
just unregistering a back pointer Member to the Performance object from the map
in PerformanceMonitor.

BUG=

Review-Url: https://codereview.chromium.org/2584733003
==========

[commit-bot: I haz the power, 2016-12-16 10:01:08]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-16 10:03:22]

Description was changed from

==========
Remove performanceMonitor()->unsubscribeAll() from Performance's destructor

Performance object is on Oilpan's heap, so its destructor is not allowed to
touch
other on-heap objects.

Also we don't need to call performanceMonitor()->unsubscribeAll() because it is
just unregistering a back pointer Member to the Performance object from the map
in PerformanceMonitor.

BUG=

Review-Url: https://codereview.chromium.org/2584733003
==========

to

==========
Remove performanceMonitor()->unsubscribeAll() from Performance's destructor

Performance object is on Oilpan's heap, so its destructor is not allowed to
touch
other on-heap objects.

Also we don't need to call performanceMonitor()->unsubscribeAll() because it is
just unregistering a back pointer Member to the Performance object from the map
in PerformanceMonitor.

BUG=

Committed: https://crrev.com/18502d9c2a5ec6c54e8f7cd2a11bb2e45e78e331
Cr-Commit-Position: refs/heads/master@{#439081}
==========

[commit-bot: I haz the power, 2016-12-16 10:03:22]

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