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

media/base/android/media_player_manager.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef MEDIA_BASE_ANDROID_MEDIA_PLAYER_MANAGER_H_
 #define MEDIA_BASE_ANDROID_MEDIA_PLAYER_MANAGER_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 88 matching lines @@
   virtual void OnKeyError(int media_keys_id,
                           const std::string& session_id,
                           media::MediaKeys::KeyError error_code,
                           int system_code) = 0;
 
   // Called when MediaDrmBridge wants to send a KeyMessage.
   virtual void OnKeyMessage(int media_keys_id,
                             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
 
 #endif  // MEDIA_BASE_ANDROID_MEDIA_PLAYER_MANAGER_H_