Fire CanPlayThrough immediately for local and streaming media files

media/base/download_rate_monitor.h

Index: media/base/download_rate_monitor.h
diff --git a/media/base/download_rate_monitor.h b/media/base/download_rate_monitor.h
index ac88f7ffbed5b58a967cf04df407f4d64f44559c..775c3ce28355b7b2a4542f3aad1ec5cc54eb4d60 100644
--- a/media/base/download_rate_monitor.h
+++ b/media/base/download_rate_monitor.h
@@ -42,6 +42,10 @@ class MEDIA_EXPORT DownloadRateMonitor {
 
   void set_loaded(bool loaded) { loaded_ = loaded; }
 
+  void set_media_stream(bool media_stream) { media_stream_ = media_stream; }
+
+  void set_streaming(bool streaming) { streaming_ = streaming; }
+
   // Stop monitoring download rate. This does not discard previously learned
   // information, but it will no longer factor incoming information into its
   // canplaythrough estimation.
@@ -147,6 +151,13 @@ class MEDIA_EXPORT DownloadRateMonitor {
   // otherwise.
   bool stopped_;
 
+  // True if the data source is a media stream, e.g. video input from a local
+  // capture device, false otherwise.
+  bool media_stream_;
+
+  // True if the data source is a streaming source, false otherwise.
+  bool streaming_;

[scherkus (not reviewing), 2011/11/23 01:59:44]

wouldn't setting streaming=true for web cams also work w/o introducing the
concept of a media stream?

[vrk (LEFT CHROMIUM), 2011/11/23 21:29:45]

There were 2 reasons why I introduced media stream:

1) Technically there is a slight difference in their definitions, the most
important difference being that a local webcam stream does not download data,
whereas every other stream type does. In terms of DownloadRateMonitor, this
means a local webcam stream *must* immediately fire CanPlayThrough, but firing
CanPlayThrough immediately for other types of streams is a design decision.

2) The SetStreaming method in PipelineImpl is part of the FilterHost interface,
meaning I had to add a differently named method to the Pipeline interface anyway
to get the WebMediaPlayerImpl -> PipelineImpl communication. So I figured I may
as well be specific.

I agree it's not ideal :/ so I'm willing to change.

Btw, in the currently checked-in code, can MediaStream be anything but a local
webcam feed? (I know the spec says it can be a remote source, but is that
implemented and checked in?) If it's always a local source, I could at least
rename things to make things clearer.

+
   DISALLOW_COPY_AND_ASSIGN(DownloadRateMonitor);
 };