Enable all Android builders to use 'chrome_with_codecs'.

tools/mb/mb.py

 #!/usr/bin/env python
 # Copyright 2015 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 """MB - the Meta-Build wrapper around GYP and GN
 
 MB is a wrapper script for GYP and GN that can be used to generate build files
 for sets of canned configurations and analyze them.
 """
@@ skipping 287 matching lines @@
         if not sub_mixin in self.mixins:
           errs.append('Unknown mixin "%s" referenced by mixin "%s".' %
                       (sub_mixin, mixin))
         referenced_mixins.add(sub_mixin)
 
     # Check that every mixin defined is actually referenced somewhere.
     for mixin in self.mixins:
       if not mixin in referenced_mixins:
         errs.append('Unreferenced mixin "%s".' % mixin)
 
+    # Check that 'chromium' bots which build public artifacts do not include
+    # the chrome_with_codecs 'mixin'.
+    if not 'chromium' in self.masters:
+      errs.append('Missing "chromium" master. Please update this proprietary '

[kjellander_chromium, 2016/03/15 03:54:59]

In WebRTC, we have own mb_config.pyl for the standalone build:
https://chromium.googlesource.com/external/webrtc/+/master/webrtc/build/mb_co...

This check causes an error when we run the mb.py validation on our config. Would
it be possible to just remove the error here?
It feels wrong that we would just add a fake 'chromium' master to our config.

+                  'codecs check with the name of the master responsible for '
+                  'public build artifacts.')
+    else:
+      for builder in self.masters['chromium']:
+        config = self.masters['chromium'][builder]
+        def RecurseMixins(current_mixin):
+          if current_mixin == 'chrome_with_codecs':
+            errs.append('Public artifact builder "%s" can not contain the '
+                        '"chrome_with_codecs" mixin.' % builder)
+            return
+          if not 'mixins' in self.mixins[current_mixin]:
+            return
+          for mixin in self.mixins[current_mixin]['mixins']:
+            RecurseMixins(mixin)
+
+        for mixin in self.configs[config]:
+          RecurseMixins(mixin)
+
     if errs:
       raise MBErr(('mb config file %s has problems:' % self.args.config_file) +
                     '\n  ' + '\n  '.join(errs))
 
     self.Print('mb config file %s looks ok.' % self.args.config_file)
     return 0
 
   def GetConfig(self):
     build_dir = self.args.path[0]
 
@@ skipping 837 matching lines @@
 
 if __name__ == '__main__':
   try:
     sys.exit(main(sys.argv[1:]))
   except MBErr as e:
     print(e)
     sys.exit(1)
   except KeyboardInterrupt:
     print("interrupted, exiting", stream=sys.stderr)
     sys.exit(130)