New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 727902 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jun 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 1
Type: Feature



Sign in to add a comment

Enable Vp09 codec string in WebM by default

Project Member Reported by chcunningham@chromium.org, May 30 2017

Issue description

vp09 codec string in WebM is already implemented under run time flag 

--enable-new-vp9-codec-string

We intend to ship this by default. 

https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/Ado2yvWol38

Separate  issue 726759  tracks default-enable for the same string (and vp9  in general) in MP4.

 
Project Member

Comment 2 by bugdroid1@chromium.org, Jun 7 2017

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

commit 6d43fc8beaa41811acc6c77a56ab2174ca283788
Author: chcunningham <chcunningham@chromium.org>
Date: Wed Jun 07 20:22:14 2017

Enable vp09 codec string in WebM by default

Removes need for command line switch --enable-new-vp9-codec-string

BUG= 727902 

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

[modify] https://crrev.com/6d43fc8beaa41811acc6c77a56ab2174ca283788/chrome/browser/media/encrypted_media_supported_types_browsertest.cc
[modify] https://crrev.com/6d43fc8beaa41811acc6c77a56ab2174ca283788/content/browser/media/media_canplaytype_browsertest.cc
[modify] https://crrev.com/6d43fc8beaa41811acc6c77a56ab2174ca283788/content/browser/media/media_capabilities_browsertest.cc
[modify] https://crrev.com/6d43fc8beaa41811acc6c77a56ab2174ca283788/content/browser/renderer_host/render_process_host_impl.cc
[modify] https://crrev.com/6d43fc8beaa41811acc6c77a56ab2174ca283788/content/renderer/render_thread_impl.cc
[modify] https://crrev.com/6d43fc8beaa41811acc6c77a56ab2174ca283788/media/base/decode_capabilities.cc
[modify] https://crrev.com/6d43fc8beaa41811acc6c77a56ab2174ca283788/media/base/media.cc
[modify] https://crrev.com/6d43fc8beaa41811acc6c77a56ab2174ca283788/media/base/media.h
[modify] https://crrev.com/6d43fc8beaa41811acc6c77a56ab2174ca283788/media/base/media_switches.cc
[modify] https://crrev.com/6d43fc8beaa41811acc6c77a56ab2174ca283788/media/base/media_switches.h
[modify] https://crrev.com/6d43fc8beaa41811acc6c77a56ab2174ca283788/media/base/mime_util_internal.cc
[modify] https://crrev.com/6d43fc8beaa41811acc6c77a56ab2174ca283788/media/base/mime_util_unittest.cc

Labels: Merge-Request-60
Labels: -Merge-Request-60
Labels: Merge-Request-60
Project Member

Comment 6 by sheriffbot@chromium.org, Jun 8 2017

Labels: -Merge-Request-60 Hotlist-Merge-Approved Merge-Approved-60
Your change meets the bar and is auto-approved for M60. Please go ahead and merge the CL to branch 3112 manually. Please contact milestone owner if you have questions.
Owners: amineer@(Android), cmasso@(iOS), josafat@(ChromeOS), bustamante@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 7 by bugdroid1@chromium.org, Jun 8 2017

Labels: -merge-approved-60 merge-merged-3112
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/384201c1babf1efffbf2db9760df832f6fd77c18

commit 384201c1babf1efffbf2db9760df832f6fd77c18
Author: Chris Cunningham <chcunningham@chromium.org>
Date: Thu Jun 08 20:52:42 2017

Enable vp09 codec string in WebM by default

Removes need for command line switch --enable-new-vp9-codec-string

BUG= 727902 

Review-Url: https://codereview.chromium.org/2917463002
Cr-Original-Commit-Position: refs/heads/master@{#477749}
Review-Url: https://codereview.chromium.org/2926393002 .
Cr-Commit-Position: refs/branch-heads/3112@{#267}
Cr-Branched-From: b6460e24cf59f429d69de255538d0fc7a425ccf9-refs/heads/master@{#474897}

[modify] https://crrev.com/384201c1babf1efffbf2db9760df832f6fd77c18/chrome/browser/media/encrypted_media_supported_types_browsertest.cc
[modify] https://crrev.com/384201c1babf1efffbf2db9760df832f6fd77c18/content/browser/media/media_canplaytype_browsertest.cc
[modify] https://crrev.com/384201c1babf1efffbf2db9760df832f6fd77c18/content/browser/media/media_capabilities_browsertest.cc
[modify] https://crrev.com/384201c1babf1efffbf2db9760df832f6fd77c18/content/browser/renderer_host/render_process_host_impl.cc
[modify] https://crrev.com/384201c1babf1efffbf2db9760df832f6fd77c18/content/renderer/render_thread_impl.cc
[modify] https://crrev.com/384201c1babf1efffbf2db9760df832f6fd77c18/media/base/decode_capabilities.cc
[modify] https://crrev.com/384201c1babf1efffbf2db9760df832f6fd77c18/media/base/media.cc
[modify] https://crrev.com/384201c1babf1efffbf2db9760df832f6fd77c18/media/base/media.h
[modify] https://crrev.com/384201c1babf1efffbf2db9760df832f6fd77c18/media/base/media_switches.cc
[modify] https://crrev.com/384201c1babf1efffbf2db9760df832f6fd77c18/media/base/media_switches.h
[modify] https://crrev.com/384201c1babf1efffbf2db9760df832f6fd77c18/media/base/mime_util_internal.cc
[modify] https://crrev.com/384201c1babf1efffbf2db9760df832f6fd77c18/media/base/mime_util_unittest.cc

Status: Fixed (was: Started)

Sign in to add a comment