Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(18)

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: build/config/features.gni

Issue 1577433004: Flip proprietary codecs to false for Android chromium builds. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Remove mp4. Fix comment. Created 4 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« build/common.gypi ('K') | « build/common.gypi ('k') | chrome/android/javatests/src/org/chromium/chrome/browser/contextmenu/ContextMenuTest.java » ('j') | media/base/android/BUILD.gn » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/config/features.gni
diff --git a/build/config/features.gni b/build/config/features.gni
index 8efa7cf3c5d32e4bd30b837324d01635dee76f87..8f0be39d0aee052244e7cb96be066fa91729cd83 100644
--- a/build/config/features.gni
+++ b/build/config/features.gni
@@ -47,10 +47,8 @@ declare_args() {
enable_media_router = !is_ios
# Enables proprietary codecs and demuxers; e.g. H264, MOV, AAC, and MP3.
ddorwin 2016/03/10 01:46:38 Oh, now I see the source. Suggest cleaning it up h
DaleCurtis 2016/03/10 02:00:02 Done.
- # Android OS includes support for proprietary codecs regardless of building
- # Chromium or Google Chrome. We also ship Google Chrome and Chromecast with
- # proprietary codecs.
- proprietary_codecs = is_android || is_chrome_branded || is_chromecast
+ # We always build Google Chrome and Chromecast with proprietary codecs.
+ proprietary_codecs = is_chrome_branded || is_chromecast
enable_configuration_policy = !is_ios
« build/common.gypi ('K') | « build/common.gypi ('k') | chrome/android/javatests/src/org/chromium/chrome/browser/contextmenu/ContextMenuTest.java » ('j') | media/base/android/BUILD.gn » ('J')

Powered by Google App Engine
This is Rietveld 408576698