Bug 1445104 - Remove HTMLMediaElement::{mStatsShowing,mMediaSecurityVerified} as they're unused. r=jya
authorChris Pearce <cpearce@mozilla.com>
Tue, 13 Mar 2018 13:33:42 +1300
changeset 464088 e8a5c35df168c72c085ca8ea5d03ad8408e60979
parent 464087 e865836987b36817a8883cfc2bab34421daf0a83
child 464089 d9c664130c22e6d34bed79662275e90267658f48
push id1728
push userjlund@mozilla.com
push dateMon, 18 Jun 2018 21:12:27 +0000
treeherdermozilla-release@c296fde26f5f [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1445104 - Remove HTMLMediaElement::{mStatsShowing,mMediaSecurityVerified} as they're unused. r=jya MozReview-Commit-ID: LtKw4Hj3M0G
--- a/dom/html/HTMLMediaElement.cpp
+++ b/dom/html/HTMLMediaElement.cpp
@@ -4870,19 +4870,16 @@ nsresult HTMLMediaElement::InitializeDec
   return SetupDecoder(decoder.get(), aChannel, isPrivateBrowsing, aListener);
 HTMLMediaElement::FinishDecoderSetup(MediaDecoder* aDecoder)
-  // Force a same-origin check before allowing events for this media resource.
-  mMediaSecurityVerified = false;
   // Set mDecoder now so if methods like GetCurrentSrc get called between
   // here and Load(), they work.
   // Notify the decoder of the initial activity status.
   // Update decoder principal before we start decoding, since it
--- a/dom/html/HTMLMediaElement.h
+++ b/dom/html/HTMLMediaElement.h
@@ -1594,20 +1594,16 @@ protected:
   // 'mAutoplaying' flag, which indicates whether the current playback
   // is a result of the autoplay attribute.
   bool mAutoplaying = true;
   // Playback of the video is paused either due to calling the
   // 'Pause' method, or playback not yet having started.
   WakeLockBoolWrapper mPaused;
-  // True if the media statistics are currently being shown by the builtin
-  // video controls
-  bool mStatsShowing = false;
   // The following two fields are here for the private storage of the builtin
   // video controls, and control 'casting' of the video to external devices
   // (TVs, projectors etc.)
   // True if casting is currently allowed
   bool mAllowCasting = false;
   // True if currently casting this video
   bool mIsCasting = false;
@@ -1675,19 +1671,16 @@ protected:
   // due to loading a preload:none media. When true, the resource we'll
   // load when the user initiates either playback or an explicit load is
   // stored in mPreloadURI.
   bool mSuspendedForPreloadNone = false;
   // True if we've connected mSrcStream to the media element output.
   bool mSrcStreamIsPlaying = false;
-  // True if a same-origin check has been done for the media element and resource.
-  bool mMediaSecurityVerified = false;
   // True if we should set nsIClassOfService::UrgentStart to the channel to
   // get the response ASAP for better user responsiveness.
   bool mUseUrgentStartForChannel = false;
   // The CORS mode when loading the media element
   // Info about the played media.