Adding callback support to media filter Initialize() and Seek().

Also includes unit tests for AudioRendererBase and VideoRendererBase.

I had to rollback my first attempt at this change.  Original review: http://codereview.chromium.org/155469

BUG=16014,16031
TEST=media_unittests, layout tests

Review URL: http://codereview.chromium.org/155608

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

1 files changed, 4 insertions, 5 deletions

diff --git a/media/base/pipeline_impl.h b/media/base/pipeline_impl.h
index 911d253..e9f44d5 100644
--- a/media/base/pipeline_impl.h
+++ b/media/base/pipeline_impl.h
@@ -191,11 +191,6 @@ class PipelineInternal : public base::RefCountedThreadSafe<PipelineInternal> {
   // Methods called by a FilterHostImpl object.  These methods may be called
   // on any thread, either the pipeline's thread or any other.
 
-  // When a filter calls it's FilterHost, the filter host calls back to the
-  // pipeline thread.  If the pipeline thread is running a nested message loop
-  // then it will be exited.
-  void InitializationComplete(FilterHostImpl* host);
-
   // Sets the pipeline time and schedules a task to call back to any filters
   // that have registered a time update callback.
   void SetTime(base::TimeDelta time);
@@ -245,6 +240,10 @@ class PipelineInternal : public base::RefCountedThreadSafe<PipelineInternal> {
            state_ == kInitVideoRenderer;
   }
 
+  // Callback executed by filters upon completing initialization and seeking.
+  void OnFilterInitialize();
+  void OnFilterSeek();
+
   // The following "task" methods correspond to the public methods, but these
   // methods are run as the result of posting a task to the PipelineInternal's
   // message loop.