Issue metadata
Sign in to add a comment
|
Unable to load video with flash in Beta/Canary
Reported by
da...@jwplayer.com,
Jan 6 2017
|
||||||||||||||||||||||
Issue descriptionUserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/56.0.2924.51 Safari/537.36 Example URL: http://www.pbs.org/video/2365921903/ Steps to reproduce the problem: 1. Go to the test page provided 2. Wait for the video player to load with flash content What is the expected behavior? SWF properly loads and user can start content. What went wrong? The SWF object never loads. The player will show a setup timeout error after 30 seconds if unable to set up. Did this work before? Yes Chrome 55 Is it a problem with Flash or HTML5? Flash Does this work in other browsers? Yes Chrome version: 56.0.2924.51 Channel: beta OS Version: OS X 10.12.0 Flash Version: Shockwave Flash 24.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: Hardware accelerated Rasterization: Hardware accelerated Video Decode: Hardware accelerated Video Encode: Hardware accelerated VPx Video Decode: Hardware accelerated WebGL: Hardware accelerated WebGL2: Hardware accelerated Driver Bug Workarounds add_and_true_to_loop_condition adjust_src_dst_region_for_blitframebuffer decode_encode_srgb_for_generatemipmap disable_framebuffer_cmaa disable_multimonitor_multisampling disable_webgl_rgb_multisampling_usage emulate_abs_int_function get_frag_data_info_bug init_two_cube_map_levels_before_copyteximage msaa_is_slow pack_parameters_workaround_with_pack_buffer rebind_transform_feedback_before_resume regenerate_struct_names remove_invariant_and_centroid_for_essl3 reset_base_mipmap_level_before_texstorage rewrite_texelfetchoffset_to_texelfetch scalarize_vec_and_mat_constructor_args set_zero_level_before_generating_mipmap unfold_short_circuit_as_ternary_operation unpack_alignment_workaround_with_unpack_buffer unpack_image_height_workaround_with_unpack_buffer use_intermediary_for_copy_texture_image use_shadowed_tex_level_params use_unused_standard_shared_blocks Problems Detected Multisampling is buggy on OSX when multiple monitors are connected: 237931 Applied Workarounds: disable_multimonitor_multisampling Unfold short circuit on Mac OS X: 307751 Applied Workarounds: unfold_short_circuit_as_ternary_operation Always rewrite vec/mat constructors to be consistent: 398694 Applied Workarounds: scalarize_vec_and_mat_constructor_args Mac drivers handle struct scopes incorrectly: 403957 Applied Workarounds: regenerate_struct_names On Intel GPUs MSAA performance is not acceptable for GPU rasterization: 527565 Applied Workarounds: msaa_is_slow glGenerateMipmap fails if the zero texture level is not set on some Mac drivers: 560499 Applied Workarounds: set_zero_level_before_generating_mipmap Pack parameters work incorrectly with pack buffer bound: 563714 Applied Workarounds: pack_parameters_workaround_with_pack_buffer Alignment works incorrectly with unpack buffer bound: 563714 Applied Workarounds: unpack_alignment_workaround_with_unpack_buffer copyTexImage2D fails when reading from IOSurface on multiple GPU types.: 581777 Applied Workarounds: use_intermediary_for_copy_texture_image Multisample renderbuffers with format GL_RGB8 have performance issues on Intel GPUs.: 607130 Applied Workarounds: disable_webgl_rgb_multisampling_usage Mac Drivers store texture level parameters on int16_t that overflow: 610153 Applied Workarounds: use_shadowed_tex_level_params Limited enabling of Chromium GL_INTEL_framebuffer_CMAA: 535198 Applied Workarounds: disable_framebuffer_cmaa glGetFragData{Location|Index} works incorrectly on Max: 638340 Applied Workarounds: get_frag_data_info_bug glResumeTransformFeedback works incorrectly on Intel GPUs: 638514 Applied Workarounds: rebind_transform_feedback_before_resume glTexStorage* are buggy when base mipmap level is not 0: 640506 Applied Workarounds: reset_base_mipmap_level_before_texstorage Result of abs(i) where i is an integer in vertex shader is wrong: 642227 Applied Workarounds: emulate_abs_int_function Rewrite texelFetchOffset to texelFetch for Intel Mac: 642605 Applied Workarounds: rewrite_texelfetchoffset_to_texelfetch Rewrite condition in for and while loops for Intel Mac: 644669 Applied Workarounds: add_and_true_to_loop_condition Decode and encode before generateMipmap for srgb format textures on macosx: 634519 Applied Workarounds: decode_encode_srgb_for_generatemipmap Init first two levels before CopyTexImage2D for cube map texture on Intel Mac 10.12: 648197 Applied Workarounds: init_two_cube_map_levels_before_copyteximage Insert statements to reference all members in unused std140/shared blocks on Mac: 618464 Applied Workarounds: use_unused_standard_shared_blocks Tex(Sub)Image3D performs incorrectly when uploading from unpack buffer with GL_UNPACK_IMAGE_HEIGHT greater than zero on Intel Macs: 654258 Applied Workarounds: unpack_image_height_workaround_with_unpack_buffer adjust src/dst region if blitting pixels outside read framebuffer on Mac: 644740 Applied Workarounds: adjust_src_dst_region_for_blitframebuffer Mac driver GL 4.1 requires invariant and centroid to match between shaders: 639760, 641129 Applied Workarounds: remove_invariant_and_centroid_for_essl3 Version Information Data exported 1/6/2017, 1:04:36 PM Chrome version Chrome/56.0.2924.51 Operating system Mac OS X 10.12.0 Software rendering list version 12.06 Driver bug list version 9.24 ANGLE commit id 555009cefe4c 2D graphics backend Skia/56 5c25cbedfb54b8b4bb383f775eb454c842e4fa55 Command Line Args Chrome Beta.app/Contents/MacOS/Google Chrome --flag-switches-begin --flag-switches-end Driver Information Initialization time 65 In-process GPU false Sandboxed true GPU0 VENDOR = 0x8086, DEVICE= 0x0a2e *ACTIVE* Optimus false AMD switchable false Driver vendor Driver version 10.18.65 Driver date Pixel shader version 4.10 Vertex shader version 4.10 Max. MSAA samples 8 Machine model name MacBookPro Machine model version 11.1 GL_VENDOR Intel Inc. GL_RENDERER Intel Iris OpenGL Engine GL_VERSION 4.1 INTEL-10.18.65 GL_EXTENSIONS GL_ARB_blend_func_extended GL_ARB_draw_buffers_blend GL_ARB_draw_indirect GL_ARB_ES2_compatibility GL_ARB_explicit_attrib_location GL_ARB_gpu_shader_fp64 GL_ARB_gpu_shader5 GL_ARB_instanced_arrays GL_ARB_internalformat_query GL_ARB_occlusion_query2 GL_ARB_sample_shading GL_ARB_sampler_objects GL_ARB_separate_shader_objects GL_ARB_shader_bit_encoding GL_ARB_shader_subroutine GL_ARB_shading_language_include GL_ARB_tessellation_shader GL_ARB_texture_buffer_object_rgb32 GL_ARB_texture_cube_map_array GL_ARB_texture_gather GL_ARB_texture_query_lod GL_ARB_texture_rgb10_a2ui GL_ARB_texture_storage GL_ARB_texture_swizzle GL_ARB_timer_query GL_ARB_transform_feedback2 GL_ARB_transform_feedback3 GL_ARB_vertex_attrib_64bit GL_ARB_vertex_type_2_10_10_10_rev GL_ARB_viewport_array GL_EXT_debug_label GL_EXT_debug_marker GL_EXT_framebuffer_multisample_blit_scaled GL_EXT_texture_compression_s3tc GL_EXT_texture_filter_anisotropic GL_EXT_texture_sRGB_decode GL_APPLE_client_storage GL_APPLE_container_object_shareable GL_APPLE_flush_render GL_APPLE_object_purgeable GL_APPLE_rgb_422 GL_APPLE_row_bytes GL_APPLE_texture_range GL_ATI_texture_mirror_once GL_NV_texture_barrier Disabled Extensions Window system binding vendor Window system binding version Window system binding extensions Direct rendering Yes Reset notification strategy 0x0000 GPU process crash count 0 Compositor Information Tile Update Mode Zero-copy Partial Raster Enabled GpuMemoryBuffers Status ATC Software only ATCIA Software only DXT1 Software only DXT5 Software only ETC1 Software only R_8 GPU_READ_CPU_READ_WRITE, GPU_READ_CPU_READ_WRITE_PERSISTENT RG_88 Software only BGR_565 Software only RGBA_4444 Software only RGBX_8888 Software only RGBA_8888 GPU_READ, SCANOUT BGRX_8888 GPU_READ, SCANOUT BGRA_8888 GPU_READ, SCANOUT, GPU_READ_CPU_READ_WRITE, GPU_READ_CPU_READ_WRITE_PERSISTENT YVU_420 Software only YUV_420_BIPLANAR GPU_READ_CPU_READ_WRITE, GPU_READ_CPU_READ_WRITE_PERSISTENT UYVY_422 GPU_READ_CPU_READ_WRITE, GPU_READ_CPU_READ_WRITE_PERSISTENT Log Messages [2347:775:0106/123049.217838:ERROR:latency_info.cc(163)] : GpuCommandBufferStub::OnAsyncFlush, LatencyInfo vector size 456 is too big. Issue is also observable in the Canary channel (Chrome 57)
,
Jan 6 2017
I'm seeing intermittent failure setting properties on "object" elements. JW Player depends on methods it adds to the object element for communication between Flash and JavaScript. When object.prop = x does not actually update object.prop, this breaks our player when configured to play using Flash.
We have taken steps to attempt an embed with flash and if there is no "ack" ignore the developers choice to force Flash playback, however the exception thrown in this example breaks this failsafe.
Reproduction Steps: Here is a block of code that reproduces the issue. I've found that results vary depending on the page's protocol or number of times this block is executed. The page should be set to "Allow" or "Ask" for Flash plugin playback.
Expected: The code should run without any exceptions.
Actual: Intermittently (more that not with 56.0.2924.51) setting properties on the object element fails. Attempting to access the property returns the previous value (undefined or in some cases where it was set previously, it is not updated)
"Uncaught TypeError: swf.on is not a function"
var swf = document.createElement('object');
swf.setAttribute('type', 'application/x-shockwave-flash');
swf.setAttribute('data', '//ssl.p.jwpcdn.com/player/v/7.8.7/jwplayer.loader.swf');
swf.setAttribute('width', '100%');
swf.setAttribute('height', '100%');
swf.setAttribute('id', 'jwplayer');
swf.setAttribute('name', 'jwplayer');
document.body.appendChild(swf);
swf.className = 'jw-swf';
swf.style.display = 'block';
swf.style.position = 'absolute';
swf.style.left = 0;
swf.style.right = 0;
swf.style.top = 0;
swf.style.bottom = 0;
// Add custom properties
swf['on'] = function() {};
swf['once'] = function() {};
swf.trigger = function() {};
// Attempt to envoke functions added to object element
swf.on();
swf.once();
swf.trigger();
,
Jan 6 2017
The problem is visible here as well. https://erikdonohoo.github.io/jwplayer-flash-player-issue/ On Mac OSX 10.12.2 and Chrome 56
,
Jan 6 2017
,
Jan 6 2017
I could not repro this given the PBS & support.jswplayer examples on Win10+chrome canary 57.0.2970.0 (Official Build) canary (64-bit). videos both played fine. For the erikdonohoo.github.io example (same OS+chrome canary), I get the "Error loading player : no playable sources found". @danny, rob, erik : can you check chrome://flags/#prefer-html-over-flash and set to Enabled? this is the HTML5-by-default feature (see also : https://blog.chromium.org/2016/12/roll-out-plan-for-html5-by-default.html). If I "Allow" Flash for erik's example (after HTML5 by default is enabled) I get a JWPlayer timeout error : "Setup Timeout Error: setup took longer than 30 seconds to complete". what does JWPlayer do in the case that Flash is unavailable (doesn't appear in navigator.plugins or navigator.mimetypes)? moving this back to untriaged as I cannot repro the issues outside of erik's site. Even when I disable HTML5 by default (so flash is available), i get the same setup timeout error on erik's example.
,
Jan 6 2017
That timeout issue is the issue. You will notice a console error from jwplayer trying to access the object tag and it isn't there to work with
,
Jan 6 2017
,
Jan 6 2017
JW Player does not use Flash when it is not available. "Error loading player : no playable sources found" means that an HTML5 compatible source was not provided. Our player works as expected with HTML5-by-default - no bugs there.
Developers can configure JW Player to use Flash if available. Since Flash can still be blocked when installed (navigator.plugins reports a Flash version, but the plugin will not run) we still require the swf to execute a JavaScript callback before JW Player considers Flash "available".
The bug Danny and I are concerned with here is a new issue with setting properties on object elements. The behavior I described ("Uncaught TypeError: swf.on is not a function") prevents our flash swf from calling back to JavaScript.
I came up with a workaround for the next release of JW Player https://github.com/jwplayer/jwplayer/pull/1686, but there will still be many sites hosting their own jwplayer.js that will be impacted by this issue, so we're hoping the cause can be identified and fix made before 56 is released.
I am able to reproduce the problem in the console. If you'd like a standalone test page (so you don't hit CORS issues loading the swf) we can put one up.
var swf = document.createElement('object');
swf.setAttribute('type', 'application/x-shockwave-flash');
swf.setAttribute('data', '//ssl.p.jwpcdn.com/player/v/7.8.7/jwplayer.loader.swf');
document.body.appendChild(swf);
swf['on'] = function() {};
console.log(swf.on);
// logs "undefined"
swf.on();
// Uncaught TypeError: swf.on is not a function
,
Jan 6 2017
What @rob mentioned is the issue at hand. Even with chrome://flags/#prefer-html-over-flash set to Enabled the issue is reproducible.
,
Jan 6 2017
Here's an isolated test case: http://demo.jwplayer.com/flash-object/ `swf.embedCallback` is never set and can never be invoked by Flash.
,
Jan 6 2017
> If I "Allow" Flash for erik's example (after HTML5 by default is enabled) I get a JWPlayer timeout error : "Setup Timeout Error: setup took longer than 30 seconds to complete". That's the bug, if you have a look in the console you'll see "Uncaught TypeError: x.once is not a function"
,
Jan 10 2017
Unable to reproduce the issue on Mac 10.12.2 , Linux Ubuntu 14.04 & windows 10 with latest Canary version-57.0.2977.0 ,reported beta version-56.0.2924.51 & observed below: 1.URL->http://www.pbs.org/video/2365921903/ ---> loaded successfully. 2."Failed to set property on object element: [object HTMLObjectElement]" message observed in Devtools->Console as per the steps mentioned in Comment#9,10 & 11. Note: Above error not observed in Chrome stable version-55.0.2883.87 Please find the attached screencast for reference . Could you please check the issue in latest Canary and update the thread with clear reproduce steps if still issue persists. Thanks in advance.
,
Jan 10 2017
I can still reproduce the issue on the latest chrome canary. I'm not sure why you aren't seeing it. This is on Mac 10.12.2 having just downloaded and installed Chrome Canary. I also show the issue still occurring on Chrome 56 as well
,
Jan 10 2017
@jmukthavaram The page should be set to "Allow" or "Ask" for Flash plugin playback. "Failed to set property on object element" is not an issue in stable. Only in beta and canary.
,
Jan 11 2017
@jmukthavaram were you able to verify on Beta or Canary?
,
Jan 11 2017
Here is an additional example: http://www.wwe.com/videos/john-cena-vs-baron-corbin-smackdown-live-jan-10-2017
,
Jan 11 2017
Hey Prudhvi/ Krishna, Could you help me find someone to bisect this issue? It looks like a potential regression over 54.
,
Jan 12 2017
I am working on the Bisect will update soon.
,
Jan 13 2017
Please find the bisect range below : Last good build : 56.0.2924.36 First bad build : 56.0.2924.37 Change log : https://chromium.googlesource.com/chromium/src/+log/56.0.2924.36..56.0.2924.37?pretty=fuller&n=10000
,
Jan 13 2017
,
Jan 13 2017
The issue is present on Windows 10(Not on Windows 7), Mac(10.12). Note: Since the machine where I was able to reproduce the bug are non corp machine and I can't perform the per-cl bisect. The range which provided has 13 Cl's can't figure out which would have caused this to issue.
,
Jan 13 2017
,
Jan 16 2017
Able to reproduce the issue on Mac 10.12.2 and Win-10 using chrome reported version #56.0.2924.51 but unable to reproduce using latest canary #57.0.2982.0. Issue is not reproducible on Linux. Reverse Bisect Information: ============================ Good build: 57.0.2980.0 Revision(443474) Bad Build : 57.0.2979.0 Revision(443120) Change Log URL: https://chromium.googlesource.com/chromium/src/+log/f651acf25aaea54d6aed2488c8237214f1a2b38f..314c0c8fcb9c97c7588a2302df4df7b4b0097c0c From the above change log suspecting below change Review url: https://codereview.chromium.org/2610393003 yunchao.he@ - Could you please check whether this is caused with respect to your change, if not please help us in assigning it to the right owner. Thanks...!!
,
Jan 17 2017
We will be affected by this bug too once version 56 build is released. Will this be fixed in the version 56 build?
,
Jan 17 2017
Mo, could you please confirm whether the patch mentioned in #23 would have fixed the issue in canary? If yes, could we merge the Cl to M56.
,
Jan 17 2017
I don't know what tool the reverse bisect used, but the result is wrong. I did another reverse bisect, and last bad 443211, first good, 443216. I think crrev.com/443212 fixed the bug.
,
Jan 17 2017
Thanks Mo. Looks like - https://chromium.googlesource.com/chromium/src/+/8a4fe2fc28bb45f58445d5802057efdafbe7719b seems to be the Cl which fixed the issue in canary. This is already patched to M56 - https://chromium.googlesource.com/chromium/src/+/45f356d8a3036166754170d36019bb25ab128328 We will verify in today's RC.
,
Jan 18 2017
Verified the fix on Mac 10.12.2 and Win-10 using Chrome beta version #56.0.2924.67 as per the comment #0. Observed that the video player loaded with flash content and played without any issues. Hence, the fix is working as expected. Attaching the screencast for reference Adding the verified labels. Thanks...!!
,
Jan 18 2017
Thanks, tagging as verified.
,
Jan 18 2017
[Auto-generated comment by a script] We noticed that this issue is targeted for M-56; it appears the fix may have landed after branch point, meaning a merge might be required. Please confirm if a merge is required here - if so add Merge-Request-56 label, otherwise remove Merge-TBD label. Thanks.
,
Jan 18 2017
,
Jan 18 2017
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by da...@jwplayer.com
, Jan 6 2017