From 5c40fdbb57a208bc832cd9109a0e0d305b668113 Mon Sep 17 00:00:00 2001
From: Brian Vaughn <bvaughn@fb.com>
Date: Thu, 20 Aug 2020 14:43:40 -0400
Subject: [PATCH] Visually distinguish user timing marks from React events
 (#19663)

---
 .../src/content-views/UserTimingMarksView.js  | 22 ++++++++++++-------
 .../src/content-views/constants.js            |  1 +
 2 files changed, 15 insertions(+), 8 deletions(-)

diff --git a/packages/react-devtools-scheduling-profiler/src/content-views/UserTimingMarksView.js b/packages/react-devtools-scheduling-profiler/src/content-views/UserTimingMarksView.js
index 1078ecc26b384..4a73c48aae076 100644
--- a/packages/react-devtools-scheduling-profiler/src/content-views/UserTimingMarksView.js
+++ b/packages/react-devtools-scheduling-profiler/src/content-views/UserTimingMarksView.js
@@ -26,11 +26,12 @@ import {
 import {
   COLORS,
   EVENT_ROW_PADDING,
-  EVENT_DIAMETER,
+  USER_TIMING_MARK_SIZE,
   BORDER_SIZE,
 } from './constants';
 
-const ROW_HEIGHT_FIXED = EVENT_ROW_PADDING + EVENT_DIAMETER + EVENT_ROW_PADDING;
+const ROW_HEIGHT_FIXED =
+  EVENT_ROW_PADDING + USER_TIMING_MARK_SIZE + EVENT_ROW_PADDING;
 
 export class UserTimingMarksView extends View {
   _marks: UserTimingMark[];
@@ -81,13 +82,15 @@ export class UserTimingMarksView extends View {
     const {timestamp} = mark;
 
     const x = timestampToPosition(timestamp, scaleFactor, frame);
-    const radius = EVENT_DIAMETER / 2;
+    const size = USER_TIMING_MARK_SIZE;
+    const halfSize = size / 2;
+
     const markRect: Rect = {
       origin: {
-        x: x - radius,
+        x: x - halfSize,
         y: baseY,
       },
-      size: {width: EVENT_DIAMETER, height: EVENT_DIAMETER},
+      size: {width: size, height: size},
     };
     if (!rectIntersectsRect(markRect, rect)) {
       return; // Not in view
@@ -98,11 +101,14 @@ export class UserTimingMarksView extends View {
       : COLORS.USER_TIMING;
 
     if (fillStyle !== null) {
-      const y = markRect.origin.y + radius;
+      const y = baseY + halfSize;
 
       context.beginPath();
       context.fillStyle = fillStyle;
-      context.arc(x, y, radius, 0, 2 * Math.PI);
+      context.moveTo(x, y - halfSize);
+      context.lineTo(x + halfSize, y);
+      context.lineTo(x, y + halfSize);
+      context.lineTo(x - halfSize, y);
       context.fill();
     }
   }
@@ -198,7 +204,7 @@ export class UserTimingMarksView extends View {
     );
     const hoverTimestamp = positionToTimestamp(location.x, scaleFactor, frame);
     const markTimestampAllowance = widthToDuration(
-      EVENT_DIAMETER / 2,
+      USER_TIMING_MARK_SIZE / 2,
       scaleFactor,
     );
 
diff --git a/packages/react-devtools-scheduling-profiler/src/content-views/constants.js b/packages/react-devtools-scheduling-profiler/src/content-views/constants.js
index 468f223be295f..7b0fb87260769 100644
--- a/packages/react-devtools-scheduling-profiler/src/content-views/constants.js
+++ b/packages/react-devtools-scheduling-profiler/src/content-views/constants.js
@@ -33,6 +33,7 @@ export const MIN_INTERVAL_SIZE_PX = 70;
 
 export const EVENT_ROW_PADDING = 4;
 export const EVENT_DIAMETER = 6;
+export const USER_TIMING_MARK_SIZE = 8;
 export const REACT_MEASURE_HEIGHT = 9;
 export const BORDER_SIZE = 1;