Make clock sync latency metric go between each pair of clock domains.

tracing/tracing/metrics/system_health/clock_sync_latency_metric_test.html

Index: tracing/tracing/metrics/system_health/clock_sync_latency_metric_test.html
diff --git a/tracing/tracing/metrics/system_health/clock_sync_latency_metric_test.html b/tracing/tracing/metrics/system_health/clock_sync_latency_metric_test.html
index 3a5090123e6ae52479c2c4a2c64558910bb30977..a93bc849dbfda5ddad942aaaf76ad24051f54ba3 100644
--- a/tracing/tracing/metrics/system_health/clock_sync_latency_metric_test.html
+++ b/tracing/tracing/metrics/system_health/clock_sync_latency_metric_test.html
@@ -25,42 +25,40 @@ tr.b.unittest.testSuite(function() {
     model.clockSyncManager.addClockSyncMarker(
         tr.model.ClockDomainId.WIN_QPC, 'ID02', 5.0);
 
-    var battorName = 'clock_sync_latency_' +
+    var battorToWinQpcName = 'clock_sync_latency_' +
+        tr.model.ClockDomainId.WIN_QPC.toLowerCase() + '_to_' +
+        tr.model.ClockDomainId.BATTOR.toLowerCase();
+    var winQpcToTelemetryName = 'clock_sync_latency_' +
+        tr.model.ClockDomainId.WIN_QPC.toLowerCase() + '_to_' +
+        tr.model.ClockDomainId.TELEMETRY.toLowerCase();
+    var battorToTelemetryName = 'clock_sync_latency_' +
+        tr.model.ClockDomainId.TELEMETRY.toLowerCase() + '_to_' +
         tr.model.ClockDomainId.BATTOR.toLowerCase();
-    var winQpcName = 'clock_sync_latency_' +
-        tr.model.ClockDomainId.WIN_QPC.toLowerCase();
 
     var valueSet = new tr.v.ValueSet();
     tr.metrics.sh.clockSyncLatencyMetric(valueSet, model);
 
-    var battorValue = undefined;
-    var winQpcValue = undefined;
+    var battorToWinQpcValue = undefined;
+    var winQpcToTelemetryValue = undefined;
+    var battorToTelemetryValue = undefined;
     for (var value of valueSet) {
-      if (value.name === battorName)
-        battorValue = value;
-      else if (value.name === winQpcName)
-        winQpcValue = value;
+      if (value.name === battorToWinQpcName)
+        battorToWinQpcValue = value;
+      else if (value.name === winQpcToTelemetryName)
+        winQpcToTelemetryValue = value;
+      else if (value.name === battorToTelemetryName)
+        battorToTelemetryValue = value;
     }
 
-    assert.isDefined(battorValue);
-    assert.isDefined(winQpcValue);
-    assert.closeTo(battorValue.numeric.average, 3.0, 1e-5);
-    assert.closeTo(winQpcValue.numeric.average, 6.0, 1e-5);
-  });
-
-  test('clockSyncLatencyMetric_noTelemetry', function() {
-    var model = new tr.Model();
-    model.clockSyncManager.addClockSyncMarker(
-        tr.model.ClockDomainId.TELEMETRY, 'ID01', 1.0, 4.0);
-    model.clockSyncManager.addClockSyncMarker(
-        tr.model.ClockDomainId.BATTOR, 'ID02', 2.5);
-    model.clockSyncManager.addClockSyncMarker(
-        tr.model.ClockDomainId.WIN_QPC, 'ID02', 5.0);
-
-    var valueSet = new tr.v.ValueSet();
-    tr.metrics.sh.clockSyncLatencyMetric(valueSet, model);
+    // Clock sync graph is:
+    // [WIN_QPC] --6ms-> [TELEMETRY] --3ms-> [BATTOR]
 
-    assert.lengthOf(valueSet.valueDicts, 0);
+    assert.isDefined(battorToWinQpcValue);
+    assert.isDefined(winQpcToTelemetryValue);
+    assert.isDefined(battorToTelemetryValue);
+    assert.closeTo(battorToWinQpcValue.numeric.average, 9.0, 1e-5);
+    assert.closeTo(winQpcToTelemetryValue.numeric.average, 6.0, 1e-5);
+    assert.closeTo(battorToTelemetryValue.numeric.average, 3.0, 1e-5);
   });
 });
 </script>