VR: Update UiSceneManager with audio/video capturing flags

chrome/browser/android/vr_shell/vr_shell.cc

Index: chrome/browser/android/vr_shell/vr_shell.cc
diff --git a/chrome/browser/android/vr_shell/vr_shell.cc b/chrome/browser/android/vr_shell/vr_shell.cc
index c32e477f7b76dcd5eecf5d42f5a2828e2f32c90f..dd734df55aa8ce787166bb3a01c507c88135ae1f 100644
--- a/chrome/browser/android/vr_shell/vr_shell.cc
+++ b/chrome/browser/android/vr_shell/vr_shell.cc
@@ -551,23 +551,28 @@ void VrShell::PollMediaAccessFlag() {
           ->GetMediaStreamCaptureIndicator();
   bool is_capturing_audio = indicator->IsCapturingAudio(web_contents_);
   if (is_capturing_audio != is_capturing_audio_)
-    PostToGlThreadWhenReady(base::Bind(&VrShellGl::SetAudioCapturingWarning,
-                                       gl_thread_->GetVrShellGl(),
-                                       is_capturing_audio));
+    gl_thread_->task_runner()->PostTask(

[cjgrant, 2017/05/16 19:09:27]

These calls should go through the ui_ interface.  See other similar calls.  We
should only be using this direct approach for things that are directly dependent
on the GL implementation.  Note that you'll have to add methods to the interface
as well.

The point of all this is to keep this module cleaner, hide the cross-thread
implementation, and make pieces more testable.

[asimjour1, 2017/05/16 20:13:55]

Done.

+        FROM_HERE,
+        base::Bind(&UiSceneManager::SetAudioCapturingIndicator,
+                   gl_thread_->GetSceneManager(), is_capturing_audio));
+
   is_capturing_audio_ = is_capturing_audio;
 
   bool is_capturing_video = indicator->IsCapturingVideo(web_contents_);
   if (is_capturing_video != is_capturing_video_)
-    PostToGlThreadWhenReady(base::Bind(&VrShellGl::SetVideoCapturingWarning,
-                                       gl_thread_->GetVrShellGl(),
-                                       is_capturing_video));
+    gl_thread_->task_runner()->PostTask(
+        FROM_HERE,
+        base::Bind(&UiSceneManager::SetVideoCapturingIndicator,
+                   gl_thread_->GetSceneManager(), is_capturing_video));
   is_capturing_video_ = is_capturing_video;
 
   bool is_capturing_screen = indicator->IsBeingMirrored(web_contents_);
   if (is_capturing_screen != is_capturing_screen_)
-    PostToGlThreadWhenReady(base::Bind(&VrShellGl::SetScreenCapturingWarning,
-                                       gl_thread_->GetVrShellGl(),
-                                       is_capturing_screen));
+    gl_thread_->task_runner()->PostTask(
+        FROM_HERE,
+        base::Bind(&UiSceneManager::SetScreenCapturingIndicator,
+                   gl_thread_->GetSceneManager(), is_capturing_screen));
+
   is_capturing_screen_ = is_capturing_screen;
 }