Abort MSP::OnPrefetchDone() if just after MSP::Release(). Let seek and config change survive.

content/browser/media/android/browser_media_player_manager.h

Index: content/browser/media/android/browser_media_player_manager.h
diff --git a/content/browser/media/android/browser_media_player_manager.h b/content/browser/media/android/browser_media_player_manager.h
index 93b75927d3ed7d910398791ab8c4a2b7e0bcac63..dc0f0fe00953a0501205785ae9c9db0e832a1733 100644
--- a/content/browser/media/android/browser_media_player_manager.h
+++ b/content/browser/media/android/browser_media_player_manager.h
@@ -100,6 +100,7 @@ class CONTENT_EXPORT BrowserMediaPlayerManager
                             const std::string& session_id,
                             const std::vector<uint8>& message,
                             const std::string& destination_url) OVERRIDE;
+  virtual void OnMediaDecoderCallback() OVERRIDE {}

[qinmin, 2013/10/29 23:49:34]

For test functions, normally use names like "OnMediaDecoderCallbackForTesting()"

[wolenetz, 2013/10/30 00:06:45]

Done. Note that there is now an associated presubmit warning for the call to
this from production code (in media_source_player.cc:560):
"You might be calling functions intended only for testing from
production code.  It is OK to ignore this warning if you know what
you are doing, as the heuristics used to detect the situation are
not perfect.  The commit queue will not block on this warning."

 
 #if defined(GOOGLE_TV)
   void AttachExternalVideoSurface(int player_id, jobject surface);