Bug 1273405. Part 1 - move some functions to private. r=jya. draft 1273405_VorbisDataDecoder_Flush
authorJW Wang <jwwang@mozilla.com>
Tue, 17 May 2016 16:03:31 +0800
branch1273405_VorbisDataDecoder_Flush
changeset 367721 157cb461c28ff065bae10ae7a6ab11349e68ff38
parent 367683 437d2f8ef153bf0f8f362add63b7e215bc98f724
child 367722 d0d230e622670b956213e7ae88b008915629d569
push id18338
push userjwwang@mozilla.com
push dateTue, 17 May 2016 09:49:04 +0000
reviewersjya
bugs1273405
milestone49.0a1
Bug 1273405. Part 1 - move some functions to private. r=jya. MozReview-Commit-ID: CXoBsYyT55r
dom/media/platforms/agnostic/VorbisDecoder.h
--- a/dom/media/platforms/agnostic/VorbisDecoder.h
+++ b/dom/media/platforms/agnostic/VorbisDecoder.h
@@ -19,33 +19,32 @@
 namespace mozilla {
 
 class VorbisDataDecoder : public MediaDataDecoder
 {
 public:
   VorbisDataDecoder(const AudioInfo& aConfig,
                 FlushableTaskQueue* aTaskQueue,
                 MediaDataDecoderCallback* aCallback);
+  // Return true if mimetype is Vorbis
+  static bool IsVorbis(const nsACString& aMimeType);
+  static const AudioConfig::Channel* VorbisLayout(uint32_t aChannels);
+
+private:
   ~VorbisDataDecoder();
-
   RefPtr<InitPromise> Init() override;
   nsresult Input(MediaRawData* aSample) override;
   nsresult Flush() override;
   nsresult Drain() override;
   nsresult Shutdown() override;
   const char* GetDescriptionName() const override
   {
     return "vorbis audio decoder";
   }
 
-  // Return true if mimetype is Vorbis
-  static bool IsVorbis(const nsACString& aMimeType);
-  static const AudioConfig::Channel* VorbisLayout(uint32_t aChannels);
-
-private:
   nsresult DecodeHeader(const unsigned char* aData, size_t aLength);
 
   void Decode (MediaRawData* aSample);
   int DoDecode (MediaRawData* aSample);
   void DoDrain ();
 
   const AudioInfo& mInfo;
   RefPtr<FlushableTaskQueue> mTaskQueue;