Commit 362af2a5 authored by Brett Wilson's avatar Brett Wilson

Link VPX into GN build more.

These cases were commented out (VPX used to not work). They should now match the GYP build. I think these deps aren't strictly necessary because dependent targets link vpx in, but it's a good idea to match GYP, and I think this will make the header checker happier.

R=jamesr@chromium.org

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

Cr-Commit-Position: refs/heads/master@{#297039}
parent 88a08a6c
...@@ -117,7 +117,7 @@ static_library("test_support") { ...@@ -117,7 +117,7 @@ static_library("test_support") {
} }
if (!is_android && !is_ios) { if (!is_android && !is_ios) {
#deps += [ "//third_party/libvpx" ] TODO(GYP) deps += [ "//third_party/libvpx" ]
} }
if (is_android) { if (is_android) {
......
...@@ -78,7 +78,7 @@ static_library("base") { ...@@ -78,7 +78,7 @@ static_library("base") {
"//media", "//media",
"//media:shared_memory_support", "//media:shared_memory_support",
"//remoting/resources", "//remoting/resources",
#"//third_party/libvpx", TODO(GYP) "//third_party/libvpx",
"//third_party/libyuv", "//third_party/libyuv",
"//third_party/opus", "//third_party/opus",
"//third_party/webrtc/modules/desktop_capture", "//third_party/webrtc/modules/desktop_capture",
...@@ -86,6 +86,5 @@ static_library("base") { ...@@ -86,6 +86,5 @@ static_library("base") {
"//ui/gfx", "//ui/gfx",
"//ui/gfx/geometry", "//ui/gfx/geometry",
] ]
} }
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment