Remove use of PipelineStatus from BufferedDataSource and FileDataSource.

media/tools/player_wtl/movie.cc

 // Copyright (c) 2012 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 "media/tools/player_wtl/movie.h"
 
 #include "base/bind.h"
 #include "base/memory/singleton.h"
 #include "base/threading/platform_thread.h"
 #include "base/utf_string_conversions.h"
@@ skipping 48 matching lines @@
 
   message_loop_factory_.reset(new media::MessageLoopFactory());
 
   MessageLoop* pipeline_loop =
       message_loop_factory_->GetMessageLoop("PipelineThread");
   pipeline_ = new Pipeline(pipeline_loop, new media::MediaLog());
 
   // Open the file.
   std::string url_utf8 = WideToUTF8(string16(url));
   scoped_refptr<FileDataSource> data_source = new FileDataSource();
-  if (data_source->Initialize(url_utf8) != PIPELINE_OK) {
+  if (!data_source->Initialize(url_utf8)) {

[Ami GONE FROM CHROMIUM, 2012/07/23 21:11:05]

just return it

[scherkus (not reviewing), 2012/07/23 21:20:18]

?

don't we want to keep executing if true?

[Ami GONE FROM CHROMIUM, 2012/07/23 21:30:59]

haha just kidding

     return false;
   }
 
   // Create filter collection.
   scoped_ptr<FilterCollection> collection(new FilterCollection());
   collection->SetDemuxer(new FFmpegDemuxer(pipeline_loop, data_source));
   collection->AddAudioDecoder(new FFmpegAudioDecoder(
       base::Bind(&MessageLoopFactory::GetMessageLoop,
                  base::Unretained(message_loop_factory_.get()),
                  "AudioDecoderThread")));
@@ skipping 99 matching lines @@
 void Movie::Close() {
   if (pipeline_) {
     pipeline_->Stop(base::Closure());
     pipeline_ = NULL;
   }
 
   message_loop_factory_.reset();
 }
 
 }  // namespace media