Few Videos on cnn page is not playing |
||||||||||||
Issue descriptionApp Version: 61.0.3160.0 Canary iOS Version: 10.3.2 Only Device: iPhone 6, iPad URL: http://www.cnn.com/videos/politics/2017/07/18/trump-very-disappointed-health-bill-failure-sot.cnn Steps to reproduce: 1. Launch the app 2. Navigate to the URL 3. Play the videos in the URL Observed results: Notice that some of the Videos are not playing Note: This video play works fine in Stable, Beta build. No download progress bar is showing up. This issue appears only in Canary & Dev build Expected results: All Videos should be playing at all times Number of times you were able to reproduce: 5/5 Bug reproducible after clean install: Yes Bug reproducible after clearing cache and cookies: Yes Bug reproducible on Chrome Mobile on Android: Bug reproducible on Safari/Firefox: Firefox: no, Safari: no Bug reproducible on current stable build (App Version, iOS Version): No, M59 Bug reproducible on the current beta channel build (App Version, iOS Version): No, M60 Autofill bugs: Bug reproducible on Chrome desktop? No Type-bug-regression? Yes Link to video/image: https://drive.google.com/a/google.com/file/d/0BwSBFDzHIX-mdlFuLW9RMU1UUjg/view
,
Jul 19 2017
eugenebut@ could you PTAL, we're not completely sure the component is right.
,
Jul 19 2017
jdhakshinamoor@ Could you please bisect good and bad revision and update this bug. Revision can be found on chrome://version page
,
Jul 19 2017
Good Build: Google Chrome 61.0.3156.0 canary (64-bit) Revision 7a4241c698a71a5cc10ec6135060f4d99af2b96c-refs/heads/master@{#486205} Bad Build: Google Chrome 61.0.3158.0 canary (64-bit) Revision 97e40196944c926a7b805cdfe67611832da7d457-refs/heads/master@{#486952} Test Device - iPad Pro , iOS 10.3.2 & iPhone 6+, iOS 10.3.2
,
Jul 21 2017
NSAllowsArbitraryLoadsInWebContent is a culprit. Reverting https://chromium-review.googlesource.com/568791
,
Jul 21 2017
,
Jul 21 2017
[Auto-generated comment by a script] We noticed that this issue is targeted for M-61; 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-61 label, otherwise remove Merge-TBD label. Thanks.
,
Jul 21 2017
,
Jul 22 2017
Your change meets the bar and is auto-approved for M61. Please go ahead and merge the CL to branch 3163 manually. Please contact milestone owner if you have questions. Owners: amineer@(Android), cmasso@(iOS), ketakid @(ChromeOS), govind@(Desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jul 24 2017
This change got into the branch
,
Jul 26 2017
Change is not on the branch.
,
Jul 27 2017
Verified on latest beta build 61.0.3163.15 on iPad Air(iOS 10.3.3) and iPhone 6s plus(iOS 10.3.1) Followed the steps mentioned in Comment#0, All Videos are playing as expected. Link to the video: https://drive.google.com/a/google.com/file/d/0B8Cek8RsDbF8Q0MzM0ZycVllWmM/view?usp=sharing
,
Jul 31 2017
|
||||||||||||
►
Sign in to add a comment |
||||||||||||
Comment 1 by jdhakshinamoor@chromium.org
, Jul 18 2017