Made MediaFilter::host_ and MediaFilter::message_loop_ private.

media/base/filters.h

 // Copyright (c) 2006-2009 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.
 
 // Filters are connected in a strongly typed manner, with downstream filters
 // always reading data from upstream filters.  Upstream filters have no clue
 // who is actually reading from them, and return the results via callbacks.
 //
 //                         DemuxerStream(Video) <- VideoDecoder <- VideoRenderer
 // DataSource <- Demuxer <
@@ skipping 40 matching lines @@
   FILTER_VIDEO_DECODER,
   FILTER_AUDIO_RENDERER,
   FILTER_VIDEO_RENDERER
 };
 
 
 class MediaFilter : public base::RefCountedThreadSafe<MediaFilter> {
  public:
   MediaFilter() : host_(NULL), message_loop_(NULL) {}
 
-  // Sets the protected member |host_|. This is the first method called by
+  // Sets the private member |host_|. This is the first method called by
   // the FilterHost after a filter is created.  The host holds a strong
   // reference to the filter.  The reference held by the host is guaranteed
   // to be released before the host object is destroyed by the pipeline.
-  virtual void SetFilterHost(FilterHost* host) {
+  virtual void set_host(FilterHost* host) {
     DCHECK(host);
     DCHECK(!host_);
     host_ = host;
   }
 
-  // Sets the protected member |message_loop_|, which is used by filters for
+  virtual FilterHost* host() {
+    return host_;
+  }
+
+  // Sets the private member |message_loop_|, which is used by filters for
   // processing asynchronous tasks and maintaining synchronized access to
   // internal data members.  The message loop should be running and exceed the
   // lifetime of the filter.
-  virtual void SetMessageLoop(MessageLoop* message_loop) {
+  virtual void set_message_loop(MessageLoop* message_loop) {
     DCHECK(message_loop);
     DCHECK(!message_loop_);
     message_loop_ = message_loop;
   }
 
+  virtual MessageLoop* message_loop() {
+    return message_loop_;
+  }
+
   // The pipeline is being stopped either as a result of an error or because
   // the client called Stop().
   virtual void Stop() = 0;
 
   // The pipeline playback rate has been changed.  Filters may implement this
   // method if they need to respond to this call.
   virtual void SetPlaybackRate(float playback_rate) {}
 
   // The pipeline is seeking to the specified time.  Filters may implement
   // this method if they need to respond to this call.
   virtual void Seek(base::TimeDelta time) {}
 
  protected:
   // Only allow scoped_refptr<> to delete filters.
   friend class base::RefCountedThreadSafe<MediaFilter>;
   virtual ~MediaFilter() {}
 
-  // TODO(scherkus): make these private with public/protected accessors.
+ private:
   FilterHost* host_;
   MessageLoop* message_loop_;
 
- private:
   DISALLOW_COPY_AND_ASSIGN(MediaFilter);
 };
 
 
 class DataSource : public MediaFilter {
  public:
   static const FilterType filter_type() {
     return FILTER_DATA_SOURCE;
   }
 
@@ skipping 164 matching lines @@
   // Initializes this filter, returns true if successful, false otherwise.
   virtual bool Initialize(AudioDecoder* decoder) = 0;
 
   // Sets the output volume.
   virtual void SetVolume(float volume) = 0;
 };
 
 }  // namespace media
 
 #endif  // MEDIA_BASE_FILTERS_H_