Media Controls: do not use events but C++ callbacks for insertion/removal from document.

Media Controls: do not use events but C++ callbacks for insertion/removal from document.

For some reasons, the overhead of the events create some oilpan
allocated free memory. It's not a memory leak but it increases the
visible memory usage of Chrome for some perf bots. Using C++ callbacks
resolve the issue and should be fairly similar as far as the
functionality goes.

BUG=697980
Review-Url: https://codereview.chromium.org/2724983006
Cr-Commit-Position: refs/heads/master@{#454772}
Committed: https://chromium.googlesource.com/chromium/src/+/8e679697d635baa92d4927b2b4e838f125266af1

[mlamouri (slow - plz ping), 2017-03-03 17:24:21]

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

[mlamouri (slow - plz ping), 2017-03-03 17:24:44]

mlamouri@chromium.org changed reviewers:
+ zqzhang@chromium.org

[mlamouri (slow - plz ping), 2017-03-03 17:24:44]

PTAL. Can you commit if it lgty :)

[commit-bot: I haz the power, 2017-03-03 17:24:55]

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

[mlamouri (slow - plz ping), 2017-03-03 17:33:02]

+primiano@ FYI

[commit-bot: I haz the power, 2017-03-03 18:41:50]

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

[commit-bot: I haz the power, 2017-03-03 18:41:51]

Dry run: This issue passed the CQ dry run.

[Zhiqiang Zhang (Slow), 2017-03-04 13:49:06]

The CQ bit was checked by zqzhang@chromium.org

[Zhiqiang Zhang (Slow), 2017-03-04 13:49:07]

lgtm

[commit-bot: I haz the power, 2017-03-04 13:49:15]

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-04 13:57:23]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1488635346476930, "parent_rev":
"48a24fcc7bcf3f9c02faf6b1e949eed37f246c81", "commit_rev":
"8e679697d635baa92d4927b2b4e838f125266af1"}

[commit-bot: I haz the power, 2017-03-04 13:58:08]

Description was changed from

==========
Media Controls: do not use events but C++ callbacks for insertion/removal from
document.

For some reasons, the overhead of the events create some oilpan
allocated free memory. It's not a memory leak but it increases the
visible memory usage of Chrome for some perf bots. Using C++ callbacks
resolve the issue and should be fairly similar as far as the
functionality goes.

BUG=697980
==========

to

==========
Media Controls: do not use events but C++ callbacks for insertion/removal from
document.

For some reasons, the overhead of the events create some oilpan
allocated free memory. It's not a memory leak but it increases the
visible memory usage of Chrome for some perf bots. Using C++ callbacks
resolve the issue and should be fairly similar as far as the
functionality goes.

BUG=697980

Review-Url: https://codereview.chromium.org/2724983006
Cr-Commit-Position: refs/heads/master@{#454772}
Committed:
https://chromium.googlesource.com/chromium/src/+/8e679697d635baa92d4927b2b4e8...
==========

[commit-bot: I haz the power, 2017-03-04 13:58:09]

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