diff options
author | dalecurtis@google.com <dalecurtis@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-10-21 22:20:04 +0000 |
---|---|---|
committer | dalecurtis@google.com <dalecurtis@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-10-21 22:20:04 +0000 |
commit | 243d52e3d20fbd72d5fdb9cb0a57dccf10600742 (patch) | |
tree | 7f545abe0006fb7ecfa9e50b4ce9ff7737950998 /chrome/unit_tests.isolate | |
parent | c9def35e9f827c073ccd217bab9bc2398b8a0c38 (diff) | |
download | chromium_src-243d52e3d20fbd72d5fdb9cb0a57dccf10600742.zip chromium_src-243d52e3d20fbd72d5fdb9cb0a57dccf10600742.tar.gz chromium_src-243d52e3d20fbd72d5fdb9cb0a57dccf10600742.tar.bz2 |
Using Ronald's libclang based converter, we're able to translate
ffmpeg from C99 to C89 syntax and compile through MSVC++! This is
awesome for many reasons:
- Better performance! MSVC compiled code is as fast as some of the
hand rolled assembly in certain cases (per rbultje).
- Debugging symbols! (!!!) Crash dumps will now be symbolized with
all symbols instead of the missing symbols traces we get in crash
reports today.
- No more hand rolling binaries every time ffmpeg needs a security
fix or someone wants to make a change!
- Greatly reduced binary size, the Chrome DLLs are ~3.2mb smaller
and the Chromium DLLs ~2.3mb smaller: Chrome: 5205049 -> 1817600,
Chromium: 3587185 -> 1147904.
This change also cleans up the media library initialization code
and gets rid of the old dll names in various places. There is
still one other locations which need to be updated separately:
http://src.chromium.org/viewvc/chrome/trunk/tools/build/site_config/config.py
The downside is this will add ~3minutes of compile time (on T3500)
for non-GOMA users and ~30 seconds for GOMA users (-j 256).
More information available here:
https://gerrit.chromium.org/gerrit/#/c/34728/
BUG=39887
TEST=Manual testing. Unit tests. It actually compiles!
Review URL: https://codereview.chromium.org/11036039
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@163213 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/unit_tests.isolate')
-rw-r--r-- | chrome/unit_tests.isolate | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/chrome/unit_tests.isolate b/chrome/unit_tests.isolate index 4f76b82..d36f7c9 100644 --- a/chrome/unit_tests.isolate +++ b/chrome/unit_tests.isolate @@ -870,9 +870,7 @@ '../third_party/python_26/Lib/xmlrpclib.py', '../third_party/python_26/python.exe', '../third_party/python_26/python26.dll', - '<(PRODUCT_DIR)/avcodec-54.dll', - '<(PRODUCT_DIR)/avformat-54.dll', - '<(PRODUCT_DIR)/avutil-51.dll', + '<(PRODUCT_DIR)/ffmpegsumo.dll', '<(PRODUCT_DIR)/icudt.dll', 'test/data/extensions/permissions/platform_app_all_urls.json', 'test/data/extensions/permissions/platform_app_hosts.json', |