Release video resources when switching tabs or leave clank

Release video resources when switching tabs or leave clank

The media framework prefer at most 2 players playing. Release video decoding resources when switching tabs.


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=169505

[qinmin, 2012-11-21 19:22:03]

PTAL

[scherkus (not reviewing), 2012-11-21 21:44:30]

https://codereview.chromium.org/11418120/diff/1/content/renderer/render_view_...
File content/renderer/render_view_impl.cc (right):

https://codereview.chromium.org/11418120/diff/1/content/renderer/render_view_...
content/renderer/render_view_impl.cc:5732:
media_player_manager_->ReleaseMediaResources();
would it be more accurate to call this "ReleaseVideoResources()" seeing as this
no-ops if it's only audio?

[qinmin, 2012-11-21 21:53:53]

https://codereview.chromium.org/11418120/diff/1/content/renderer/render_view_...
File content/renderer/render_view_impl.cc (right):

https://codereview.chromium.org/11418120/diff/1/content/renderer/render_view_...
content/renderer/render_view_impl.cc:5732:
media_player_manager_->ReleaseMediaResources();
I think video is not correct, as it also released paused audio decoders.

On 2012/11/21 21:44:30, scherkus wrote:
> would it be more accurate to call this "ReleaseVideoResources()" seeing as
this
> no-ops if it's only audio?

[scherkus (not reviewing), 2012-11-21 22:24:41]

OK -- LGTM

[brettw, 2012-11-26 19:20:30]

owners lgtm rubberstamp

[commit-bot: I haz the power, 2012-11-26 19:26:34]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/qinmin@chromium.org/11418120/1

[commit-bot: I haz the power, 2012-11-26 22:25:31]

Change committed as 169505