Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(144)

Unified Diff: chrome/browser/android/vr_shell/ui_scene_manager.cc

Issue 2903673002: VR: Update UiSceneManager with screen capturing flag (Closed)
Patch Set: UI webvr unittest is added Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/android/vr_shell/ui_scene_manager.cc
diff --git a/chrome/browser/android/vr_shell/ui_scene_manager.cc b/chrome/browser/android/vr_shell/ui_scene_manager.cc
index e8a62ad89d2e8117d898443d11a75404e96663fe..8d6660eb90870ba1d7525d95f2f498c282f7e60d 100644
--- a/chrome/browser/android/vr_shell/ui_scene_manager.cc
+++ b/chrome/browser/android/vr_shell/ui_scene_manager.cc
@@ -395,15 +395,15 @@ void UiSceneManager::ConfigureBackgroundColor(vr::Colorf center_color,
}
void UiSceneManager::SetAudioCapturingIndicator(bool enabled) {
- audio_capture_indicator_->set_visible(enabled);
+ audio_capture_indicator_->set_visible(enabled && !web_vr_mode_);
}
void UiSceneManager::SetVideoCapturingIndicator(bool enabled) {
- video_capture_indicator_->set_visible(enabled);
+ video_capture_indicator_->set_visible(enabled && !web_vr_mode_);
}
void UiSceneManager::SetScreenCapturingIndicator(bool enabled) {
- // TODO(asimjour) add the indicator and change the visibility here.
+ screen_capture_indicator_->set_visible(enabled && !web_vr_mode_);
}
void UiSceneManager::SetWebVrSecureOrigin(bool secure) {

Powered by Google App Engine
This is Rietveld 408576698