From 50c6ea3aaf580c1a414a187f4782855bb862c604 Mon Sep 17 00:00:00 2001
From: Adam Gastineau <adam@agg.im>
Date: Thu, 19 Dec 2024 12:39:06 -0800
Subject: [PATCH] Remove error-prone wall time

---
 packages/trace-viewer/src/ui/callTab.tsx              | 11 +++--------
 .../trace-viewer/src/ui/networkResourceDetails.tsx    |  3 ---
 packages/trace-viewer/src/ui/workbench.tsx            |  2 +-
 3 files changed, 4 insertions(+), 12 deletions(-)

diff --git a/packages/trace-viewer/src/ui/callTab.tsx b/packages/trace-viewer/src/ui/callTab.tsx
index 6c1547a9a3..1c78920f86 100644
--- a/packages/trace-viewer/src/ui/callTab.tsx
+++ b/packages/trace-viewer/src/ui/callTab.tsx
@@ -27,10 +27,9 @@ import type { ActionTraceEventInContext } from './modelUtil';
 
 export const CallTab: React.FunctionComponent<{
   action: ActionTraceEventInContext | undefined,
-  executionStartTime: number,
-  executionStartWallTime: number,
+  startTimeOffset: number,
   sdkLanguage: Language | undefined,
-}> = ({ action, executionStartTime, executionStartWallTime, sdkLanguage }) => {
+}> = ({ action, startTimeOffset, sdkLanguage }) => {
   // We never need the waitForEventInfo (`info`).
   const paramKeys = React.useMemo(() => Object.keys(action?.params ?? {}).filter(name => name !== 'info'), [action]);
 
@@ -38,12 +37,9 @@ export const CallTab: React.FunctionComponent<{
     return <PlaceholderPanel text='No action selected' />;
 
   // Calculate execution time relative to the test runner's start time
-  const startTimeMillis = action.startTime - executionStartTime;
+  const startTimeMillis = action.startTime - startTimeOffset;
   const startTime = msToString(startTimeMillis);
 
-  const wallTimeMillis = startTimeMillis + executionStartWallTime;
-  const wallTime = new Date(wallTimeMillis).toLocaleString(undefined, { timeZoneName: 'short' });
-
   const duration = action.endTime ? msToString(action.endTime - action.startTime) : 'Timed Out';
 
   return (
@@ -52,7 +48,6 @@ export const CallTab: React.FunctionComponent<{
       {
         <>
           <div className='call-section'>Time</div>
-          <DateTimeCallLine name='wall time:' value={wallTime} />
           <DateTimeCallLine name='start:' value={startTime} />
           <DateTimeCallLine name='duration:' value={duration} />
         </>
diff --git a/packages/trace-viewer/src/ui/networkResourceDetails.tsx b/packages/trace-viewer/src/ui/networkResourceDetails.tsx
index f39fe9127f..aaa78d1786 100644
--- a/packages/trace-viewer/src/ui/networkResourceDetails.tsx
+++ b/packages/trace-viewer/src/ui/networkResourceDetails.tsx
@@ -65,8 +65,6 @@ const RequestTab: React.FunctionComponent<{
 }> = ({ resource, sdkLanguage, startTimeOffset }) => {
   const [requestBody, setRequestBody] = React.useState<{ text: string, mimeType?: string } | null>(null);
 
-  const wallTimeString = React.useMemo(() => resource.startedDateTime.length > 0 ? new Date(resource.startedDateTime).toLocaleString(undefined, { timeZoneName: 'short' }) : '-', [resource.startedDateTime]);
-
   React.useEffect(() => {
     const readResources = async  () => {
       if (resource.request.postData) {
@@ -102,7 +100,6 @@ const RequestTab: React.FunctionComponent<{
     <div className='network-request-details-header'>Request Headers</div>
     <div className='network-request-details-headers'>{resource.request.headers.map(pair => `${pair.name}: ${pair.value}`).join('\n')}</div>
     <div className='network-request-details-header'>Time</div>
-    <div className='network-request-details-general'>{`Wall Time: ${wallTimeString}`}</div>
     <div className='network-request-details-general'>{`Start: ${msToString(startTimeOffset)}`}</div>
     <div className='network-request-details-general'>{`Duration: ${msToString(resource.time)}`}</div>
 
diff --git a/packages/trace-viewer/src/ui/workbench.tsx b/packages/trace-viewer/src/ui/workbench.tsx
index deb325d271..7a14d495c6 100644
--- a/packages/trace-viewer/src/ui/workbench.tsx
+++ b/packages/trace-viewer/src/ui/workbench.tsx
@@ -176,7 +176,7 @@ export const Workbench: React.FunctionComponent<{
   const callTab: TabbedPaneTabModel = {
     id: 'call',
     title: 'Call',
-    render: () => <CallTab action={activeAction} executionStartTime={model?.startTime ?? 0} executionStartWallTime={model?.wallTime ?? 0} sdkLanguage={sdkLanguage} />
+    render: () => <CallTab action={activeAction} startTimeOffset={model?.startTime ?? 0} sdkLanguage={sdkLanguage} />
   };
   const logTab: TabbedPaneTabModel = {
     id: 'log',