Suppress pause and buffer for local resources

content/renderer/media/buffered_data_source_unittest.cc

Index: content/renderer/media/buffered_data_source_unittest.cc
diff --git a/content/renderer/media/buffered_data_source_unittest.cc b/content/renderer/media/buffered_data_source_unittest.cc
index 6cf7ef42b6e5114bc4a474c5ea5c46bbdb5447ea..c6ed70d671813a21e90104fc6c0d8dcb02014b88 100644
--- a/content/renderer/media/buffered_data_source_unittest.cc
+++ b/content/renderer/media/buffered_data_source_unittest.cc
@@ -219,6 +219,10 @@ class BufferedDataSourceTest : public testing::Test {
   int data_source_playback_rate() { return data_source_->playback_rate_; }
   int loader_bitrate() { return loader()->bitrate_; }
   int loader_playback_rate() { return loader()->playback_rate_; }
+  bool is_local_source() { return data_source_->assume_fully_buffered_; }
+  void set_might_be_reused_from_cache_in_future(bool value) {
+    loader()->might_be_reused_from_cache_in_future_ = value;
+  }
 
   scoped_ptr<MockBufferedDataSource> data_source_;
 
@@ -669,4 +673,33 @@ TEST_F(BufferedDataSourceTest, File_FinishLoading) {
   Stop();
 }
 
+TEST_F(BufferedDataSourceTest, LocalResource_DeferStrategy) {
+  InitializeWithFileResponse();
+
+  EXPECT_TRUE(is_local_source());
+  EXPECT_EQ(BufferedResourceLoader::kCapacityDefer, defer_strategy());
+
+  // Test for pause-and-buffer

[scherkus (not reviewing), 2014/05/30 20:37:39]

nit: comments should end with periods

you should also expand this to cover why we get this result

[amogh.bihani, 2014/06/02 05:01:28]

Done.

+  data_source_->MediaIsPlaying();
+  data_source_->MediaIsPaused();
+  EXPECT_EQ(BufferedResourceLoader::kReadThenDefer, defer_strategy());
+
+  Stop();
+}
+
+TEST_F(BufferedDataSourceTest, ExternalResource_DeferStrategy) {
+  InitializeWith206Response();
+
+  EXPECT_FALSE(is_local_source());
+  EXPECT_EQ(BufferedResourceLoader::kCapacityDefer, defer_strategy());
+
+  // Test for pause-and-buffer
+  data_source_->MediaIsPlaying();
+  set_might_be_reused_from_cache_in_future(true);

[amogh.bihani, 2014/05/29 09:38:57]

Need to do this as buffered_resource_loader set it false changes the defer
strategy[1] for the test URL.
---------
[1]
https://code.google.com/p/chromium/codesearch#chromium/src/content/renderer/m...

+  data_source_->MediaIsPaused();
+  EXPECT_EQ(BufferedResourceLoader::kNeverDefer, defer_strategy());
+
+  Stop();
+}
+
 }  // namespace content