convert most webrtc builders to remote_run

masters/master.client.webrtc/master_mac_cfg.py

Index: masters/master.client.webrtc/master_mac_cfg.py
diff --git a/masters/master.client.webrtc/master_mac_cfg.py b/masters/master.client.webrtc/master_mac_cfg.py
index 6fc0d0d383f48e1622494f09ff332be580629ed2..3f589abc289064383f00cf11925e78de42eb029f 100644
--- a/masters/master.client.webrtc/master_mac_cfg.py
+++ b/masters/master.client.webrtc/master_mac_cfg.py
@@ -4,9 +4,20 @@
 
 from buildbot.schedulers.basic import SingleBranchScheduler
 
-from master.factory import annotator_factory
+from master.factory import remote_run_factory
+
+import master_site_config
+ActiveMaster = master_site_config.WebRTC
+
+
+def m_remote_run(recipe, **kwargs):
+  return remote_run_factory.RemoteRunFactory(
+      active_master=ActiveMaster,
+      repository='https://chromium.googlesource.com/chromium/tools/build.git',

[kjellander_chromium, 2016/08/09 14:16:21]

This URL is duplicated a lot. Is that a good long-term solution for all masters?

[Paweł Hajdan Jr., 2016/08/09 15:04:52]

The long term solution is to get rid of buildbot.

Current way master configs are done leads to a lot of duplication like this.

+      recipe=recipe,
+      factory_properties={'path_config': 'kitchen'},
+      **kwargs)
 
-m_annotator = annotator_factory.AnnotatorFactory()
 
 def Update(c):
   c['schedulers'].extend([
@@ -95,8 +106,7 @@ def Update(c):
   c['builders'].extend([
       {
         'name': spec['name'],
-        'factory': m_annotator.BaseFactory(spec.get('recipe',
-                                                    'webrtc/standalone')),
+        'factory': m_remote_run(spec.get('recipe', 'webrtc/standalone')),
         'notify_on_missing': True,
         'category': spec.get('category', 'compile|testers'),
         'slavebuilddir': spec['slavebuilddir'],