Issue metadata
Sign in to add a comment
|
Mic not working using flash plugin
Reported by
chris.ma...@glynnk12.org,
Dec 12
|
||||||||||||||||||||||
Issue descriptionUserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/71.0.3578.80 Safari/537.36 Example URL: https://www.onlinemictest.com/microphone-test-in-adobe-flash/ Steps to reproduce the problem: 1. open https://www.onlinemictest.com/microphone-test-in-adobe-flash/ 2. accept flash plugin 3. talk in mic What is the expected behavior? would show activity on page What went wrong? Mic is showing as blocked and showing any activity. Did this work before? Yes 70.0.3538.110 Is it a problem with Flash or HTML5? Flash Does this work in other browsers? Yes Chrome version: 71.0.3578.80 Channel: stable OS Version: 10.0 Flash Version: Shockwave Flash 32.0 r0 Contents of chrome://gpu: Graphics Feature Status Canvas: Hardware accelerated Flash: Hardware accelerated Flash Stage3D: Hardware accelerated Flash Stage3D Baseline profile: Hardware accelerated Compositing: Hardware accelerated Multiple Raster Threads: Enabled Native GpuMemoryBuffers: Software only. Hardware acceleration disabled Out-of-process Rasterization: Disabled Hardware Protected Video Decode: Unavailable Rasterization: Hardware accelerated Skia Deferred Display List: Disabled Skia Renderer: Disabled Surface Control: Disabled Surface Synchronization: Enabled Video Decode: Hardware accelerated Viz Service Display Compositor: Disabled WebGL: Hardware accelerated WebGL2: Hardware accelerated Driver Bug Workarounds clear_uniforms_before_first_program_use decode_encode_srgb_for_generatemipmap disable_discard_framebuffer disable_dxgi_zero_copy_video disable_framebuffer_cmaa exit_on_context_lost force_cube_complete scalarize_vec_and_mat_constructor_args disabled_extension_GL_KHR_blend_equation_advanced disabled_extension_GL_KHR_blend_equation_advanced_coherent Problems Detected Protected video decoding with swap chain is for Windows and Intel only Disabled Features: protected_video_decode Some drivers are unable to reset the D3D device in the GPU process sandbox Applied Workarounds: exit_on_context_lost Clear uniforms before first program use on all platforms: 124764, 349137 Applied Workarounds: clear_uniforms_before_first_program_use Always rewrite vec/mat constructors to be consistent: 398694 Applied Workarounds: scalarize_vec_and_mat_constructor_args ANGLE crash on glReadPixels from incomplete cube map texture: 518889 Applied Workarounds: force_cube_complete Framebuffer discarding can hurt performance on non-tilers: 570897 Applied Workarounds: disable_discard_framebuffer Use GL_INTEL_framebuffer_CMAA on ChromeOS: 535198 Applied Workarounds: disable_framebuffer_cmaa Disable KHR_blend_equation_advanced until cc shaders are updated: 661715 Applied Workarounds: disable(GL_KHR_blend_equation_advanced), disable(GL_KHR_blend_equation_advanced_coherent) Decode and Encode before generateMipmap for srgb format textures on Windows: 634519 Applied Workarounds: decode_encode_srgb_for_generatemipmap Zero-copy DXGI video hangs or displays incorrect colors on AMD drivers: 623029 Applied Workarounds: disable_dxgi_zero_copy_video Native GpuMemoryBuffers have been disabled, either via about:flags or command line. Disabled Features: native_gpu_memory_buffers Viz service display compositor is not enabled by default. Disabled Features: viz_display_compositor Skia renderer is not used by default. Disabled Features: skia_renderer Skia deferred display list is not used by default. Disabled Features: skia_deferred_display_list Version Information Data exported 2018-12-12T16:39:49.375Z Chrome version Chrome/71.0.3578.80 Operating system Windows NT 10.0.17134 Software rendering list URL https://chromium.googlesource.com/chromium/src/+/2ac50e7249fbd55e6f517a28131605c9fb9fe897/gpu/config/software_rendering_list.json Driver bug list URL https://chromium.googlesource.com/chromium/src/+/2ac50e7249fbd55e6f517a28131605c9fb9fe897/gpu/config/gpu_driver_bug_list.json ANGLE commit id f2ed299569c0 2D graphics backend Skia/71 7469a9341afab19271b8ef07af5c16a0f2c4ccc1- Command Line "C:\Program Files (x86)\Google\Chrome\Application\chrome.exe" --flag-switches-begin --flag-switches-end Driver Information Initialization time 699 In-process GPU false Passthrough Command Decoder false Sandboxed true GPU0 VENDOR = 0x1002, DEVICE= 0x6611 *ACTIVE* GPU1 VENDOR = 0x17e9, DEVICE= 0x037a GPU2 VENDOR = 0x17e9, DEVICE= 0x037a Optimus false AMD switchable false Desktop compositing Aero Glass Direct Composition true Supports overlays false Overlay capabilities Diagonal Monitor Size of \\.\DISPLAY1 22.9" Diagonal Monitor Size of \\.\DISPLAY3 22.9" Diagonal Monitor Size of \\.\DISPLAY4 22.9" Driver D3D12 feature level Not supported Driver Vulkan API version Not supported Driver vendor Advanced Micro Devices, Inc. Driver version 22.19.128.0 Driver date 8-31-2017 GPU CUDA compute capability major version 0 Pixel shader version 5.0 Vertex shader version 5.0 Max. MSAA samples 8 Machine model name Machine model version GL_VENDOR Google Inc. GL_RENDERER ANGLE (AMD Radeon(TM) R5 240 Direct3D11 vs_5_0 ps_5_0) GL_VERSION OpenGL ES 2.0 (ANGLE 2.1.0.f2ed299569c0) GL_EXTENSIONS GL_ANGLE_client_arrays GL_ANGLE_depth_texture GL_ANGLE_explicit_context GL_ANGLE_explicit_context_gles1 GL_ANGLE_framebuffer_blit GL_ANGLE_framebuffer_multisample GL_ANGLE_instanced_arrays GL_ANGLE_lossy_etc_decode GL_ANGLE_multiview_multisample GL_ANGLE_pack_reverse_row_order GL_ANGLE_program_cache_control GL_ANGLE_request_extension GL_ANGLE_robust_client_memory GL_ANGLE_texture_compression_dxt3 GL_ANGLE_texture_compression_dxt5 GL_ANGLE_texture_usage GL_ANGLE_translated_shader_source GL_CHROMIUM_bind_generates_resource GL_CHROMIUM_bind_uniform_location GL_CHROMIUM_color_buffer_float_rgb GL_CHROMIUM_color_buffer_float_rgba GL_CHROMIUM_copy_compressed_texture GL_CHROMIUM_copy_texture GL_CHROMIUM_sync_query GL_EXT_blend_minmax GL_EXT_color_buffer_half_float GL_EXT_debug_marker GL_EXT_discard_framebuffer GL_EXT_disjoint_timer_query GL_EXT_draw_buffers GL_EXT_frag_depth GL_EXT_map_buffer_range GL_EXT_occlusion_query_boolean GL_EXT_read_format_bgra GL_EXT_robustness GL_EXT_sRGB GL_EXT_shader_texture_lod GL_EXT_texture_compression_bptc GL_EXT_texture_compression_dxt1 GL_EXT_texture_compression_s3tc_srgb GL_EXT_texture_filter_anisotropic GL_EXT_texture_format_BGRA8888 GL_EXT_texture_rg GL_EXT_texture_storage GL_EXT_unpack_subimage GL_KHR_debug GL_KHR_parallel_shader_compile GL_NV_EGL_stream_consumer_external GL_NV_fence GL_NV_pack_subimage GL_NV_pixel_buffer_object GL_OES_EGL_image GL_OES_EGL_image_external GL_OES_compressed_ETC1_RGB8_texture GL_OES_depth32 GL_OES_element_index_uint GL_OES_get_program_binary GL_OES_mapbuffer GL_OES_packed_depth_stencil GL_OES_rgb8_rgba8 GL_OES_standard_derivatives GL_OES_surfaceless_context GL_OES_texture_float GL_OES_texture_float_linear GL_OES_texture_half_float GL_OES_texture_half_float_linear GL_OES_texture_npot GL_OES_vertex_array_object OES_compressed_EAC_R11_signed_texture OES_compressed_EAC_R11_unsigned_texture OES_compressed_EAC_RG11_signed_texture OES_compressed_EAC_RG11_unsigned_texture OES_compressed_ETC2_RGB8_texture OES_compressed_ETC2_RGBA8_texture OES_compressed_ETC2_punchthroughA_RGBA8_texture OES_compressed_ETC2_punchthroughA_sRGB8_alpha_texture OES_compressed_ETC2_sRGB8_alpha8_texture OES_compressed_ETC2_sRGB8_texture Disabled Extensions GL_KHR_blend_equation_advanced GL_KHR_blend_equation_advanced_coherent Disabled WebGL Extensions Window system binding vendor Google Inc. (adapter LUID: 000000000000704e) Window system binding version 1.4 (ANGLE 2.1.0.f2ed299569c0) Window system binding extensions EGL_EXT_create_context_robustness EGL_ANGLE_d3d_share_handle_client_buffer EGL_ANGLE_d3d_texture_client_buffer EGL_ANGLE_surface_d3d_texture_2d_share_handle EGL_ANGLE_query_surface_pointer EGL_ANGLE_window_fixed_size EGL_ANGLE_keyed_mutex EGL_ANGLE_surface_orientation EGL_ANGLE_direct_composition EGL_NV_post_sub_buffer EGL_KHR_create_context EGL_EXT_device_query EGL_KHR_image EGL_KHR_image_base EGL_KHR_gl_texture_2D_image EGL_KHR_gl_texture_cubemap_image EGL_KHR_gl_renderbuffer_image EGL_KHR_get_all_proc_addresses EGL_KHR_stream EGL_KHR_stream_consumer_gltexture EGL_NV_stream_consumer_gltexture_yuv EGL_ANGLE_flexible_surface_compatibility EGL_ANGLE_stream_producer_d3d_texture EGL_ANGLE_create_context_webgl_compatibility EGL_CHROMIUM_create_context_bind_generates_resource EGL_CHROMIUM_sync_control EGL_EXT_pixel_format_float EGL_KHR_surfaceless_context EGL_ANGLE_display_texture_share_group EGL_ANGLE_create_context_client_arrays EGL_ANGLE_program_cache_control EGL_ANGLE_robust_resource_initialization EGL_ANGLE_create_context_extensions_enabled EGL_ANDROID_blob_cache Direct rendering Yes Reset notification strategy 0x8252 GPU process crash count 0 Compositor Information Tile Update Mode One-copy Partial Raster Enabled GpuMemoryBuffers Status R_8 Software only R_16 Software only RG_88 Software only BGR_565 Software only RGBA_4444 Software only RGBX_8888 GPU_READ, SCANOUT RGBA_8888 GPU_READ, SCANOUT BGRX_8888 Software only BGRX_1010102 Software only RGBX_1010102 Software only BGRA_8888 Software only RGBA_F16 Software only YVU_420 Software only YUV_420_BIPLANAR Software only UYVY_422 Software only Display(s) Information Info Display[2528732444] bounds=[0,0 1920x1080], workarea=[0,0 1920x1040], scale=1, external. Color space information {primaries:BT709, transfer:IEC61966_2_1, matrix:RGB, range:FULL} Bits per color component 8 Bits per pixel 24 Info Display[2841568472] bounds=[1920,0 1920x1080], workarea=[1920,0 1920x1040], scale=1, external. Color space information {primaries:BT709, transfer:IEC61966_2_1, matrix:RGB, range:FULL} Bits per color component 8 Bits per pixel 24 Info Display[2480450848] bounds=[-1920,0 1920x1080], workarea=[-1920,0 1920x1040], scale=1, external. Color space information {primaries:BT709, transfer:IEC61966_2_1, matrix:RGB, range:FULL} Bits per color component 8 Bits per pixel 24 Video Acceleration Information Decode h264 baseline up to 1920x1088 pixels Decode h264 baseline up to 1088x1920 pixels Decode h264 main up to 1920x1088 pixels Decode h264 main up to 1088x1920 pixels Decode h264 high up to 1920x1088 pixels Decode h264 high up to 1088x1920 pixels Encode h264 baseline up to 3840x2176 pixels and/or 30.000 fps Encode h264 main up to 3840x2176 pixels and/or 30.000 fps Encode h264 high up to 3840x2176 pixels and/or 30.000 fps Diagnostics ... loading ... Log Messages GpuProcessHostUIShim: [11720:8484:1212/111100.036:WARNING:angle_platform_impl.cc(52)] : compileToBinary(232): C:\fakepath(73,10-42): warning X3571: pow(f, e) will not work for negative f, use abs(f) or conditionally handle negative values if you expect them C:\fakepath(95,10-42): warning X3571: pow(f, e) will not work for negative f, use abs(f) or conditionally handle negative values if you expect them [11720:1552:1212/111545.888:WARNING:ipc_message_attachment_set.cc(49)] : MessageAttachmentSet destroyed with unconsumed attachments: 0/1 [11720:1552:1212/112300.628:WARNING:ipc_message_attachment_set.cc(49)] : MessageAttachmentSet destroyed with unconsumed attachments: 0/1 we have found this on multiple websites that us flash microphone. We have been able to reproduce on multiple computer.
,
Dec 13
,
Dec 13
Unable to reproduce the issue on chrome reported version# 71.0.3578.80 using Windows-10 with steps mentioned below: 1) Launched chrome reported version and navigated to URL: https://www.onlinemictest.com/microphone-test-in-adobe-flash/ 2) Accepted flash plugin, mic is working as excepted Note: Also tested the issue on latest chrome stable# 71.0.3578.98, unable to reproduce the issue. @Reporter: Please find the attached screencast for your reference, try to test this issue on latest chrome stable# 71.0.3578.98 or by creating a new person with no apps and extensions in it and let us know if the issue still persists. You can download latest chrome from URL: https://www.chromium.org/getting-involved/dev-channel. Attached details of GPU. Thanks!
,
Dec 13
,
Dec 13
I watched your video and noticed you used a camera and mic. We are using only a mic headset. if we use the webcam with built in mic it does work. But a headset with mic either usb or TRS jack do not work.
,
Dec 13
Thank you for providing more feedback. Adding the requester to the cc list. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Dec 14
Tried testing the issue as per comment# 5 using Windows-10 with steps mentioned below: 1) Launched chrome reported version and navigated to URL: https://www.onlinemictest.com/microphone-test-in-adobe-flash/ 2) Clicked on ADOBE flash player icon, and clicked on Allow in permission pop-up 3) Didn't seen 'Allow' message for ADOBE flash player, seen blank page @Reporter: Please find the attached screencast for your reference and let us know if we missed anything in reproducing the issue, provide your feedback on it which help in further triaging it in better way. Thanks!
,
Dec 14
++ Attachment
,
Dec 14
It looks like we are not the only people having this issue. please see the link https://productforums.google.com/forum/#!topic/chrome/tVz3mbIMvww;context-place=topicsearchin/chrome/chrome$2071$20microphone$20blocked
,
Dec 14
Thank you for providing more feedback. Adding the requester to the cc list. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Dec 14
I'm the head of support at a webinar provider that uses Flash and I can report that we are running into this issue as well. I'm posting here so that the Chrome support person can see others are still affected by this issue: As the OP stated, Chrome is not allowing permission to use the microphone in these applications if there is no camera present. Even if you explicitly allow a site to access it from the chrome://settings page, or via the privacy menu in the address bar, without a camera, the site will always be blocked. A short video is attached showing the problem, as experienced in our platform: We see that the reproduction attempts made in this thread they use a webcam and microphone, which will work fine. Please remove the camera and use a simple headset and you will be able to reproduce the issue every time. My questions are: Is this expected behavior with the latest update, and we will have to change our code to compensate? If so, do you have any recommendations on things to look for and change? Is this a bug that will be patched? Any advice would be appreciated. Thank you.
,
Dec 17
Tried testing the issue on chrome version# 71.0.3578.80 and on # 71.0.3578.98 using Windows-10 where we have only access for Mic, but somehow unable to reproduce it. Hence routing this issue to Inhouse for further triaging it, adding TE-NeedsTriageFromHYD label to it. Thanks!
,
Dec 17
Tested this on Windows-10, Linux Debian Rodete Desktop(with no camera and only microphone) on the latest stable 71.0.3578.98 and canary 73.0.3643.0 as per below test steps: Steps followed: =============== 1. Opened https://www.onlinemictest.com/microphone-test-in-adobe-flash/. 2. Clicked the 'Get Adobe flash player icon' and 'Allowed' the page to run flash. 3. On good build 'Allowing' the access on 'Adobe flash player settings' dialog shows the 'line' and it moves when I speak. On Bad build page is blank when allowed the access on 'Adobe flash player settings' dialog. Attaching the screen-cast of good and bad behavior. Considered C#3 behaviour for bisecting the issue and below is the result. Last good build: 71.0.3570.0 First bad build: 71.0.3571.0 Changelog: ========== https://chromium.googlesource.com/chromium/src/+log/36f28038d708bc1976b0c975de0ea95c747fa58b..ac00a5f3b9ba87fe2b59ff66d9907c3fb56deb96 guidou@: Could you please take a look at this. Thank you!
,
Dec 17
Removing RBS label since the impact of this bug is reduced to flash request for audio/video where there is no webcam in the system. Working on a fix that might get merged to 72 and 71, but not blocking 71 release on this bug.
,
Dec 17
The impact of this bug is NOT reduced to a flash request for audio/video where there is no webcam in the system. You INTRODUCED the bug by what you did that you claim was to bring this back into spec. Clearly, the conditions you claim must not be allowed, MUST BE allowed. If you claim there is a spec that's being violated, then the SPEC ITSELF is flawed because it fails to consider standard operating conditions that are encountered in normal use.
,
Dec 18
jeffrey.hildack@gmail.com: To answer your questions. * Is this expected behavior with the latest update, and we will have to change our code to compensate? There is a behavior change that exposed a pre-existing bug. The behavior change is that if you request video and audio, and video permission is denied and audio permission is granted, then getUserMedia() will fail. Before, you would get an audio track, which is against the spec. Unfortunately, this change exposed a bug in a separate, but related part of the code that manifests in some situations where there is no camera. For example, in onlinemictest.com, the flash version hits the bug, but the HTML5 version does not. * If so, do you have any recommendations on things to look for and change? If you are not requesting both audio and video and expecting to get only audio, you shouldn't change anything in your application once we fix the bug. A fix will be available shortly in Canary 73. * Is this a bug that will be patched? Yes, it is. And we plan to merge the fix back to 72 and 71.
,
Dec 18
barrycoupang@: It looks like your bug is different from this one. There is nothing wrong with the spec. There is just a bug that affects some scenarios where there is no webcam installed and it will be fixed shortly. If your bug is not related to having no webcams, it is a different issue. Once the fix for this bug lands, test it anyway to see if things improve for the other bug.
,
Dec 18
barrycoupang@: Note also that this bug is strictly related to flash requests for audio when there are no webcams. If your bug has nothing to do with this scenario, then it's a different bug.
,
Dec 19
+Adobe folks for awareness. "With the proposed fix for 71+, if an application makes two requests it will see two permission prompts (one for each request/device type) if the permission hasn't been previously granted." - Guido
,
Dec 19
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/9695ac9fec245f1f303c991a86c61bc73e3ab39d commit 9695ac9fec245f1f303c991a86c61bc73e3ab39d Author: Guido Urdaneta <guidou@chromium.org> Date: Wed Dec 19 16:49:09 2018 Fix media-device permission handling of OpenDevice Pepper requests Prior to this CL, all Pepper OpenDevice request were treated as if they were requesting both audio and video in terms of requesting permissions. When crrev.com/596698 landed, it changed general permission handling to comply with the Media Capture and Streams spec, which states that requests for audio and video should fail if permission for one of the device types cannot be obtained. Prior to crrev.com/596698, requests succeeded and returned a track for each authorized device types, and no tracks for unauthorized types. This broke Pepper requests for audio when there are no webcams in the system. The reason is that the pepper requests were treated as requesting both device types, but if there is no webcam, the corresponding permission is blocked due to lack of hardware. This CL fixes that by treating Pepper requests similarly to getUserMedia() requests, where permissions are checked only for device types that are requested. Bug: 914398 Change-Id: Ic79102d69efb93f874295dcb5a2c6fa463fd67be Reviewed-on: https://chromium-review.googlesource.com/c/1382271 Reviewed-by: Raymes Khoury <raymes@chromium.org> Commit-Queue: Guido Urdaneta <guidou@chromium.org> Cr-Commit-Position: refs/heads/master@{#617858} [modify] https://crrev.com/9695ac9fec245f1f303c991a86c61bc73e3ab39d/chrome/browser/media/webrtc/media_stream_devices_controller.cc [modify] https://crrev.com/9695ac9fec245f1f303c991a86c61bc73e3ab39d/chrome/browser/media/webrtc/media_stream_devices_controller_browsertest.cc
,
Dec 19
Thanks for the head's up. We're seeing a modest amount of traffic around this topic on our forums, just FYI.
,
Dec 19
Has this been pushed out yet? What version n umber should this be? Last Chrome version I'm seeing here when checking for an update is still 71.0.3578.98.
,
Dec 20
This will be available in an upcoming Canary M73. After verifying we will merge the change to 72 and 71.
,
Dec 20
chris.martin@glynnk12.org, jeffrey.hildack@gmail.com: The fix is availiable in Canary 73.0.3646.0. Please try it and let us know how it goes.
,
Dec 21
guidou@chromium.org: I can confirm that audio now works in Canary 73 as expected after allowing a microphone access and then activating it our software. I see that the allowances for mic and cam are more separate, with cam permissions not even showing up if no cam device is connected. Apart from some odd application flickering, which may be unrelated, I can say that your fix worked for us. Can you please keep us all updated here as to the release of this fix for Chrome 71 and 72? It would be greatly appreciated. Thanks, and if any others have any news, I'd be interested to see how this worked for you.
,
Dec 21
,
Dec 21
Requesting merge to 72 based on independent verification in #25.
,
Dec 21
This bug requires manual review: M72 has already been promoted to the beta branch, so this requires manual review Please contact the milestone owner if you have questions. Owners: govind@(Android), kariahda@(iOS), djmm@(ChromeOS), abdulsyed@(Desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Dec 21
,
Dec 21
Also requesting merge to 71.
,
Dec 21
branch:3626
,
Dec 21
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/be6c07a1380c2006d46eaa826573d9c544dd49a7 Commit: be6c07a1380c2006d46eaa826573d9c544dd49a7 Author: guidou@chromium.org Commiter: guidou@chromium.org Date: 2018-12-21 21:35:42 +0000 UTC Fix media-device permission handling of OpenDevice Pepper requests Prior to this CL, all Pepper OpenDevice request were treated as if they were requesting both audio and video in terms of requesting permissions. When crrev.com/596698 landed, it changed general permission handling to comply with the Media Capture and Streams spec, which states that requests for audio and video should fail if permission for one of the device types cannot be obtained. Prior to crrev.com/596698, requests succeeded and returned a track for each authorized device types, and no tracks for unauthorized types. This broke Pepper requests for audio when there are no webcams in the system. The reason is that the pepper requests were treated as requesting both device types, but if there is no webcam, the corresponding permission is blocked due to lack of hardware. This CL fixes that by treating Pepper requests similarly to getUserMedia() requests, where permissions are checked only for device types that are requested. Bug: 914398 Change-Id: Ic79102d69efb93f874295dcb5a2c6fa463fd67be Reviewed-on: https://chromium-review.googlesource.com/c/1382271 Reviewed-by: Raymes Khoury <raymes@chromium.org> Commit-Queue: Guido Urdaneta <guidou@chromium.org> Cr-Original-Commit-Position: refs/heads/master@{#617858}(cherry picked from commit 9695ac9fec245f1f303c991a86c61bc73e3ab39d) Reviewed-on: https://chromium-review.googlesource.com/c/1389115 Reviewed-by: Guido Urdaneta <guidou@chromium.org> Cr-Commit-Position: refs/branch-heads/3626@{#507} Cr-Branched-From: d897fb137fbaaa9355c0c93124cc048824eb1e65-refs/heads/master@{#612437}
,
Dec 21
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/be6c07a1380c2006d46eaa826573d9c544dd49a7 commit be6c07a1380c2006d46eaa826573d9c544dd49a7 Author: Guido Urdaneta <guidou@chromium.org> Date: Fri Dec 21 21:35:42 2018 Fix media-device permission handling of OpenDevice Pepper requests Prior to this CL, all Pepper OpenDevice request were treated as if they were requesting both audio and video in terms of requesting permissions. When crrev.com/596698 landed, it changed general permission handling to comply with the Media Capture and Streams spec, which states that requests for audio and video should fail if permission for one of the device types cannot be obtained. Prior to crrev.com/596698, requests succeeded and returned a track for each authorized device types, and no tracks for unauthorized types. This broke Pepper requests for audio when there are no webcams in the system. The reason is that the pepper requests were treated as requesting both device types, but if there is no webcam, the corresponding permission is blocked due to lack of hardware. This CL fixes that by treating Pepper requests similarly to getUserMedia() requests, where permissions are checked only for device types that are requested. Bug: 914398 Change-Id: Ic79102d69efb93f874295dcb5a2c6fa463fd67be Reviewed-on: https://chromium-review.googlesource.com/c/1382271 Reviewed-by: Raymes Khoury <raymes@chromium.org> Commit-Queue: Guido Urdaneta <guidou@chromium.org> Cr-Original-Commit-Position: refs/heads/master@{#617858}(cherry picked from commit 9695ac9fec245f1f303c991a86c61bc73e3ab39d) Reviewed-on: https://chromium-review.googlesource.com/c/1389115 Reviewed-by: Guido Urdaneta <guidou@chromium.org> Cr-Commit-Position: refs/branch-heads/3626@{#507} Cr-Branched-From: d897fb137fbaaa9355c0c93124cc048824eb1e65-refs/heads/master@{#612437} [modify] https://crrev.com/be6c07a1380c2006d46eaa826573d9c544dd49a7/chrome/browser/media/webrtc/media_stream_devices_controller.cc [modify] https://crrev.com/be6c07a1380c2006d46eaa826573d9c544dd49a7/chrome/browser/media/webrtc/media_stream_devices_controller_browsertest.cc
,
Dec 21
,
Dec 21
,
Dec 26
guidou@chromium.org: Just checking in on this issue. Will we receive a separate reply when the changes are merged with 72 and 71, or have the changes been rolled out already to 72? If it's the latter, then there is no change to the behavior and the bug is not fixed in 72.0.3626.28 released 4 days ago, or so. The bug still appears to be fixed in Canary. Any updates would be helpful. Thanks!
,
Dec 27
The fix is not in 72.0.3626.28. It will probably be available in the next beta. It hasn't been merged to 71 yet.
,
Jan 2
At the moment there is no plan for M71 stable respin. If change looks good in M72 beta, we can decide to take it in for M71 respin (if any).
,
Jan 7
FYI, my tests show that, as of 1/7/19, this issue is fixed in the 73.0.3664.0 Canary release, but has not made it to the Dev channel 73.0.3642.0 release. A number of students using our software in classrooms are affected by this issue, and any info regarding when we can expect to see this fix pushed to the stable release would be appreciated. Thanks.
,
Jan 9
The fix is now available in the Dev channel (currently 73.0.3664.3). It is not yet available on the Beta channel, but should be available when a new beta is released.
,
Jan 9
This is a critical issue currently for our school district. I see that the issue is fixed for future versions however there is no solution (in Chrome) for us at the moment so I am adding this comment for reference. We use a website called Read 180 which still uses flash. It requires a headset with microphone to use. No matter what there is no way to allow the microphone when the headset is connected. If I were to use a webcam with microphone it works as it should. We have had this issue for weeks and I have been scrambling to set up Chrome's Legacy Browser Support, however many of our students who use Read 180 would have trouble signing into the Chrome browser in order to get the Legacy Browser Support extension installed/added via admin console policy. I thought to add the LBS extension, using the instructions provided in the link below, for our Windows machines through GPO so that the extension could install without the need for the students to login to the browser first. https://support.google.com/chrome/a/answer/3062037?hl=en I found that adding this registry entry to a GPO ends up overriding the Google admin console forced installed extensions setting and makes the LBS extension the only forced installed extension. Now we are back to square one. We really don't want our students using another browser without using LBS but it might be our only option until this is released to a stable update.
,
Jan 9
mlametta@, are you seeing this issue on Chromebook?
,
Jan 9
In reference to the flash/microphone permissions issue, no, our chromebooks have not had this issue, only our Windows devices. Our students use both PCs and chromebooks to access Read 180. All of our chromebooks have cameras as well as microphones so that is probably why they work fine. It might be after the fact now but this screenshot was a good indicator that something was interpreting camera vs microphone wrong. There was a headset with mic connected but no camera, notice 'Camera blocked'. https://drive.google.com/file/d/1KymJJcxjHymsuejih-zRb2fn5k0Gg3Zh/view I know this question is out of the scope of this topic and I will look to proper support channels but should the ExtensionInstallForcelist Chrome policy be overriding the Google admin console setting?
,
Jan 9
I will second mlame...@trschools.com's comments. Our school district also uses Read 180 and we are in the same boat. Added that we have Windows machines, Chromebooks and Chromeboxes. The Chromebooks don't run into the issue because they have a built in camera - though our Chromeboxes do not have cameras connected thus all are failing. I have a test Chromebox that I have connected one of our standard headsets w/ microphone to and it failed as expected then I connected an external USB camera to the same Chromebox and the same headset works fine.
,
Jan 9
This bug is reproducible on all platforms in M71 if there is no camera installed. Possible workarounds are to install a webcam (even a fake/virtual software-based webcam should work), or install the new beta 72.0.3626.53, which contains the fix and was released yesterday for Mac, Windows and Linux (please test to verify that it works). Note that M72 Stable will be released shortly (see https://chromiumdash.appspot.com/schedule), so it is unlikely that there will be another M71 release, but if there is one it will include the fix. mlametta@: I don't have enough context to provide a proper answer to your question about ExtensionInstallForcelist, but it won't fix this bug in M71.
,
Jan 10
Confirmed working in the new Chrome 72 beta on Windows. Our solution will be to push this out to the machines experiencing this issue and create a shortcut specifically for the Read 180 website on the desktop. Thank you Chromium team. Google Chrome 72.0.3626.53 (Official Build) beta (64-bit) (cohort: Beta) Revision 98434e6cd182d68ce396daa92e9c6310422e6763-refs/branch-heads/3626@{#620} OS Windows JavaScript V8 7.2.502.19 Flash 32.0.0.114 C:\windows\system32\Macromed\Flash\pepflashplayer64_32_0_0_114.dll
,
Jan 10
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by chris.ma...@glynnk12.org
, Dec 12