| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
We are not advertising audio decoders (including Vorbis) as secure decoders
[1], hence there's no need to put any audio codec in the CDM supported codec
list.
Also Vorbis support will be dropped from the newer version CDM. See BUG for
details. With that change, Vorbis will also be removed from the component
manifest, which is used to populate the CDM supported codec list for component
updated CDM.
[1] https://code.google.com/p/chromium/codesearch#chromium/src/chrome/renderer/media/chrome_key_systems.cc&l=185
TBR=thestig@chromium.org
BUG=586354
Review URL: https://codereview.chromium.org/1690353002
Cr-Commit-Position: refs/heads/master@{#375257}
|
|
|
|
|
|
|
|
|
|
|
|
| |
This should have been removed as part of
https://chromium.googlesource.com/chromium/src/+/269a17f7ed102b5f4cdeb0d676b107cbd7fd7402
TBR=thestig@chromium.org
BUG=349185
Review URL: https://codereview.chromium.org/1691203002
Cr-Commit-Position: refs/heads/master@{#375218}
|
|
|
|
|
|
|
|
|
|
|
| |
The new name is "Widevine Content Decryption Module" which is more user
friendly.
BUG=579145
Review URL: https://codereview.chromium.org/1609013002
Cr-Commit-Position: refs/heads/master@{#370190}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
It's needed by clock_gettime() on glibc < 2.17, and that function is
called in various places in the codebase. This library is part of glibc,
so just put in the default toolchain libs so we get it everywhere.
It fixes the following build error:
gn gen out_gn_ozone --args='use_ozone=true toolkit_views=false is_component_build=true'
ninja -C out_gn_ozone content_shell
../ui/gl/sync_control_vsync_provider.cc:70: error: undefined reference to
'clock_gettime'
../ui/gl/sync_control_vsync_provider.cc:71: error: undefined reference to
'clock_gettime'
Review URL: https://codereview.chromium.org/1565983002
Cr-Commit-Position: refs/heads/master@{#368383}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The recent formatter alphebetizing change is causing a lot of noise in
code reviews. Figured it'd be worth a clean-up CL.
Exact command I ran:
find . -name "*.gn*" -exec gn format --in-place "{}" \;
TBR=ddorwin@chromium.org
BUG=554928
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1464873002
Cr-Commit-Position: refs/heads/master@{#360891}
|
|
|
|
|
|
|
|
|
|
|
|
| |
As the current cdm_adapter is specific to ppapi, rename it to make it
clearer. This also moves some of the generic files out of the ppapi
folder so that they can be used by other implementations.
TEST=compiles
Review URL: https://codereview.chromium.org/1402413005
Cr-Commit-Position: refs/heads/master@{#356196}
|
|
|
|
|
|
|
|
|
|
| |
The latter is the canonical name, however both are supported.
TBR=xhwang@chromium.org, mkwst@chromium.org
Review URL: https://codereview.chromium.org/1335313003
Cr-Commit-Position: refs/heads/master@{#348825}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
These are needed to link in asan/lsan/etc. mode. GYP injects these automatically but GN doesn't have automatic dependency injection.
All tests and components (which cover the vast majority of such targets in actual use) are templates which add these dependencies already, so most of the time it never comes up.
Only a few of these changes are actually necessary to link all tests run on the asan bot (which is also what engineers will typically build, since almost nobody will build all in asan mode). But to reduce confusion from linker errors in asan mode, I annotated all executables and shared libraries I could find with the sanitizer dependency.
I moved a config out of a target in yasm, and sorted a fews deps on other targets.
TBR=xhwang@chromium.org (widevine)
TBR=achuith@chromium.org (tools/perf, tools/telemetry)
Review URL: https://codereview.chromium.org/1318343003
Cr-Commit-Position: refs/heads/master@{#346490}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Changes the version processing templates to not implicitly include the chrome version .rc file. This was something I've been meaning to do for a while which was forced by this patch, because it wants to call version.py with no sources or template file. I also had to remove some checks that either a template file or sources were defined (I wasn't aware this was valid to do when I wrote these checks).
The above-mentioned change necessitated referencing the version template file from a number of existing calls.
The version script now explicitly sets the output file with -o. Previously this was the implicit second argument, but this didn't work if there was no template file. -o is the same and it's better to be more explicit anyway.
Added copy rules for xdg-mime related scripts which are required by the installer.
Added a meta "installer" target which links to the Linux installers. This will also be a good place to hook up the Windows installer when we write it.
BUG=525839
TBR=thestig@chromium.org (linux installer), ddorwin@chromium.org (widevine)
Review URL: https://codereview.chromium.org/1311543003
Cr-Commit-Position: refs/heads/master@{#346279}
|
|
|
|
|
|
|
|
|
|
| |
I've moved on to a different project and am no longer reviewing code.
Thanks for the memories!
Review URL: https://codereview.chromium.org/1215253002
Cr-Commit-Position: refs/heads/master@{#337084}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The C++ define OFFICIAL_BUILD wasn't getting set, and the widevine version .rc files was getting linked twice.
CQ_EXTRA_TRYBOTS=tryserver.chromium.linux:android_chromium_gn_compile_dbg,android_chromium_gn_compile_rel;tryserver.chromium.win:win8_chromium_gn_rel,win8_chromium_gn_dbg;tryserver.chromium.mac:mac_chromium_gn_rel,mac_chromium_gn_dbg
R=dpranke
TBR=ddorwin@chromium.org
Review URL: https://codereview.chromium.org/1134623003
Cr-Commit-Position: refs/heads/master@{#329308}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The first pass of moving the other Chrome-specific build flags out of the root build config file and into the new one. This requires both ffmpeg and Blink changes to finish landing.
Originally landed as https://codereview.chromium.org/1120803003/
TBR=scottmg@chromium.org
TBR=ddorwin@chromium.org (widevine)
CQ_EXTRA_TRYBOTS=tryserver.chromium.linux:android_chromium_gn_compile_dbg,android_chromium_gn_compile_rel;tryserver.chromium.win:win8_chromium_gn_rel,win8_chromium_gn_dbg;tryserver.chromium.mac:mac_chromium_gn_rel,mac_chromium_gn_dbg
Review URL: https://codereview.chromium.org/1123433002
Cr-Commit-Position: refs/heads/master@{#327998}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
id:40001 of https://codereview.chromium.org/1120803003/)
Reason for revert:
May have caused some link errors on windows: https://build.chromium.org/p/chromium.win/builders/Win%20x64%20GN/builds/814
Original issue's description:
> Add a GN flag for the Windows multi-dll build.
>
> The first pass of moving the other Chrome-specific build flags out of the root build config file and into the new one. This requires both ffmpeg and Blink changes to finish landing.
>
> R=scottmg@chromium.org
> TBR=ddorwin@chromium.org (widevine)
>
> Committed: https://crrev.com/edff53a35780ba2f515c7e8d4f7131c91151e1a3
> Cr-Commit-Position: refs/heads/master@{#327854}
TBR=scottmg@chromium.org,brettw@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
Review URL: https://codereview.chromium.org/1117063004
Cr-Commit-Position: refs/heads/master@{#327861}
|
|
|
|
|
|
|
|
|
|
|
| |
The first pass of moving the other Chrome-specific build flags out of the root build config file and into the new one. This requires both ffmpeg and Blink changes to finish landing.
R=scottmg@chromium.org
TBR=ddorwin@chromium.org (widevine)
Review URL: https://codereview.chromium.org/1120803003
Cr-Commit-Position: refs/heads/master@{#327854}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Please take look at
https://codereview.chromium.org/1090213002/patch/40001/50001
for details
BUG=
Review URL: https://codereview.chromium.org/1106503003
Cr-Commit-Position: refs/heads/master@{#326963}
|
|
|
|
|
|
|
|
| |
BUG=468567
Review URL: https://codereview.chromium.org/1070253002
Cr-Commit-Position: refs/heads/master@{#325363}
|
|
|
|
|
|
|
|
|
|
|
| |
Adds a stub CDM that is used to link widevinecdmadapter when enable_widevine=1.
BUG=468567
TEST=Widevine EME browser tests run in Chromium with enable_widevine=1
Review URL: https://codereview.chromium.org/1043673002
Cr-Commit-Position: refs/heads/master@{#323862}
|
|
|
|
|
|
|
|
|
|
|
| |
For this first cl, it's only checking checked in code. I've had to add DEPS files to make it pass, since some of this code legitimately uses base etc...
BUG=469692
R=brettw@chromium.org
Review URL: https://codereview.chromium.org/1045863002
Cr-Commit-Position: refs/heads/master@{#323302}
|
|
|
|
|
|
|
|
| |
BUG=349185
Review URL: https://codereview.chromium.org/1000863003
Cr-Commit-Position: refs/heads/master@{#321407}
|
|
|
|
|
|
|
|
|
|
|
|
| |
Changes process_version template. In GYP this is used in two ways and the GN version only supported one of these. This change also deletes an unnecessary fork of the template file, and fixes the dependencies for the extra files (previously they were just passed with "-f" and not added to the inputs of the target.
Adds a cdm_adapter template which basically matches the GYP version. Uses this for the media clearkey adapter and the widevine one in the official build.
Fixes for the crypto targets when compiling with the official build's checked-in sysroots.
Review URL: https://codereview.chromium.org/949233003
Cr-Commit-Position: refs/heads/master@{#318083}
|
|
|
|
|
|
|
|
|
|
|
|
| |
This fills out a number of GN build TODOs.
Updates some out-of-date documentation I noticed.
TBR=ddorwin@chromium.org for widevine
Review URL: https://codereview.chromium.org/950893005
Cr-Commit-Position: refs/heads/master@{#317894}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The latest GN binaries rename cpu_arch to current_cpu and/or target_cpu
as appropriate, build_cpu_arch to host_cpu, and os to current_os and target_os as appropriate.
R=brettw@chromium.org
TBR=ddorwin@chromium.org
EXTRA_TRYBOTS=tryserver.chromium.linux:android_chromium_gn_compile_dbg,android_chromium_gn_compile_rel;tryserver.chromium.win:win8_chromium_gn_rel,win8_chromium_gn_dbg;tryserver.chromium.mac:mac_chromium_gn_rel,mac_chromium_gn_dbg
BUG=344767
Review URL: https://codereview.chromium.org/913373002
Cr-Commit-Position: refs/heads/master@{#317223}
|
|
|
|
|
|
|
|
|
|
|
|
| |
This puts BUILD files in the corresponding directories without duplicate names.
Adds a template for ppapi examples and writes the build for one example. I only did one at this point to test the infrastructure without making this patch too large.
NOPRESUBMIT=true
Review URL: https://codereview.chromium.org/921953002
Cr-Commit-Position: refs/heads/master@{#316352}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Fixing a compilation problem for debug version.
Because of the latest changes, references which are underspacified cause compilation error. By adding -lrt command makes the compiler to search for rt library(librt.so) and link to the output file.
See discussion here: https://groups.google.com/a/chromium.org/forum/#!topic/chromium-dev/I6eBfNaAV0s
BUG=371125
Review URL: https://codereview.chromium.org/858333005
Cr-Commit-Position: refs/heads/master@{#312681}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Other misc fixes including:
Add "data_deps" to the templates in BUILDCONFIG. This was renamed from "datadeps" a while ago but both are still supported.
extension_management_test_util.cc includes a generated file policy_constants. GYP generates this with an extra "policy" directory but GN doesn't. The way GYP has the dirs set up, though, using only one "policy" works (and this is what other files do) so I just changed the include.
TBR=dpranke
Review URL: https://codereview.chromium.org/824373003
Cr-Commit-Position: refs/heads/master@{#310276}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Excluded mojo because I think that needs to happen on the other side.
At gn --version = 306668 for which roll is in CQ.
R=brettw@chromium.org
TBR=scherkus@chromium.org
BUG=348474
Review URL: https://codereview.chromium.org/774353003
Cr-Commit-Position: refs/heads/master@{#306708}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Uses gn binary at head.
No intended functionality change.
R=brettw@chromium.org
TBR=scherkus@chromium.org
BUG=348474
Review URL: https://codereview.chromium.org/779473002
Cr-Commit-Position: refs/heads/master@{#306567}
|
|
|
|
|
|
|
|
|
|
| |
The CDM adapter stuff is pretty hairy so I didn't get very far down that road.
TBR=scottmg
Review URL: https://codereview.chromium.org/756723002
Cr-Commit-Position: refs/heads/master@{#305842}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is needed to support Encrypted Media Extensions (EME) without relying
content.
A MediaClient interface is added so that embedders (e.g. content) can inject
embedder specific data.
BUG=422730
Review URL: https://codereview.chromium.org/680383002
Cr-Commit-Position: refs/heads/master@{#303907}
|
|
|
|
|
|
|
|
| |
BUG=426252
Review URL: https://codereview.chromium.org/701223002
Cr-Commit-Position: refs/heads/master@{#303279}
|
|
|
|
|
|
|
|
|
| |
BUG=426252
TEST=compiles, verified version number shows up on Windows
Review URL: https://codereview.chromium.org/681983004
Cr-Commit-Position: refs/heads/master@{#302485}
|
|
|
|
|
|
|
|
|
| |
BUG=
R=viettrungluu@chromium.org
Review URL: https://codereview.chromium.org/371363006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@281832 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
Fix up some references in content and webkit for targets that now exist.
TBR=hclam@chromium.org
Review URL: https://codereview.chromium.org/344873006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278616 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=361318
TEST=Add VP9 to unittests.
Review URL: https://codereview.chromium.org/252513005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@267047 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The DEPS roll for test license server has been fixed.
This CL updates WV encrypted media browser tests to:
- use test license server in third_party/widevine/test.
- dynamically select an open port to start the test server on.
It also adds a new gyp target for test_license_server.
BUG=339983
Review URL: https://codereview.chromium.org/186243003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@255004 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
(https://codereview.chromium.org/143003026/)
Reason for revert:
Broke gclient runhooks on Google Chrome ChromeOS:
http://build.chromium.org/p/chromium.chrome/builders/Google%20Chrome%20ChromeOS/builds/65059/steps/gclient%20runhooks/logs/stdio
gyp: /b/build/slave/google-chrome-rel-chromeos/build/src/third_party/widevine/test/license_server/license_server.gyp not found
Original issue's description:
> Update WV test license server config to use portable sdk server.
>
> This also makes the test server dynamically select an open port to use.
>
> BUG=339983
>
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=254258
TBR=ddorwin@chromium.org,xhwang@chromium.org,shadi@google.com,shadi@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=339983
Review URL: https://codereview.chromium.org/183973009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@254272 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
This also makes the test server dynamically select an open port to use.
BUG=339983
Review URL: https://codereview.chromium.org/143003026
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@254258 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=334126
Review URL: https://codereview.chromium.org/136343002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245449 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=328642
Review URL: https://codereview.chromium.org/99533005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@242247 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This removes duplicate code in CDM adapter gyp targets.
Also move more code under the condition enable_pepper_cdms==1 so that when enable_pepper_cdms==0, clearkeycdm, clearkeycdmadapter and widevinecdmadapter targets won't exist.
BUG=329097
TEST=Widevine and ClearKey CDM adapters still work.
Review URL: https://codereview.chromium.org/108703004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@241736 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
R=ddorwin@chromium.org
TBR=ddorwin@chromium.org
BUG=324134
TEST=Tests will follow in another CL. All tests cases pass with that test.
Review URL: https://codereview.chromium.org/103853005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@241176 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Fixed widevine_cdm.gyp.
Original CL description:
The CDM adapter is a pepper plugin and it should not depend on base/. Therefore
we cannot use base/logging.h directly. Instead, add CDM_DLOG, which is a
simplified version of it.
Also add CdmAdapter::ConsoleLog() for JS console logs.
BUG=320065,328486
TEST=Log works on Linux.
R=dalecurtis@chromium.org, ddorwin@chromium.org
TBR=dalecurtis@chromium.org, ddorwin@chromium.org
Review URL: https://codereview.chromium.org/100413006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@240944 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Compatibility is not checked before installing a component. This should be prevented on the server side.
BUG=311374
TEST=video.canPlayType('video/webm; codecs="vp8"', 'com.widevine.alpha'); is affirmative when manifest contains supported "x-cdm-*-versions" values and "" when any value is unsupported.
Review URL: https://codereview.chromium.org/47513020
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@232424 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Look for "x-cdm-codecs" in the manifest and set its value in
WebPluginInfo::WebPluginMimeType::additional_param_* where it can be used to
determine the types supported.
The manifest is read when registering existing components in addition to new
installs so that the manifest information is always available.
Also updated the non-component path so the behavior is consistent for the
consumer, which will be updated in a separate CL.
BUG=311370
TEST=canPlayType() with various manifest configurations
Review URL: https://codereview.chromium.org/42003002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@231197 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
CdmWrapper wraps different versions of ContentDecryptionModule interfaces and exposes a common interface to the caller.
BUG=306647
TEST=none
Review URL: https://codereview.chromium.org/26155003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@229571 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- Add cdm_helper.h to hold types used by CdmWrapper.
- Add cdm_wrapper.h to separate declaration and definition.
- Rename CdmWrapper to CdmAdater to better match the binary name, which is
"*cdmadapter.*".
BUG=306647
TEST=none
R=dalecurtis@chromium.org, ddorwin@chromium.org
Review URL: https://codereview.chromium.org/27774002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@229494 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
Passed the canplaytype check when tested with http://dash-mse-test.appspot.com/append-all.html?sd=1&keysystem=widevine
BUG=224793
Review URL: https://chromiumcodereview.appspot.com/24072009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@224312 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Previously it was only enabled on Chrome for Android builds.
BUG=none
TEST=KeySystem tests still pass on Chrome and Chromium builds.
Review URL: https://chromiumcodereview.appspot.com/23707048
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@223966 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: https://chromiumcodereview.appspot.com/23558014
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@222745 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- EME API works on Chrome build.
- CanPlayType() is suppressed (returns "") by default.
- User can choose to override the CanPlayType() suppression with a flag
"--override-encrypted-media-canplaytype".
- For Android "video/mp4" is supported, while "video/webm" isn't.
BUG=163552
TEST=Tested all scenarios listed above.
R=ddorwin@chromium.org
Review URL: https://codereview.chromium.org/23975003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@221548 0039d316-1c4b-4281-b951-d872f2087c98
|