[Pinboard] Add patch to aggregated results

BUG=v8:9684

Change-Id: I91f0abfec542b81daaf113e8e7982d661ef7fdd0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2629514
Commit-Queue: Ross McIlroy <rmcilroy@chromium.org>
Commit-Queue: Sami Kyöstilä <skyostil@chromium.org>
Auto-Submit: Ross McIlroy <rmcilroy@chromium.org>
Reviewed-by: Sami Kyöstilä <skyostil@chromium.org>
Cr-Commit-Position: refs/heads/master@{#843663}

tools/perf/cli_tools/pinboard/pinboard.py

@@ -129,10 +129,12 @@ def StartPinpointJobs(state, date):
     logging.info(output)
     assert 'https://pinpoint' in output
     bot = config['configuration']
+    patch = config['patch']
     item['jobs'].append({
         'id': output.split('/')[-1],
         'status': 'queued',
-        'bot': bot
+        'bot': bot,
+        'patch': patch,
     })
   state.append(item)
   state.sort(key=lambda p: p['timestamp'])  # Keep items sorted by date.
@@ -324,13 +326,16 @@ def GetRevisionResults(item):
       df['timestamp'] - pd.DateOffset(years=1))
 
   df['bot'] = 'unknown'
+  df['patch'] = 'unknown'
   for j in item['jobs']:
     bot = j.get('bot', 'unknown')
+    patch = j.get('patch', 'unknown')
     df.loc[df['job_id'].str.contains(str(j['id'])), 'bot'] = bot
+    df.loc[df['job_id'].str.contains(str(j['id'])), 'patch'] = patch
 
   return df[[
-      'revision', 'timestamp', 'bot', 'label', 'benchmark', 'name', 'mean',
-      'count'
+      'revision', 'timestamp', 'bot', 'patch', 'label', 'benchmark', 'name',
+      'mean', 'count'
   ]]
 
 

tools/perf/cli_tools/pinboard/pinboard_unittest.py

@@ -54,10 +54,12 @@ class PinboardToolTests(unittest.TestCase):
   def testStartPinpointJobs(self, load_configs, get_last_commit):
     load_configs.return_value = [{
         'name': 'config1',
-        'configuration': 'AndroidGo'
+        'configuration': 'AndroidGo',
+        'patch': '1234'
     }, {
         'name': 'config2',
-        'configuration': 'Pixel2'
+        'configuration': 'Pixel2',
+        'patch': '5678'
     }]
     get_last_commit.return_value = ('2a66bac4', '2019-03-17T23:50:16-07:00')
     self.subprocess.check_output.side_effect = [
@@ -75,11 +77,13 @@ class PinboardToolTests(unittest.TestCase):
         'jobs': [{
             'id': '14b4c451f40000',
             'status': 'queued',
-            'bot': 'AndroidGo'
+            'bot': 'AndroidGo',
+            'patch': '1234'
         }, {
             'id': '11fae481f40000',
             'status': 'queued',
-            'bot': 'Pixel2'
+            'bot': 'Pixel2',
+            'patch': '5678'
         }]
     }])