Issue metadata
Sign in to add a comment
|
mp3 radio stream no longer plays beyond the first 13 seconds
Reported by
kennerly...@gmail.com,
Mar 15 2018
|
||||||||||||||||||||||
Issue descriptionUserAgent: Mozilla/5.0 (Windows NT 6.1; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/65.0.3325.162 Safari/537.36 Example URL: http://audio-mp3.ibiblio.org:8000/wncw-128k Steps to reproduce the problem: 1. Open tab to http://audio-mp3.ibiblio.org:8000/wncw-128k 2. Click play (if not already playing) 3. Listen/watch stream stop after 13 seconds What is the expected behavior? The stream should play on indefinitely. What went wrong? The stream automatically stops after 13 seconds no matter what. Did this work before? Yes No sure. But it worked last week. Is it a problem with Flash or HTML5? N/A Does this work in other browsers? Yes Chrome version: 65.0.3325.162 Channel: stable OS Version: 6.1 (Windows 7, Windows Server 2008 R2) Flash Version: Contents of chrome://gpu: Graphics Feature Status Canvas: Hardware accelerated CheckerImaging: Disabled 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 Rasterization: Hardware accelerated Video Decode: Hardware accelerated WebGL: Hardware accelerated WebGL2: Hardware accelerated Driver Bug Workarounds clear_uniforms_before_first_program_use decode_encode_srgb_for_generatemipmap disable_accelerated_vpx_decode disable_delayed_copy_nv12 disable_direct_composition disable_discard_framebuffer disable_dxgi_zero_copy_video disable_framebuffer_cmaa disable_larger_than_screen_overlays exit_on_context_lost force_cube_complete msaa_is_slow scalarize_vec_and_mat_constructor_args texsubimage_faster_than_teximage Problems Detected Some drivers are unable to reset the D3D device in the GPU process sandbox Applied Workarounds: exit_on_context_lost TexSubImage is faster for full uploads on ANGLE Applied Workarounds: texsubimage_faster_than_teximage 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 On Intel GPUs MSAA performance is not acceptable for GPU rasterization: 527565 Applied Workarounds: msaa_is_slow Framebuffer discarding can hurt performance on non-tilers: 570897 Applied Workarounds: disable_discard_framebuffer Direct composition flashes black initially on Win <10: 588588 Applied Workarounds: disable_direct_composition Zero copy DXGI video hangs on shutdown on Win < 8.1: 621190 Applied Workarounds: disable_dxgi_zero_copy_video 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 VPx decoding isn't supported well before Windows 10 creators update.: 616318, 667532 Applied Workarounds: disable_accelerated_vpx_decode Accelerated VPx decoding is hanging on some videos.: 654111 Applied Workarounds: disable_accelerated_vpx_decode Overlay sizes bigger than screen aren't accelerated on some Intel drivers: 720059 Applied Workarounds: disable_larger_than_screen_overlays Delayed copy NV12 crashes on Intel on Windows <= 8.1.: 727216 Applied Workarounds: disable_delayed_copy_nv12 Don't expose disjoint_timer_query extensions to WebGL: 808744 Native GpuMemoryBuffers have been disabled, either via about:flags or command line. Disabled Features: native_gpu_memory_buffers Checker-imaging has been disabled via finch trial or the command line. Disabled Features: checker_imaging Version Information Data exported 2018-03-15T17:37:41.974Z Chrome version Chrome/65.0.3325.162 Operating system Windows NT 6.1.7601 SP1 Software rendering list URL https://chromium.googlesource.com/chromium/src/+/5d04e9e9c8ce31bee0923a8c326a7e9e19c492a3/gpu/config/software_rendering_list.json Driver bug list URL https://chromium.googlesource.com/chromium/src/+/5d04e9e9c8ce31bee0923a8c326a7e9e19c492a3/gpu/config/gpu_driver_bug_list.json ANGLE commit id 2c9cc8b6e810 2D graphics backend Skia/65 bd0dafbc8112f6cfa92a8096d8cb5696d8535ef9- Command Line "C:\Program Files (x86)\Google\Chrome\Application\chrome.exe" --flag-switches-begin --enable-tab-audio-muting --flag-switches-end Driver Information Initialization time 41 In-process GPU false Passthrough Command Decoder false Direct Composition false Supports overlays false Sandboxed false GPU0 VENDOR = 0x8086, DEVICE= 0x0412 *ACTIVE* Optimus false Optimus false AMD switchable false Desktop compositing none Diagonal Monitor Size of \\.\DISPLAY1 24.0" Driver vendor Intel Corporation Driver version 10.18.14.4889 Driver date 12-14-2017 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 (Intel(R) HD Graphics 4600 Direct3D11 vs_5_0 ps_5_0) GL_VERSION OpenGL ES 2.0 (ANGLE 2.1.0.2c9cc8b6e810) GL_EXTENSIONS GL_ANGLE_client_arrays GL_ANGLE_depth_texture GL_ANGLE_framebuffer_blit GL_ANGLE_framebuffer_multisample GL_ANGLE_instanced_arrays GL_ANGLE_lossy_etc_decode 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_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_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 Disabled Extensions GL_KHR_blend_equation_advanced GL_KHR_blend_equation_advanced_coherent Disabled WebGL Extensions EXT_disjoint_timer_query EXT_disjoint_timer_query_webgl2 Window system binding vendor Google Inc. (adapter LUID: 000000000000af8d) Window system binding version 1.4 (ANGLE 2.1.0.2c9cc8b6e810) 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_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_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 Direct rendering Yes Reset notification strategy 0x8252 GPU process crash count 0 Compositor Information Tile Update Mode One-copy Partial Raster Enabled GpuMemoryBuffers Status ATC Software only ATCIA Software only DXT1 Software only DXT5 Software only ETC1 Software only 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 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 1920x1200, workarea=0,0 1920x1160, scale=1, external Color space information {primaries:[[0.4360,0.3851,0.1431,],[0.2225,0.7169,0.0606,],[0.0139,0.0971,0.7139,],], transfer:0.0000*x + 0.0000 if x < 0.0000 else (1.0000*x + 0.0000)**2.2000 + 0.0000, 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 main up to 1920x1088 pixels Decode h264 high up to 1920x1088 pixels Diagnostics ... loading ... Log Messages [12212:7772:0315/130637.000:WARNING:angle_platform_impl.cc(59)] : compileToBinary(228): C:\fakepath(49,8-58): 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(53,8-28): warning X3571: pow(f, e) will not work for negative f, use abs(f) or conditionally handle negative values if you expect them [12212:7772:0315/130637.012:WARNING:angle_platform_impl.cc(59)] : compileToBinary(228): C:\fakepath(29,8-58): 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(33,8-28): warning X3571: pow(f, e) will not work for negative f, use abs(f) or conditionally handle negative values if you expect them [12212:7768:0315/130637.065:WARNING:angle_platform_impl.cc(59)] : compileToBinary(228): C:\fakepath(46,8-58): 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(50,8-28): warning X3571: pow(f, e) will not work for negative f, use abs(f) or conditionally handle negative values if you expect them [12212:7768:0315/130637.575:WARNING:angle_platform_impl.cc(59)] : compileToBinary(228): C:\fakepath(65,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(87,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 [12212:7768:0315/130641.277:WARNING:angle_platform_impl.cc(59)] : compileToBinary(228): C:\fakepath(46,8-58): 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(50,8-28): warning X3571: pow(f, e) will not work for negative f, use abs(f) or conditionally handle negative values if you expect them [12212:7768:0315/130701.698:WARNING:angle_platform_impl.cc(59)] : compileToBinary(228): C:\fakepath(47,8-58): 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(51,8-28): warning X3571: pow(f, e) will not work for negative f, use abs(f) or conditionally handle negative values if you expect them This station WNCW has had a working mp3 audio stream for many years. Their stream continues to work flawlessly in the Firefox and Eclipse browsers. It has also always worked with Google Chrome (my primary browser) until recently. Chrome now only plays the first 13 seconds of the stream intro.
,
Mar 15 2018
can you provide a download link to m66+
,
Mar 15 2018
,
Mar 15 2018
same failure, stream stops after 13 seconds. Question: You'll notice my ticket says I'm using Chrome version: 65.0.3325.162 But the m66+ download you gave me installs Chrome Version 65.0.3325.125 (Official Build) beta (64-bit) Why is (latest) .162 > .125 (m66+)?
,
Mar 15 2018
Ah, sorry, I thought beta was on M66 now. The easiest way is probably just to check Canary then, https://www.google.com/chrome/browser/canary.html
,
Mar 15 2018
I tried Canary first and opened these two tickets when they referred me to chromium.org. It fails the same way in Canary for both of these modes: Mobile: https://github.com/GoogleChrome/lighthouse/issues/4775 Desktop: https://github.com/GoogleChrome/lighthouse/issues/4776
,
Mar 15 2018
Ah, thanks. It's still related to issue 794782 , but the stream has a lot of damage; which results in decoding errors stopping the stream. We got more strict about these in M64. $ mp3val temp.mp3 Analyzing file "temp.mp3"... WARNING: "/d/code/chrome/src/temp.mp3" (offset 0x3543f): MPEG stream error, resynchronized successfully WARNING: "/d/code/chrome/src/temp.mp3" (offset 0x6daba): It seems that file is truncated or there is garbage at the end of the file WARNING: "/d/code/chrome/src/temp.mp3": No supported tags in the file INFO: "/d/code/chrome/src/temp.mp3": 1074 MPEG frames (MPEG 1 Layer III), no tags, CBR, CRC Done! This is fixable with mp3val -f temp.mp3 http://mp3val.sourceforge.net/
,
Mar 15 2018
Thanks! I'll let them know. But as a human being listening to the stream with Firefox or Eclipse, my ear can't tell these errors exist, so why does it matter? In other words, in all of my experience developing code, if an error is recoverable, then recover from it and continue streaming, like Firefox and Eclipse are doing. Can you get an answer for why they are choosing no tot recover from what seems to be a very picky error?
,
Mar 15 2018
Besides, those are only warnings. Even a compiler still produces a binary executable when there are only warnings. Just sayin' If you don't mind me asking, how did you use this tool with the url that I gave you? I get this... $ mp3val.exe http://audio-mp3.ibiblio.org:8000/wncw-128k Cannot open input file wncw-128k
,
Mar 15 2018
I just downloaded the beginning of the stream using wget. Per spec we are supposed to surface decoding errors and we generally try not to just ignore them since that hides real issues that folks should be fixing. In cases where this is audio+video these types of issues lead to hidden a/v sync problems. There is some debate on if we should just ignore certain types of errors (junk packets) coming from ffmpeg in the case of adts or mp3 streams where there is no real container.
,
Mar 15 2018
dalecur, Can I just say that I think you're awesome. That was an excellent, technical, and completely thoughtful reply. I have never experienced this much quick responsiveness on a bug in my entire life! Cheers!
,
Mar 16 2018
,
Mar 16 2018
Able to reproduce the issue on reported chrome version 65.0.3325.162 using Windows-7 & 10, Mac 10.12.6 & Ubuntu 14.04 hence providing Bisect Info Bisect Info: ================ Good build: 64.0.3257.0 Bad build: 64.0.3258.0 You are probably looking for a change made after 513689 (known good), but no later than 513690 (first known bad). https://chromium.googlesource.com/chromium/src/+log/7642e1c5fc5715dc1894eaf828ef85ae0a5d1fb7..9f57237995f7921b4dcd8855f1f6fe98874218d2 Reviewed-on: https://chromium-review.googlesource.com/738626 @Dale Curtis: Please confirm the issue and help in re-assigning if it is not related to your change. Thanks!
,
Mar 16 2018
np; happy to help! I'm going to dupe this into the other bug for now, I have a patch coming which should help out though.
,
Mar 19 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/c7916b3e214429239af483abb78efe45e925c08b commit c7916b3e214429239af483abb78efe45e925c08b Author: Dale Curtis <dalecurtis@chromium.org> Date: Mon Mar 19 22:08:04 2018 Ignore invalid MP3 packets from ffmpeg. Per upstream, this is totally normal to happen and they won't accept changes to fix this without undertaking a large refactoring of the AVParser infrastructure: http://ffmpeg.org/pipermail/ffmpeg-devel/2018-February/225800.html This behavior may also occur with ADTS streams, but is rarer in practice because ffmpeg's ADTS demuxer does more validation on the packets, so when invalid data is received, av_read_frame() fails and playback ends. Since we haven't had a rash of complaints about too-short ADTS streams, don't bother doing this for ADTS streams at this time. BUG= 822341 , 794782 TEST=existing tests all still pass. Cq-Include-Trybots: luci.chromium.try:linux_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel Change-Id: Iabcad145bb75f742e4ba7658ab95c377b811efa1 Reviewed-on: https://chromium-review.googlesource.com/965121 Commit-Queue: Dale Curtis <dalecurtis@chromium.org> Reviewed-by: Matthew Wolenetz <wolenetz@chromium.org> Cr-Commit-Position: refs/heads/master@{#544181} [modify] https://crrev.com/c7916b3e214429239af483abb78efe45e925c08b/media/filters/audio_decoder_unittest.cc [modify] https://crrev.com/c7916b3e214429239af483abb78efe45e925c08b/media/filters/ffmpeg_demuxer.cc [modify] https://crrev.com/c7916b3e214429239af483abb78efe45e925c08b/media/filters/ffmpeg_demuxer.h [modify] https://crrev.com/c7916b3e214429239af483abb78efe45e925c08b/media/formats/mpeg/mpeg1_audio_stream_parser.cc [modify] https://crrev.com/c7916b3e214429239af483abb78efe45e925c08b/media/test/pipeline_integration_test.cc
,
Mar 19 2018
Not sure how to interpret your previous comment. Does 'this is totally normal to happen' mean that Chrome will be fixed to ignore these WARNINGS? or not?
,
Mar 19 2018
They're not warnings, but are common errors with raw containers like MP3 and ADTS. As such, since there is no a/v sync concerns and these packets are trivially verifiable, we have decided to discard invalid mp3 packets instead of trying to decode them.
,
Mar 20 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/d4c3823079fe41cd1cbef1ffe2e8b24a173f5559 commit d4c3823079fe41cd1cbef1ffe2e8b24a173f5559 Author: Dale Curtis <dalecurtis@chromium.org> Date: Tue Mar 20 07:27:24 2018 Fix accidental media_log to nullptr. BUG= 823597 , 822341 , 794782 TBR=wolenetz Cq-Include-Trybots: luci.chromium.try:linux_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel Change-Id: I7000336c8c34dbb310bc40faa33d3e51c4e1efa3 Reviewed-on: https://chromium-review.googlesource.com/969984 Commit-Queue: Dale Curtis <dalecurtis@chromium.org> Reviewed-by: Dale Curtis <dalecurtis@chromium.org> Cr-Commit-Position: refs/heads/master@{#544304} [modify] https://crrev.com/d4c3823079fe41cd1cbef1ffe2e8b24a173f5559/media/filters/ffmpeg_demuxer.cc
,
Mar 20 2018
Dale, Do you know if this 'fix' will be included in the next Chrome release or when that release might be scheduled? Thanks, Ken
,
Mar 20 2018
If things look good next week I'll try to get it merged to M66; which should launch in April, if not it'll go out with M67 in ~June.
,
Mar 20 2018
awesome! thanks! |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by dalecur...@chromium.org
, Mar 15 2018