Disable libyuv jpeg support on Android

webrtc/supplement.gypi

Index: webrtc/supplement.gypi
diff --git a/webrtc/supplement.gypi b/webrtc/supplement.gypi
index 7b2e075d17537c01152813abb3f9e3eea4d31a89..f1974d73d83f1f92def606a467061ef0a4a9f93d 100644
--- a/webrtc/supplement.gypi
+++ b/webrtc/supplement.gypi
@@ -24,6 +24,11 @@
           }],
         ],
       }],
+      ['OS=="android"', {
+        # MJPEG capture is not used on Android. Disable to reduce
+        # libjingle_peerconnection_so file size.
+        'libyuv_disable_jpeg%': 1,

[kjellander_webrtc, 2016/05/16 14:23:22]

You only need to put a variable in supplement.gypi if you're going to override a
default value in Chromium's build/common.gypi.

For this case, store it in webrtc/build/common.gypi instead, since it's only
read in https://chromium.googlesource.com/libyuv/libyuv/+/master/libyuv.gyp#21

That said, it looks like we need to introduce the variable in GN as well, but
it's not important to do now since the builds we're targeting with this
optimization are based on GYP.

[magjed_webrtc, 2016/05/17 08:45:56]

I tried moving it to webrtc/build/common.gypi, but then libjpeg_turbo got
included in libjingle_peerconnection_so again, so it doesn't work. I don't know
how the include order works for these files, but I guess libyuv.gyp is included
before webrtc/build/common.gypi.

[kjellander_webrtc, 2016/05/17 14:07:49]

Oh dear, GYP trickery. We chatted offline and came to the conclusion that this
must be because libyuv.gyp doesn't include our webrtc/build/common.gypi, so it
probably gets its own environment of GYP variables. If that's how it works, we
need to set this in the global GYP scope, which means this actually is the right
place!

+      }],
     ],
   },
   'target_defaults': {