Remove VideoDecoder::natural_size() & added VideoFrame::natural_size().

media/base/video_decoder.h

 // 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.
 
 #ifndef MEDIA_BASE_VIDEO_DECODER_H_
 #define MEDIA_BASE_VIDEO_DECODER_H_
 
 #include "base/callback.h"
 #include "base/memory/ref_counted.h"
 #include "media/base/pipeline_status.h"
@@ skipping 42 matching lines @@
 
   // Reset decoder state, fulfilling all pending ReadCB and dropping extra
   // queued decoded data. After this call, the decoder is back to an initialized
   // clean state.
   virtual void Reset(const base::Closure& closure) = 0;
 
   // Stop decoder and set it to an uninitialized state. Note that a VideoDecoder
   // should/could not be re-initialized after it has been stopped.
   virtual void Stop(const base::Closure& closure) = 0;
 
-  // Returns the natural width and height of decoded video in pixels.
-  //
-  // Clients should NOT rely on these values to remain constant. Instead, use
-  // the width/height from decoded video frames themselves.
-  //
-  // TODO(scherkus): why not rely on prerolling and decoding a single frame to
-  // get dimensions?
-  virtual const gfx::Size& natural_size() = 0;
-
   // Returns true if the output format has an alpha channel. Most formats do not
   // have alpha so the default is false. Override and return true for decoders
   // that return formats with an alpha channel.
   virtual bool HasAlpha() const;
 
   // Prepare decoder for shutdown.  This is a HACK needed because
   // PipelineImpl::Stop() goes through a Pause/Flush/Stop dance to all its
   // filters, waiting for each state transition to complete before starting the
   // next, but WebMediaPlayerImpl::Destroy() holds the renderer loop hostage for
   // the duration.  Default implementation does nothing; derived decoders may
   // override as needed.  http://crbug.com/110228 tracks removing this.
   virtual void PrepareForShutdownHack();
 
  protected:
   friend class base::RefCountedThreadSafe<VideoDecoder>;
   virtual ~VideoDecoder();
   VideoDecoder();
 
   DISALLOW_COPY_AND_ASSIGN(VideoDecoder);
 };
 
 }  // namespace media
 
 #endif  // MEDIA_BASE_VIDEO_DECODER_H_