Expose a seekable range at 0 for streaming sources with duration.

content/renderer/media/android/webmediaplayer_android.cc

 // Copyright 2013 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.
 
 #include "content/renderer/media/android/webmediaplayer_android.h"
 
 #include <limits>
 
 #include "base/android/build_info.h"
 #include "base/bind.h"
 #include "base/callback_helpers.h"
 #include "base/command_line.h"
 #include "base/files/file_path.h"
+#include "base/float_util.h"
 #include "base/logging.h"
 #include "base/metrics/histogram.h"
 #include "base/single_thread_task_runner.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/utf_string_conversions.h"
 #include "cc/blink/web_layer_impl.h"
 #include "cc/layers/video_layer.h"
 #include "content/public/common/content_client.h"
 #include "content/public/common/content_switches.h"
 #include "content/public/renderer/render_frame.h"
@@ skipping 494 matching lines @@
     return media_source_delegate_->Buffered();
   return buffered_;
 }
 
 blink::WebTimeRanges WebMediaPlayerAndroid::seekable() const {
   // If we haven't even gotten to ReadyStateHaveMetadata yet then there
   // are no seekable ranges.
   if (ready_state_ < WebMediaPlayer::ReadyStateHaveMetadata)
     return blink::WebTimeRanges();
 
-  // If we have a duration then use [0, duration] as the seekable range.
+  // Media without duration are considered streaming and should not be seekable.
   const double seekable_end = duration();
-  if (!seekable_end)
+  if (!base::IsFinite(seekable_end))

[philipj_slow, 2014/10/01 09:37:50]

With this change I believe you can remove the readyState check above, because
duration() already returns NaN for that case.

[DaleCurtis, 2014/10/06 23:55:42]

Done.

     return blink::WebTimeRanges();
 
+  // If we have a finite duration then use [0, duration] as the seekable range.
   blink::WebTimeRange seekable_range(0.0, seekable_end);
   return blink::WebTimeRanges(&seekable_range, 1);
 }
 
 bool WebMediaPlayerAndroid::didLoadingProgress() {
   bool ret = did_loading_progress_;
   did_loading_progress_ = false;
   return ret;
 }
 
@@ skipping 1286 matching lines @@
 
 bool WebMediaPlayerAndroid::IsHLSStream() const {
   std::string mime;
   GURL url = redirected_url_.is_empty() ? url_ : redirected_url_;
   if (!net::GetMimeTypeFromFile(base::FilePath(url.path()), &mime))
     return false;
   return !mime.compare("application/x-mpegurl");
 }
 
 }  // namespace content