Make default media::Renderer::playback_rate = 0.

media/base/renderer.h

 // Copyright 2014 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.
 
 #ifndef MEDIA_BASE_RENDERER_H_
 #define MEDIA_BASE_RENDERER_H_
 
 #include "base/callback.h"
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
@@ skipping 31 matching lines @@
                       const CdmAttachedCB& cdm_attached_cb) = 0;
 
   // The following functions must be called after Initialize().
 
   // Discards any buffered data, executing |flush_cb| when completed.
   virtual void Flush(const base::Closure& flush_cb) = 0;
 
   // Starts rendering from |time|.
   virtual void StartPlayingFrom(base::TimeDelta time) = 0;
 
-  // Updates the current playback rate. The default playback rate should be 1.

[alokp, 2016/09/02 21:52:38]

This comment is inconsistent with that used in PipelineImpl/RendererImpl:
https://cs.chromium.org/chromium/src/media/base/pipeline_impl.cc?dr=Ss&sq=pac...

Which one is correct?

[DaleCurtis, 2016/09/06 19:15:36]

Not sure, whichever one works? :) Defaulting to zero means paused = true.

[alokp, 2016/09/06 19:26:34]

Acknowledged.

+  // Updates the current playback rate. The default playback rate should be 0.
   virtual void SetPlaybackRate(double playback_rate) = 0;
 
   // Sets the output volume. The default volume should be 1.
   virtual void SetVolume(float volume) = 0;
 
   // Returns the current media time.
   virtual base::TimeDelta GetMediaTime() = 0;
 
   // Returns whether |this| renders audio.
   virtual bool HasAudio() = 0;
 
   // Returns whether |this| renders video.
   virtual bool HasVideo() = 0;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(Renderer);
 };
 
 }  // namespace media
 
 #endif  // MEDIA_BASE_RENDERER_H_