Implemented setting memory limits through WebMediaPlayer interface

content/renderer/media/android/webmediaplayer_android.cc

Index: content/renderer/media/android/webmediaplayer_android.cc
diff --git a/content/renderer/media/android/webmediaplayer_android.cc b/content/renderer/media/android/webmediaplayer_android.cc
index dfad9db0f2f7f4c036516947b9cc7b0bb02ac0e1..4564d52d7f8c1be394cc62ad9511d7cc0633dbb5 100644
--- a/content/renderer/media/android/webmediaplayer_android.cc
+++ b/content/renderer/media/android/webmediaplayer_android.cc
@@ -759,6 +759,34 @@ unsigned WebMediaPlayerAndroid::videoDecodedByteCount() const {
   return 0;
 }
 
+size_t WebMediaPlayerAndroid::getAudioBufferSize() const {
+  if (media_source_delegate_)
+    return media_source_delegate_->getAudioBufferSize();
+  NOTIMPLEMENTED();

[ddorwin, 2015/06/17 21:02:27]

As mentioned in https://codereview.chromium.org/1182183004/, this only makes
sense in the MSE case. Thus, it would make more sense to use an MSE-specific
interface (and avoid all the NOTIMPLEMENTED()/NOTREACHED()/return 0.

Also, should this be NOTREACHED() if we only support this for MSE?

[servolk, 2015/06/17 22:20:58]

See my comments on other CLs: this could make sense also for other cases (e.g.
direct URL playback through FFmpeg or any other media playback through WMP). I
was hoping to provide a common unified interface for setting memory buffer sizes
for media playback that could be later implemented by other media players
(including WebMediaPlayerMS and for the case of the alternative non-MSE playback
here). It's just that MSE is highest-priority use case and we can live with
others being not implemented for now.

+  return 0;
+}
+
+void WebMediaPlayerAndroid::setAudioBufferSize(size_t size) {
+  if (media_source_delegate_)
+    media_source_delegate_->setAudioBufferSize(size);
+  else
+    NOTIMPLEMENTED();
+}
+
+size_t WebMediaPlayerAndroid::getVideoBufferSize() const {
+  if (media_source_delegate_)
+    return media_source_delegate_->getVideoBufferSize();
+  NOTIMPLEMENTED();
+  return 0;
+}
+
+void WebMediaPlayerAndroid::setVideoBufferSize(size_t size) {
+  if (media_source_delegate_)
+    media_source_delegate_->setVideoBufferSize(size);
+  else
+    NOTIMPLEMENTED();
+}
+
 void WebMediaPlayerAndroid::OnMediaMetadataChanged(
     const base::TimeDelta& duration, int width, int height, bool success) {
   DCHECK(main_thread_checker_.CalledOnValidThread());