--- a/dom/media/CubebUtils.cpp
+++ b/dom/media/CubebUtils.cpp
@@ -218,27 +218,37 @@ uint32_t PreferredSampleRate()
     return 44100;
   }
   MOZ_ASSERT(sPreferredSampleRate);
   return sPreferredSampleRate;
 }
 
 bool InitPreferredChannelLayout()
 {
-  StaticMutexAutoLock lock(sMutex);
-  if (sPreferredChannelLayout != 0) {
-    return true;
+  {
+    StaticMutexAutoLock lock(sMutex);
+    if (sPreferredChannelLayout != 0) {
+      return true;
+    }
   }
-  cubeb* context = GetCubebContextUnlocked();
+
+  cubeb* context = GetCubebContext();
   if (!context) {
     return false;
   }
-  return cubeb_get_preferred_channel_layout(context,
-                                            &sPreferredChannelLayout) == CUBEB_OK
-         ? true : false;
+
+  // Favor calling cubeb api with the mutex unlock, potential deadlock.
+  cubeb_channel_layout layout;
+  if (cubeb_get_preferred_channel_layout(context, &layout) != CUBEB_OK) {
+    return false;
+  }
+
+  StaticMutexAutoLock lock(sMutex);
+  sPreferredChannelLayout = layout;
+  return true;
 }
 
 uint32_t PreferredChannelMap(uint32_t aChannels)
 {
   // The first element of the following mapping table is channel counts,
   // and the second one is its bit mask. It will be used in many times,
   // so we shoule avoid to allocate it in stack, or it will be created
   // and removed repeatedly. Use static to allocate this local variable