Reverting r24002 due to test_shell_tests redness

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24004 0039d316-1c4b-4281-b951-d872f2087c98

4 files changed, 13 insertions, 49 deletions

diff --git a/webkit/glue/media/buffered_data_source.cc b/webkit/glue/media/buffered_data_source.cc
index 6726885..5f739ba 100644
--- a/webkit/glue/media/buffered_data_source.cc
+++ b/webkit/glue/media/buffered_data_source.cc
@@ -765,9 +765,6 @@ void BufferedDataSource::InitialStartCallback(int error) {
     host()->SetStreaming(true);
   }
 
-  // Currently, only files can be used reliably w/o a network.
-  host()->SetLoaded(url_.SchemeIsFile());
-
   initial_response_received_ = true;
   if (probe_response_received_)
     DoneInitialization();
diff --git a/webkit/glue/media/buffered_data_source_unittest.cc b/webkit/glue/media/buffered_data_source_unittest.cc
index 003313c..4d9efe8 100644
--- a/webkit/glue/media/buffered_data_source_unittest.cc
+++ b/webkit/glue/media/buffered_data_source_unittest.cc
@@ -30,15 +30,8 @@ using ::testing::WithArgs;
 namespace {
 
 const char* kHttpUrl = "http://test";
-const char* kFileUrl = "file://test";
 const int kDataSize = 1024;
 
-enum NetworkState {
-  NONE,
-  LOADED,
-  LOADING
-};
-
 }  // namespace
 
 namespace webkit_glue {
@@ -407,7 +400,7 @@ class BufferedDataSourceTest : public testing::Test {
     }
   }
 
-  virtual ~BufferedDataSourceTest() {
+  ~BufferedDataSourceTest() {
     if (data_source_) {
       // Release the bridge factory because we don't own it.
       // Expects bridge factory to be destroyed along with data source.
@@ -421,7 +414,7 @@ class BufferedDataSourceTest : public testing::Test {
   }
 
   void InitializeDataSource(const char* url, int error, int probe_error,
-                            int64 instance_size, NetworkState networkState) {
+                            int64 instance_size) {
     // Saves the url first.
     gurl_ = GURL(url);
 
@@ -439,14 +432,6 @@ class BufferedDataSourceTest : public testing::Test {
     loader_ = new StrictMock<MockBufferedResourceLoader>();
     probe_loader_ = new StrictMock<MockBufferedResourceLoader>();
 
-    if (networkState == LOADED) {
-      EXPECT_CALL(host_, SetLoaded(true));
-    } else if (networkState == LOADING) {
-      EXPECT_CALL(host_, SetLoaded(false));
-    }
-
-    // TODO(ajwong): This mock is too strict.  We do not need to guarantee a
-    // full sequencing each of these expectations.
     InSequence s;
     StrictMock<media::MockFilterCallback> callback;
 
@@ -463,7 +448,6 @@ class BufferedDataSourceTest : public testing::Test {
         .WillOnce(DoAll(Assign(&error_, error),
                         Invoke(this,
                                &BufferedDataSourceTest::InvokeStartCallback)));
-
     if (error == net::OK) {
       EXPECT_CALL(*loader_, instance_size())
           .WillOnce(Return(instance_size));
@@ -705,36 +689,36 @@ class BufferedDataSourceTest : public testing::Test {
 };
 
 TEST_F(BufferedDataSourceTest, InitializationSuccess) {
-  InitializeDataSource(kHttpUrl, net::OK, net::OK, 1024, LOADING);
+  InitializeDataSource(kHttpUrl, net::OK, net::OK, 1024);
   StopDataSource();
 }
 
 TEST_F(BufferedDataSourceTest, InitiailizationFailed) {
   InitializeDataSource(kHttpUrl, net::ERR_FILE_NOT_FOUND,
-                       net::ERR_FILE_NOT_FOUND, 0, NONE);
+                       net::ERR_FILE_NOT_FOUND, 0);
   StopDataSource();
 }
 
 TEST_F(BufferedDataSourceTest, MissingContentLength) {
-  InitializeDataSource(kHttpUrl, net::OK, net::OK, -1, LOADING);
+  InitializeDataSource(kHttpUrl, net::OK, net::OK, -1);
   StopDataSource();
 }
 
 TEST_F(BufferedDataSourceTest, RangeRequestNotSupported) {
   InitializeDataSource(kHttpUrl, net::OK,
-                       net::ERR_REQUEST_RANGE_NOT_SATISFIABLE, 1024, LOADING);
+                       net::ERR_REQUEST_RANGE_NOT_SATISFIABLE, 1024);
   StopDataSource();
 }
 
 TEST_F(BufferedDataSourceTest,
        MissingContentLengthAndRangeRequestNotSupported) {
   InitializeDataSource(kHttpUrl, net::OK,
-                       net::ERR_REQUEST_RANGE_NOT_SATISFIABLE, -1, LOADING);
+                       net::ERR_REQUEST_RANGE_NOT_SATISFIABLE, -1);
   StopDataSource();
 }
 
 TEST_F(BufferedDataSourceTest, ReadCacheHit) {
-  InitializeDataSource(kHttpUrl, net::OK, net::OK, 25, LOADING);
+  InitializeDataSource(kHttpUrl, net::OK, net::OK, 25);
 
   // Performs read with cache hit.
   ReadDataSourceHit(10, 10, 10);
@@ -746,27 +730,21 @@ TEST_F(BufferedDataSourceTest, ReadCacheHit) {
 }
 
 TEST_F(BufferedDataSourceTest, ReadCacheMiss) {
-  InitializeDataSource(kHttpUrl, net::OK, net::OK, 1024, LOADING);
+  InitializeDataSource(kHttpUrl, net::OK, net::OK, 1024);
   ReadDataSourceMiss(1000, 10);
   ReadDataSourceMiss(20, 10);
   StopDataSource();
 }
 
 TEST_F(BufferedDataSourceTest, ReadFailed) {
-  InitializeDataSource(kHttpUrl, net::OK, net::OK, 1024, LOADING);
+  InitializeDataSource(kHttpUrl, net::OK, net::OK, 1024);
   ReadDataSourceHit(10, 10, 10);
   ReadDataSourceFailed(10, 10, net::ERR_CONNECTION_RESET);
   StopDataSource();
 }
 
 TEST_F(BufferedDataSourceTest, ReadTimesOut) {
-  InitializeDataSource(kHttpUrl, net::OK, net::OK, 1024, LOADING);
-  ReadDataSourceTimesOut(20, 10);
-  StopDataSource();
-}
-
-TEST_F(BufferedDataSourceTest, FileHasLoadedState) {
-  InitializeDataSource(kFileUrl, net::OK, net::OK, 1024, LOADED);
+  InitializeDataSource(kHttpUrl, net::OK, net::OK, 1024);
   ReadDataSourceTimesOut(20, 10);
   StopDataSource();
 }
diff --git a/webkit/glue/media/simple_data_source.cc b/webkit/glue/media/simple_data_source.cc
index 52ea388..639aa5f 100644
--- a/webkit/glue/media/simple_data_source.cc
+++ b/webkit/glue/media/simple_data_source.cc
@@ -71,8 +71,6 @@ void SimpleDataSource::Initialize(const std::string& url,
     return;
   }
 
-  host()->SetLoaded(url_.SchemeIsFile());
-
   // Post a task to the render thread to start loading the resource.
   render_loop_->PostTask(FROM_HERE,
       NewRunnableMethod(this, &SimpleDataSource::StartTask));
diff --git a/webkit/glue/webmediaplayer_impl.cc b/webkit/glue/webmediaplayer_impl.cc
index 0438e54..d3d3679 100644
--- a/webkit/glue/webmediaplayer_impl.cc
+++ b/webkit/glue/webmediaplayer_impl.cc
@@ -272,7 +272,6 @@ void WebMediaPlayerImpl::seek(float seconds) {
 
   // Try to preserve as much accuracy as possible.
   float microseconds = seconds * base::Time::kMicrosecondsPerSecond;
-  SetReadyState(WebKit::WebMediaPlayer::HaveMetadata);
   pipeline_->Seek(
       base::TimeDelta::FromMicroseconds(static_cast<int64>(microseconds)),
       NewCallback(proxy_.get(),
@@ -362,10 +361,7 @@ bool WebMediaPlayerImpl::paused() const {
 bool WebMediaPlayerImpl::seeking() const {
   DCHECK(MessageLoop::current() == main_loop_);
 
-  if (ready_state_ == WebKit::WebMediaPlayer::HaveNothing)
-    return false;
-  
-  return ready_state_ == WebKit::WebMediaPlayer::HaveMetadata;
+  return false;
 }
 
 float WebMediaPlayerImpl::duration() const {
@@ -471,11 +467,7 @@ void WebMediaPlayerImpl::OnPipelineInitialize() {
     // TODO(hclam): change this to report the correct status.
     SetReadyState(WebKit::WebMediaPlayer::HaveMetadata);
     SetReadyState(WebKit::WebMediaPlayer::HaveEnoughData);
-    if (pipeline_->IsLoaded()) {
-      SetNetworkState(WebKit::WebMediaPlayer::Loaded);
-    } else {
-      SetNetworkState(WebKit::WebMediaPlayer::Loading);
-    }
+    SetNetworkState(WebKit::WebMediaPlayer::Loaded);
   } else {
     // TODO(hclam): should use pipeline_->GetError() to determine the state
     // properly and reports error using MediaError.
@@ -492,7 +484,6 @@ void WebMediaPlayerImpl::OnPipelineInitialize() {
 void WebMediaPlayerImpl::OnPipelineSeek() {
   DCHECK(MessageLoop::current() == main_loop_);
   if (pipeline_->GetError() == media::PIPELINE_OK) {
-    SetReadyState(WebKit::WebMediaPlayer::HaveEnoughData);
     GetClient()->timeChanged();
   }
 }