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

media/base/android/media_player_manager.h

Index: media/base/android/media_player_manager.h
diff --git a/media/base/android/media_player_manager.h b/media/base/android/media_player_manager.h
index 668613bf5ac879ed52737dea51f17d9551a48ab1..2746f4d1111f2bca4732a05906f14c589bdbed35 100644
--- a/media/base/android/media_player_manager.h
+++ b/media/base/android/media_player_manager.h
@@ -106,6 +106,9 @@ class MEDIA_EXPORT MediaPlayerManager {
                             const std::string& session_id,
                             const std::vector<uint8>& message,
                             const std::string& destination_url) = 0;
+
+  // Test-only method to hook completion of decode cycles in MediaSourcePlayer.
+  virtual void OnMediaDecoderCallbackForTesting() = 0;

[acolwell GONE FROM CHROMIUM, 2013/10/30 23:29:53]

This does not seem right. Why don't you just put a
set_decode_callback_for_testing() method on MSP? Putting this on the manager
seems like a bad idea.

[wolenetz, 2013/10/31 01:39:11]

Thanks for catching this. Putting this in MSP simplifies the test code a little,
too (one less method to mock), and keeps the callback specific to the intended
player.

 };
 
 }  // namespace media