Bug 1323100 - Use NS_NewNamedThread for the Android Audio thread. r?froydnj draft
authorMarkus Stange <mstange@themasta.com>
Wed, 21 Dec 2016 11:31:15 +0100
changeset 456870 77cbebe3a799f0c107622268fe12ade97460c4e9
parent 456869 7a0065d2d5f9d1bb0a93540895741f218dbc4c07
child 456871 a0e25a9f0ce024e53bc839491ab266800596d800
push id40636
push userbmo:mstange@themasta.com
push dateFri, 06 Jan 2017 12:38:48 +0000
reviewersfroydnj
bugs1323100
milestone53.0a1
Bug 1323100 - Use NS_NewNamedThread for the Android Audio thread. r?froydnj MozReview-Commit-ID: IcKsOZCZVwB
dom/plugins/base/android/ANPAudio.cpp
--- a/dom/plugins/base/android/ANPAudio.cpp
+++ b/dom/plugins/base/android/ANPAudio.cpp
@@ -117,18 +117,16 @@ public:
   }
 
   ANPAudioTrack* mTrack;
 };
 
 NS_IMETHODIMP
 AudioRunnable::Run()
 {
-  PR_SetCurrentThreadName("Android Audio");
-
   JNIEnv* const jenv = mozilla::jni::GetEnvForThread();
 
   mozilla::AutoLocalJNIFrame autoFrame(jenv, 2);
 
   jbyteArray bytearray = jenv->NewByteArray(mTrack->bufferSize);
   if (!bytearray) {
     LOG("AudioRunnable:: Run.  Could not create bytearray");
     return NS_ERROR_FAILURE;
@@ -316,17 +314,17 @@ anp_audio_start(ANPAudioTrack* s)
 
   s->isStopped = false;
   s->keepGoing = true;
 
   // AudioRunnable now owns the ANPAudioTrack
   RefPtr<AudioRunnable> runnable = new AudioRunnable(s);
 
   nsCOMPtr<nsIThread> thread;
-  NS_NewThread(getter_AddRefs(thread), runnable);
+  NS_NewNamedThread("Android Audio", getter_AddRefs(thread), runnable);
 }
 
 void
 anp_audio_pause(ANPAudioTrack* s)
 {
   if (s == nullptr || s->output_unit == nullptr) {
     return;
   }