Bug 1346498 part 2 - remove unused MediaDecoder::HasSuspendTaint(); r?jwwang draft
authorKaku Kuo <kaku@mozilla.com>
Fri, 10 Mar 2017 16:34:12 +0800
changeset 497022 1ad48347cea4c05fd626f53bb5ded3c991745473
parent 497021 24c72019f5f1f23841cd2e7d48aabbdd1f0d3bc1
child 497023 846c02f50409bab34b5f3a943f6414bffb67f45c
child 497029 7d9904586be2b2edefb65c29749e926059ddd397
push id48784
push userbmo:kaku@mozilla.com
push dateSat, 11 Mar 2017 08:14:59 +0000
reviewersjwwang
bugs1346498
milestone55.0a1
Bug 1346498 part 2 - remove unused MediaDecoder::HasSuspendTaint(); r?jwwang MozReview-Commit-ID: 9AUDVpfSyfL
dom/media/MediaDecoder.cpp
dom/media/MediaDecoder.h
--- a/dom/media/MediaDecoder.cpp
+++ b/dom/media/MediaDecoder.cpp
@@ -1323,23 +1323,16 @@ MediaDecoder::SetForcedHidden(bool aForc
 
 void
 MediaDecoder::SetSuspendTaint(bool aTainted)
 {
   MOZ_ASSERT(NS_IsMainThread());
   mHasSuspendTaint = aTainted;
 }
 
-bool
-MediaDecoder::HasSuspendTaint() const
-{
-  MOZ_ASSERT(NS_IsMainThread());
-  return mHasSuspendTaint;
-}
-
 void
 MediaDecoder::UpdateEstimatedMediaDuration(int64_t aDuration)
 {
   MOZ_ASSERT(NS_IsMainThread());
 
   if (mPlayState <= PLAY_STATE_LOADING) {
     return;
   }
--- a/dom/media/MediaDecoder.h
+++ b/dom/media/MediaDecoder.h
@@ -375,19 +375,16 @@ private:
 
   // Force override the visible state to hidden.
   // Called from HTMLMediaElement when testing of video decode suspend from mochitests.
   void SetForcedHidden(bool aForcedHidden);
 
   // Mark the decoder as tainted, meaning suspend-video-decoder is disabled.
   void SetSuspendTaint(bool aTaint);
 
-  // Returns true if the decoder can't participate in suspend-video-decoder.
-  bool HasSuspendTaint() const;
-
   /******
    * The following methods must only be called on the main
    * thread.
    ******/
 
   // Change to a new play state. This updates the mState variable and
   // notifies any thread blocking on this object's monitor of the
   // change. Call on the main thread only.