Please find logs and video @ http://go/chrome-androidlogs1/6/634920

Don't have this device, so can't test this. Are you saying this exists from the very first M54 build, but not the last M53 build before M54 was cut? If so the change log should just be the delta between those versions.

We have test device in MTV and we are able to repro the issue.It is broken in M54 from the 54.0.2790.2 Build.

qinmin@ - I will get the device to debug

So 54.0.2789.* does not have the issue?

i saw "MediaEvent: PIPELINE_ERROR decoder: not supported" log when playing the video

Hmm. This is a JB device so it's blacklisted from the spitzer path on 52 and 53 but not 54. I'll go ahead and blacklist for 54 and mark this as another blocker for enabling JB.

It would be good to figure out what the problem is though. Is it just h264 that's not working or vp9 as well? (test page: http://storage.googleapis.com/watk/v)

Tried the h.264 video on the test page, and it works fine on 54.0.2790.2

On YouTube.com where it's failing, what does chrome://media-internals say about the video_decoder_name and video_codec?

For youtube, the listed video_codec_name is mpeg4, video format is mpeg4 format 2
For the test h264 video, the video_codec_name is h264 

Wat? +strobe@ is YT serving mpeg4 content in any circumstance? This must be src= serving too, as that's definitely not supported in MSE.

Format 17, 3GPP. It's not an MSE format for sure. I'm not sure why MSE wouldn't be used, unless this device is falling back to Blazer Player - which might be happening. What's the exact UA?

UA should just be Chrome 52. 3gpp is no longer supported on mweb, so it shouldn't be served to Chrome 52+ (which only supports the normal Chrome web formats).

I can't debug further without an exact user agent.

qinmin: can you grab the UA from chrome://version for strobe? Thanks!

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/68770b9847ea46d6136c0a4294d33ee48911cb74

commit 68770b9847ea46d6136c0a4294d33ee48911cb74
Author: Alex Mineer <amineer@google.com>
Date: Mon Aug 08 16:06:42 2016

Revert "Re-enable the unified media pipeline for JellyBean"

This reverts commit 1580687683f0814bfb8a6238b9b9243fff335ef9.

BUG= 634920 

Cr-Commit-Position: refs/branch-heads/2823@{#2}
Cr-Branched-From: c6b7050c321314686730fb679a7122341f16c7c2-refs/heads/master@{#410278}

[modify] https://crrev.com/68770b9847ea46d6136c0a4294d33ee48911cb74/media/base/media.cc

The UA is:
Mozilla/5.0 (Linux; Android 4.1.2; GT-S7262 Build/JZO54K) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/54.0.2790.2 Mobile Safari/537.36

strobe: To correct my comment in c#13: mweb > jellybean does not support 3gpp as of m52. Hopefully the same will be true of jellybean in the M54/M55 time frame, but we continue to see higher rates of broken playbacks using MediaCodec on these older devices :/

Verified in M54-54.0.2823.2 build with Samsung Galaxy STAR PLUS(GT-S7262)/JZ054K

Re-opening since the original issue is not fixed.  kravula@ had verified fixed with the revert.

A revert landed on trunk as well, so this should be fixed globally, please reopen if it persists.

-rvg since it doesn't matter

M54 issue is fixed, 

But Youtube videos are not playing in M55
Issue is still reproducible on latest 55.0.2882.3

kgnaidu@, could you please provide the output of chrome://media-internals after trying to play it?

Youtube videos are playing when enable the flag - Disable the unified media pipeline Android

Samsung Galaxy STAR PLUS -(GT-S782)/JZ054K/55.0.2883.6

attached chrome://Media-internals and logs 

http://go/chrome-androidlogs1/6/634920

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/9f03272c365e359ef5071c6c69fa1bca5e8ea3cb

commit 9f03272c365e359ef5071c6c69fa1bca5e8ea3cb
Author: Alex Mineer <amineer@chromium.org>
Date: Mon Oct 10 23:41:02 2016

Blacklist Samsung Galaxy Star Pro (GT-S7262) for Spitzer.

Speculative fix for associated bug, on branch only

R=liberato,watk
BUG= 634920 

Review URL: https://codereview.chromium.org/2405053002 .

Cr-Commit-Position: refs/branch-heads/2883@{#27}
Cr-Branched-From: 614d31daee2f61b0180df403a8ad43f20b9f6dd7-refs/heads/master@{#423768}

[modify] https://crrev.com/9f03272c365e359ef5071c6c69fa1bca5e8ea3cb/media/base/android/media_codec_util.cc

I was able to reproduce this on 2 other devices with 55.0.2883.9. YouTube videos do not play until the "Disable the unified media pipeline on Android" flag is enabled. The devices are:

- Samsung Galaxy Star (GT-S5282) / JZO54K
- Samsung Galaxy Win (GT-I8552) / JZO54K

I'm not going to block dev on this any longer given the limited scope of devices, but I will block beta - not playing YouTube is unacceptable IMO.  PTAL ASAP, we're trying to ship M55 to dev Oct 20 with the final build being cut very soon.

i don't believe that i have either of these devices to test with, so our options are somewhat limited.  i'll take a look.

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/4a968f2146a68c3eeec767549ba4cef27a9c117a

commit 4a968f2146a68c3eeec767549ba4cef27a9c117a
Author: liberato <liberato@chromium.org>
Date: Fri Oct 14 21:20:35 2016

Blacklist Samsung Galaxy Star and Samsung Galaxy Win

BUG= 634920 
TBR=watk@chromium.org

Review-Url: https://codereview.chromium.org/2421963002
Cr-Commit-Position: refs/heads/master@{#425467}

[modify] https://crrev.com/4a968f2146a68c3eeec767549ba4cef27a9c117a/media/base/android/media_codec_util.cc

Your change meets the bar and is auto-approved for M55 (branch: 2883)

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/70214f9fd48926759cfaa81a30857a78652841e1

commit 70214f9fd48926759cfaa81a30857a78652841e1
Author: liberato@chromium.org <liberato@chromium.org>
Date: Fri Oct 14 21:50:06 2016

Blacklist Samsung Galaxy Star and Samsung Galaxy Win

BUG= 634920 
TBR=watk@chromium.org

Review-Url: https://codereview.chromium.org/2421963002
Cr-Commit-Position: refs/heads/master@{#425467}
(cherry picked from commit 4a968f2146a68c3eeec767549ba4cef27a9c117a)

Review URL: https://codereview.chromium.org/2420833003 .

Cr-Commit-Position: refs/branch-heads/2883@{#120}
Cr-Branched-From: 614d31daee2f61b0180df403a8ad43f20b9f6dd7-refs/heads/master@{#423768}

[modify] https://crrev.com/70214f9fd48926759cfaa81a30857a78652841e1/media/base/android/media_codec_util.cc

Hoping we're now fixed again, holler if we find any more devices or see any more issues related to media playback on JB (or re-open if more work is required).

Youtube videos are playing, Works as per expected behavior
Tested on 55.0.2883.18

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/9f03272c365e359ef5071c6c69fa1bca5e8ea3cb

commit 9f03272c365e359ef5071c6c69fa1bca5e8ea3cb
Author: Alex Mineer <amineer@chromium.org>
Date: Mon Oct 10 23:41:02 2016

Blacklist Samsung Galaxy Star Pro (GT-S7262) for Spitzer.

Speculative fix for associated bug, on branch only

R=liberato,watk
BUG= 634920 

Review URL: https://codereview.chromium.org/2405053002 .

Cr-Commit-Position: refs/branch-heads/2883@{#27}
Cr-Branched-From: 614d31daee2f61b0180df403a8ad43f20b9f6dd7-refs/heads/master@{#423768}

[modify] https://crrev.com/9f03272c365e359ef5071c6c69fa1bca5e8ea3cb/media/base/android/media_codec_util.cc

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/70214f9fd48926759cfaa81a30857a78652841e1

commit 70214f9fd48926759cfaa81a30857a78652841e1
Author: liberato@chromium.org <liberato@chromium.org>
Date: Fri Oct 14 21:50:06 2016

Blacklist Samsung Galaxy Star and Samsung Galaxy Win

BUG= 634920 
TBR=watk@chromium.org

Review-Url: https://codereview.chromium.org/2421963002
Cr-Commit-Position: refs/heads/master@{#425467}
(cherry picked from commit 4a968f2146a68c3eeec767549ba4cef27a9c117a)

Review URL: https://codereview.chromium.org/2420833003 .

Cr-Commit-Position: refs/branch-heads/2883@{#120}
Cr-Branched-From: 614d31daee2f61b0180df403a8ad43f20b9f6dd7-refs/heads/master@{#423768}

[modify] https://crrev.com/70214f9fd48926759cfaa81a30857a78652841e1/media/base/android/media_codec_util.cc

[Automated comment] removing mislabelled merge-merged-2840

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/184decd0afefe8f8b666b6923ab1d7c38865544a

commit 184decd0afefe8f8b666b6923ab1d7c38865544a
Author: liberato <liberato@chromium.org>
Date: Mon Nov 14 21:35:21 2016

Blacklist Samsung Galaxy Star Pro (GT-S7262) for Spitzer.

BUG= crbug.com/634920 
TBR=watk@chromium.org

Review-Url: https://codereview.chromium.org/2409843002
Cr-Commit-Position: refs/heads/master@{#431916}

[modify] https://crrev.com/184decd0afefe8f8b666b6923ab1d7c38865544a/media/base/android/media_codec_util.cc