Remove MediaPlayerAndroid::IsSurfaceInUse() call

Remove MediaPlayerAndroid::IsSurfaceInUse() call

When tearing down a MediaSourcePlayer, the video codec could still be decoding.
Therefore, we introduced IsSurfaceInUse() call to destroy the surface view after codec finishes decoding.
However, this function doesn't really help due to 2 reasons:
1. Surface view can go away any time. For example, user click exitfullscreen button.
   So waiting for decoder to gracefully finish decoding is not applicable in many cases.
2. Waiting for decoder to finish before destroying the surface view might introduce the following racing issue:
   Video A tries to exit fullscreen, but pending decoder job deletion.
   Video B calls enter fullscreen, but since the surface is in use, it will fail to do so.
   video A exits fullscreen, but video B doesn't know that, and it will wait forever.

This change removes the asynchronous behavior when ReleaseMediaResources() is called.
This makes the code more easy to read and avoids the racing issue above.

BUG=422158
Committed: https://crrev.com/5f39149e06b561f79625450c0aaf69be3c5fc95b
Cr-Commit-Position: refs/heads/master@{#299556}

[qinmin, 2014-10-14 20:21:33]

qinmin@chromium.org changed reviewers:
+ gunsch@chromium.org, xhwang@chromium.org

[qinmin, 2014-10-14 20:21:35]

PTAL

[gunsch, 2014-10-14 20:23:37]

On 2014/10/14 20:21:35, qinmin wrote:
> PTAL

lgtm

[qinmin, 2014-10-14 20:41:20]

The CQ bit was checked by qinmin@chromium.org

[commit-bot: I haz the power, 2014-10-14 20:42:19]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/651243003/1

[commit-bot: I haz the power, 2014-10-14 21:38:22]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2014-10-14 21:51:30]

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