Detaching from the VM after calling MediaStreamManager::SendMessageToNativeLog on Android.

MediaStreamManager::SendMessageToNativeLog posts a task to the UI thread, which will attach the audio thread to the Android java VM. Unlike chrome created threads, the audio thread is owned by the OS and does not detach itself from the VM on exit, causing a crash.
Since the log message is still crucial for diagnosing audio input issues, I'm detaching the VM after SendMessageToNativeLog, instead of removing the call on Android.

BUG=365915

Review URL: https://codereview.chromium.org/294803003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271764 0039d316-1c4b-4281-b951-d872f2087c98

content/browser/renderer_host/media/audio_input_sync_writer.cc

@@ -6,6 +6,9 @@
 
 #include <algorithm>
 
+#if defined(OS_ANDROID)
+#include "base/android/jni_android.h"
+#endif
 #include "base/memory/shared_memory.h"
 #include "content/browser/renderer_host/media/media_stream_manager.h"
 
@@ -50,9 +53,19 @@ uint32 AudioInputSyncWriter::Write(const void* data,
          << interval.InMilliseconds() << "ms.";
     }
   }
-  if (!oss.str().empty())
+  if (!oss.str().empty()) {
     MediaStreamManager::SendMessageToNativeLog(oss.str());
 
+    // MediaStreamManager::SendMessageToNativeLog posts a task to the UI thread,
+    // which will attach the audio thread to the Android java VM. Unlike chrome
+    // created threads, the audio thread is owned by the OS and does not detach
+    // itself from the VM on exit, causing a crash (crbug/365915). So we detach
+    // here to make sure the thread exits clean.
+#if defined(OS_ANDROID)
+    base::android::DetachFromVM();
+#endif
+  }
+
   last_write_time_ = base::Time::Now();
 
   uint8* ptr = static_cast<uint8*>(shared_memory_->memory());