Casting the first video from abc13.com/live fails |
|||||||||
Issue descriptionVersion: 49.0.2623.105 (Official Build) (32-bit) and above all the way to the Canary build OS: Android 6.0.1; Nexus 6P Build/MTC19R What steps will reproduce the problem? (1) Go to abc13.com/live (2) Play the first video on the page (3) Cast the video using the Cast icon in the media controls What is the expected output? The video playback continues on the Cast device What do you see instead? The Chrome player is loaded but then stops and the Cast device goes back to the background image. Chrome switches back to local playback. See internal issue at b/28290577
,
Apr 22 2016
I also found the following issues and changes relevant: https://bugs.chromium.org/p/chromium/issues/detail?id=366590 https://bugs.chromium.org/p/chromium/issues/detail?id=358157 https://chrome-internal-review.googlesource.com/#/c/161655/ https://chrome-internal-review.googlesource.com/#/c/159375/ https://chrome-internal-review.googlesource.com/#/c/158285/
,
Apr 25 2016
,
Apr 26 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/e358d10bb42b0699c2db6276b57d6b9963be4b24 commit e358d10bb42b0699c2db6276b57d6b9963be4b24 Author: avayvod <avayvod@chromium.org> Date: Tue Apr 26 09:48:23 2016 [Android,MediaFling] Fix the encoding issue with already encoded URLs for abc13.com/live Also some cleanup of error handling, url type conversions and media type detection, extend the UMA to cover more failure cases. BUG= 605544 TEST=unit tests + avayvod.github.io/mediaflingtest.html Review URL: https://codereview.chromium.org/1907413002 Cr-Commit-Position: refs/heads/master@{#389735} [modify] https://crrev.com/e358d10bb42b0699c2db6276b57d6b9963be4b24/chrome/android/java/src/org/chromium/chrome/browser/media/remote/DefaultMediaRouteController.java [modify] https://crrev.com/e358d10bb42b0699c2db6276b57d6b9963be4b24/chrome/android/java/src/org/chromium/chrome/browser/media/remote/MediaUrlResolver.java [modify] https://crrev.com/e358d10bb42b0699c2db6276b57d6b9963be4b24/chrome/android/junit/src/org/chromium/chrome/browser/media/remote/MediaUrlResolverTest.java [modify] https://crrev.com/e358d10bb42b0699c2db6276b57d6b9963be4b24/chrome/browser/media/android/remote/remote_media_player_bridge.cc [modify] https://crrev.com/e358d10bb42b0699c2db6276b57d6b9963be4b24/tools/metrics/histograms/histograms.xml
,
Apr 26 2016
Ramine, could you please verify the fix?
,
Apr 26 2016
Your change meets the bar and is auto-approved for M51 (branch: 2704)
,
Apr 28 2016
Tested on '52.0.2718.2' Dev On LG G3/LRX21T -> Pass Nexus 5X/NRD26 -> Pass Othere sites tested: m.youtube.com dailymotion bbc.com
,
Apr 28 2016
Other cast enabled video's tested with 52.0.2718.2' & '50.0.2661.94' LG G3/LRX21T Samsung Galaxy S7/MMB29M Nexus 5X/NRD28 www.cnn.com m.youtube.com http://videojs.com/ http://html5-demos.appspot.com/static/fullscreen.html https://developer.apple.com/streaming/examples/basic-stream.html http://www.metacafe.com/ http://people.mozilla.org/~rgiles/2013/demo.html https://www.facebook.com/ http://www.nytimes.com/ www.cnet.com
,
Apr 29 2016
,
Apr 30 2016
This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible! If all merges have been completed, please remove any remaining Merge-Approved labels from this issue. Thanks for your time! To disable nags, add the Disable-Nags label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
May 3 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/a8de7f8d6820d42bf0a476fd3029dd0cf0ae9c36 commit a8de7f8d6820d42bf0a476fd3029dd0cf0ae9c36 Author: Anton Vayvod <avayvod@google.com> Date: Tue May 03 10:07:23 2016 [Android,MediaFling] Fix the encoding issue with already encoded URLs for abc13.com/live Also some cleanup of error handling, url type conversions and media type detection, extend the UMA to cover more failure cases. BUG= 605544 TEST=unit tests + avayvod.github.io/mediaflingtest.html Review URL: https://codereview.chromium.org/1907413002 Cr-Commit-Position: refs/heads/master@{#389735} (cherry picked from commit e358d10bb42b0699c2db6276b57d6b9963be4b24) Review URL: https://codereview.chromium.org/1940273002 . Cr-Commit-Position: refs/branch-heads/2704@{#350} Cr-Branched-From: 6e53600def8f60d8c632fadc70d7c1939ccea347-refs/heads/master@{#386251} [modify] https://crrev.com/a8de7f8d6820d42bf0a476fd3029dd0cf0ae9c36/chrome/android/java/src/org/chromium/chrome/browser/media/remote/DefaultMediaRouteController.java [modify] https://crrev.com/a8de7f8d6820d42bf0a476fd3029dd0cf0ae9c36/chrome/android/java/src/org/chromium/chrome/browser/media/remote/MediaUrlResolver.java [modify] https://crrev.com/a8de7f8d6820d42bf0a476fd3029dd0cf0ae9c36/chrome/android/junit/src/org/chromium/chrome/browser/media/remote/MediaUrlResolverTest.java [modify] https://crrev.com/a8de7f8d6820d42bf0a476fd3029dd0cf0ae9c36/chrome/browser/media/android/remote/remote_media_player_bridge.cc [modify] https://crrev.com/a8de7f8d6820d42bf0a476fd3029dd0cf0ae9c36/tools/metrics/histograms/histograms.xml
,
May 3 2016
,
May 4 2016
Verified with Chrome Beta '51.0.2704.36' (Samsung Galaxy S7/MMB29M) |
|||||||||
►
Sign in to add a comment |
|||||||||
Comment 1 by avayvod@chromium.org
, Apr 21 2016