Bug 1530190. Change description of DecodePool::mMutex to be more accurate. r=aosmond
authorTimothy Nikkel <tnikkel@gmail.com>
Mon, 22 Apr 2019 17:30:40 -0500
changeset 474733 bb340bd557d44a059953eb8b5f5e48ab3761a545
parent 474732 71bf2b9ca6ea2cbd97f4a07242e2ad7f20a1a2e5
child 474734 875d43d35a71e530b84388ad55d515015c6d3a68
push id113169
push usertnikkel@gmail.com
push dateTue, 21 May 2019 21:38:13 +0000
treeherdermozilla-inbound@17018058233c [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersaosmond
bugs1530190
milestone69.0a1
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 1530190. Change description of DecodePool::mMutex to be more accurate. r=aosmond This mutex only protects the iothread, not the rest of the decodepool. There is a different monitor for that in DecodePoolImpl. Differential Revision: https://phabricator.services.mozilla.com/D28420
image/DecodePool.cpp
--- a/image/DecodePool.cpp
+++ b/image/DecodePool.cpp
@@ -328,17 +328,17 @@ DecodePool* DecodePool::Singleton() {
   }
 
   return sSingleton;
 }
 
 /* static */
 uint32_t DecodePool::NumberOfCores() { return sNumCores; }
 
-DecodePool::DecodePool() : mMutex("image::DecodePool") {
+DecodePool::DecodePool() : mMutex("image::IOThread") {
   // Determine the number of threads we want.
   int32_t prefLimit = gfxPrefs::ImageMTDecodingLimit();
   uint32_t limit;
   if (prefLimit <= 0) {
     int32_t numCores = NumberOfCores();
     if (numCores <= 1) {
       limit = 1;
     } else if (numCores == 2) {