Remove "master." when getting master name in rietveld.py.

third_party/WebKit/Tools/Scripts/webkitpy/common/net/rietveld.py

Index: third_party/WebKit/Tools/Scripts/webkitpy/common/net/rietveld.py
diff --git a/third_party/WebKit/Tools/Scripts/webkitpy/common/net/rietveld.py b/third_party/WebKit/Tools/Scripts/webkitpy/common/net/rietveld.py
index 9411f7c1feb82c1decd385ca5df2bb8e8ebdb147..26c62ce4da0dfa362cd0b88809bc4cf9a4de50bf 100644
--- a/third_party/WebKit/Tools/Scripts/webkitpy/common/net/rietveld.py
+++ b/third_party/WebKit/Tools/Scripts/webkitpy/common/net/rietveld.py
@@ -42,9 +42,14 @@ def latest_try_jobs(issue_number, builder_names, web, patchset_number=None):
     for job in patchset_data['try_job_results']:
         if job['builder'] not in builder_names:
             continue
+        # The master name may be prefixed with "master.", or possibly not;
+        # We want to normalize master name by stripping this prefix.

[wkorman, 2016/06/29 18:31:52]

It seems wacky that it's configured this way. Perhaps worth filing a bug for
chrome-infra to look into to standardize, with whatever examples you've seen and
a link to this change so they can see the hack workaround we've had to do.

[qyearsley, 2016/06/29 20:51:07]

Great idea. I hypothesize that when jobs are started via `git cl try`, the
master name is one way, and if started via Rietveld directly it may be another
way. If I can reproduce this then I'll file a bug for chrome-infra.

+        master_name = job['master']
+        if master_name.startswith('master.'):
+            master_name = master_name[len('master.'):]
         jobs.append(TryJob(
             builder_name=job['builder'],
-            master_name=job['master'],
+            master_name=master_name,
             build_number=job['buildnumber']))
     return jobs