Bug 1540136 - P4: Remove ChromiumCDMParent mMainThread as it is unused. r=cpearce a=lizzard
authorBryce Van Dyk <bvandyk@mozilla.com>
Tue, 09 Apr 2019 15:07:19 +0000
changeset 526223 ecb3ee5969c19fe8c52cedb2bd0291edf11dc83c
parent 526222 385fde9e64e20795ca20d6cb7db114b0c0bcc8fe
child 526224 5710827ee9cf5011b6698590a7d31e922f636cf2
push id2032
push userffxbld-merge
push dateMon, 13 May 2019 09:36:57 +0000
treeherdermozilla-release@455c1065dcbe [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerscpearce, lizzard
bugs1540136
milestone67.0
Bug 1540136 - P4: Remove ChromiumCDMParent mMainThread as it is unused. r=cpearce a=lizzard Differential Revision: https://phabricator.services.mozilla.com/D26207
dom/media/gmp/ChromiumCDMParent.cpp
dom/media/gmp/ChromiumCDMParent.h
--- a/dom/media/gmp/ChromiumCDMParent.cpp
+++ b/dom/media/gmp/ChromiumCDMParent.cpp
@@ -57,17 +57,16 @@ RefPtr<ChromiumCDMParent::InitPromise> C
         MediaResult(NS_ERROR_FAILURE,
                     nsPrintfCString("ChromiumCDMParent::Init() failed "
                                     "nullCallback=%s nullMainThread=%s",
                                     !aCDMCallback ? "true" : "false",
                                     !aMainThread ? "true" : "false")),
         __func__);
   }
   mCDMCallback = aCDMCallback;
-  mMainThread = aMainThread;
 
   RefPtr<ChromiumCDMParent::InitPromise> promise =
       mInitPromise.Ensure(__func__);
   RefPtr<ChromiumCDMParent> self = this;
   SendInit(aAllowDistinctiveIdentifier, aAllowPersistentState)
       ->Then(
           AbstractThread::GetCurrent(), __func__,
           [self](bool aSuccess) {
--- a/dom/media/gmp/ChromiumCDMParent.h
+++ b/dom/media/gmp/ChromiumCDMParent.h
@@ -175,17 +175,14 @@ class ChromiumCDMParent final : public P
   // The H.264 decoder in Widevine CDM versions 970 and later output in decode
   // order rather than presentation order, so we reorder in presentation order
   // before presenting. mMaxRefFrames is non-zero if we have an initialized
   // decoder and we are decoding H.264. If so, it stores the maximum length of
   // the reorder queue that we need. Note we may have multiple decoders for the
   // life time of this object, but never more than one active at once.
   uint32_t mMaxRefFrames = 0;
   ReorderQueue mReorderQueue;
-
-  // The main thread associated with the root document. Must be set in Init().
-  nsCOMPtr<nsIEventTarget> mMainThread;
 };
 
 }  // namespace gmp
 }  // namespace mozilla
 
 #endif  // ChromiumCDMParent_h_