media: Add "Hide players" button in about://media-internals

media: Add "Hide players" button in about://media-internals

What this does is:
- Adds a button to hide all current players on the page, but doesn't touch the C++ side to actually delete those players.
- After a player is hidden, new logs for that player will be dropped and will not resurrect any hidden players.
- If a user refreshes the media-internals page, or open a new one, we'll get all players back.

BUG=688607
Review-Url: https://codereview.chromium.org/2672193003
Cr-Commit-Position: refs/heads/master@{#448764}
Committed: https://chromium.googlesource.com/chromium/src/+/4ee2eeb27fb5cb49b1d5fa470528c16738d0274d

[xhwang, 2017-02-05 06:30:57]

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

[commit-bot: I haz the power, 2017-02-05 06:31:07]

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-02-05 06:37:28]

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

[commit-bot: I haz the power, 2017-02-05 06:37:29]

Dry run: This issue passed the CQ dry run.

[xhwang, 2017-02-05 06:54:34]

Description was changed from

==========
media: Add "Clear players" button in about://media-internals

BUG=688607
==========

to

==========
media: Add "Hide players" button in about://media-internals

BUG=688607
==========

[xhwang, 2017-02-05 06:55:01]

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

[commit-bot: I haz the power, 2017-02-05 06:55:08]

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-02-05 07:04:37]

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

[commit-bot: I haz the power, 2017-02-05 07:04:37]

Dry run: This issue passed the CQ dry run.

[xhwang, 2017-02-07 20:43:22]

xhwang@chromium.org changed reviewers:
+ sandersd@chromium.org

[xhwang, 2017-02-07 20:43:22]

As discussed offline with sandersd. Maybe we can land this first, then sandersd@
will add more features like hooking up with the C++ side.

With that, PTAL

[sandersd (OOO until July 31), 2017-02-07 20:45:24]

lgtm

[xhwang, 2017-02-07 20:48:26]

Description was changed from

==========
media: Add "Hide players" button in about://media-internals

BUG=688607
==========

to

==========
media: Add "Hide players" button in about://media-internals

What this does is:
- Adds a button to hide all current players on the page, but doesn't touch the
C++ side to actually delete those players.
- After a player is hidden, new logs for that player will be dropped and will
not resurrect any hidden players.
- If a user refreshes the media-internals page, or open a new one, we'll get all
players back.

BUG=688607
==========

[xhwang, 2017-02-07 20:48:37]

The CQ bit was checked by xhwang@chromium.org

[commit-bot: I haz the power, 2017-02-07 20:49:49]

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-02-07 22:56:57]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1486500517854220,
"parent_rev": "40ec5a581b662b9f120e577df28363c3e5991990", "commit_rev":
"4ee2eeb27fb5cb49b1d5fa470528c16738d0274d"}

[commit-bot: I haz the power, 2017-02-07 22:57:54]

Description was changed from

==========
media: Add "Hide players" button in about://media-internals

What this does is:
- Adds a button to hide all current players on the page, but doesn't touch the
C++ side to actually delete those players.
- After a player is hidden, new logs for that player will be dropped and will
not resurrect any hidden players.
- If a user refreshes the media-internals page, or open a new one, we'll get all
players back.

BUG=688607
==========

to

==========
media: Add "Hide players" button in about://media-internals

What this does is:
- Adds a button to hide all current players on the page, but doesn't touch the
C++ side to actually delete those players.
- After a player is hidden, new logs for that player will be dropped and will
not resurrect any hidden players.
- If a user refreshes the media-internals page, or open a new one, we'll get all
players back.

BUG=688607

Review-Url: https://codereview.chromium.org/2672193003
Cr-Commit-Position: refs/heads/master@{#448764}
Committed:
https://chromium.googlesource.com/chromium/src/+/4ee2eeb27fb5cb49b1d5fa470528...
==========

[commit-bot: I haz the power, 2017-02-07 22:57:56]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/4ee2eeb27fb5cb49b1d5fa470528...

[shimazu, 2017-02-08 01:31:08]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/2682013002/ by shimazu@chromium.org.

The reason for reverting is: Two following tests fail:
-
 WebUIResourceBrowserTest.MediaInternals_Integration
- WebUIResourceBrowserTest.MediaInternals_Manager.