Microphone in use indicator does not appear in Chrome VR |
||||||
Issue descriptionChrome Version: 62.0.3193.3 VrCore: 1.8.163477258 Device: Pixel XL What steps will reproduce the problem? (1) go to https://permission.site/ in 2D mode (2) select Microphone and grant permission for Chrome to access microphone (3) insert device into Daydream headset, to start Chrome VR What is the expected result? a microphone in use indicator msg should be displayed in Chrome VR What happens instead? There is no microphone in use indicator. See screenshot.
,
Aug 24 2017
This is a privacy issue, P1. Regression from M61, this was working previously.
,
Aug 25 2017
,
Aug 25 2017
This is happened after this https://chromium-review.googlesource.com/c/chromium/src/602470 This CL removed VrShell::UpdateVSyncInterval and as a result VrShell::PollMediaAccessFlag() is not called anymore. The fix is on its way.
,
Aug 28 2017
,
Aug 28 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/8f7814af5db67a488c7a59ca1985c99c9ad33d7b commit 8f7814af5db67a488c7a59ca1985c99c9ad33d7b Author: Amirhossein Simjour <asimjour@chromium.org> Date: Mon Aug 28 17:44:53 2017 VR: Poll media access indicators After removal of UpdateVSyncInterval from vr_shell, we missed the call to poll the media access indicators. This CL bring that method back by calling it when vr_shell::OnResume is called. BUG= 758610 , 758613 Change-Id: Ief5b1ff38f8b185f7ec7cd005283f07424dab878 Reviewed-on: https://chromium-review.googlesource.com/638135 Reviewed-by: Michael Thiessen <mthiesse@chromium.org> Commit-Queue: Amirhossein Simjour <asimjour@chromium.org> Cr-Commit-Position: refs/heads/master@{#497792} [modify] https://crrev.com/8f7814af5db67a488c7a59ca1985c99c9ad33d7b/chrome/browser/android/vr_shell/vr_shell.cc
,
Aug 28 2017
,
Sep 5 2017
Verified in 62.0.3199.4 |
||||||
►
Sign in to add a comment |
||||||
Comment 1 by dbbrooks@chromium.org
, Aug 24 2017