From 4f47f73e5eb79f844cab1d1b7eb8bf656427874e Mon Sep 17 00:00:00 2001
From: Benjamin Elo <beneloca@fb.com>
Date: Mon, 12 Aug 2019 05:49:44 -0700
Subject: [PATCH] Migrated ShareSheetExportFile from js to ts

Summary: Migrated ShareSheetExportFile.js to ShareSheetExportFile.tsx

Reviewed By: danielbuechele

Differential Revision: D16734995

fbshipit-source-id: 4528b27be0bfbefe9ddbd371c0bfc8950be6fe1f
---
 src/App.js                                    |  2 +-
 ...ExportFile.js => ShareSheetExportFile.tsx} | 46 +++++++++----------
 2 files changed, 22 insertions(+), 26 deletions(-)
 rename src/chrome/{ShareSheetExportFile.js => ShareSheetExportFile.tsx} (86%)

diff --git a/src/App.js b/src/App.js
index e2fd634ed..8b53004fd 100644
--- a/src/App.js
+++ b/src/App.js
@@ -16,7 +16,7 @@ import ErrorBar from './chrome/ErrorBar.tsx';
 import ShareSheet from './chrome/ShareSheet.tsx';
 import SignInSheet from './chrome/SignInSheet.js';
 import ExportDataPluginSheet from './chrome/ExportDataPluginSheet.tsx';
-import ShareSheetExportFile from './chrome/ShareSheetExportFile.js';
+import ShareSheetExportFile from './chrome/ShareSheetExportFile.tsx';
 import PluginContainer from './PluginContainer.js';
 import Sheet from './chrome/Sheet.js';
 import {ipcRenderer, remote} from 'electron';
diff --git a/src/chrome/ShareSheetExportFile.js b/src/chrome/ShareSheetExportFile.tsx
similarity index 86%
rename from src/chrome/ShareSheetExportFile.js
rename to src/chrome/ShareSheetExportFile.tsx
index 2ebca60ab..6f22d35e8 100644
--- a/src/chrome/ShareSheetExportFile.js
+++ b/src/chrome/ShareSheetExportFile.tsx
@@ -11,26 +11,22 @@ import {
   colors,
   Text,
   LoadingIndicator,
-  Component,
   FlexRow,
   Spacer,
 } from 'flipper';
-import {
-  setExportStatusComponent,
-  unsetShare,
-} from '../reducers/application.tsx';
+import React, {Component} from 'react';
+import {setExportStatusComponent, unsetShare} from '../reducers/application';
 import {reportPlatformFailures} from '../utils/metrics';
-import CancellableExportStatus from './CancellableExportStatus.tsx';
-// $FlowFixMe: Missing type defs for node built-in.
+import CancellableExportStatus from './CancellableExportStatus';
 import {performance} from 'perf_hooks';
-import type {Logger} from '../fb-interfaces/Logger.js';
+import {Logger} from '../fb-interfaces/Logger.js';
 import {Idler} from '../utils/Idler';
 import {
   exportStoreToFile,
   EXPORT_FLIPPER_TRACE_EVENT,
-} from '../utils/exportData.tsx';
+} from '../utils/exportData';
 import PropTypes from 'prop-types';
-import ShareSheetErrorList from './ShareSheetErrorList.tsx';
+import ShareSheetErrorList from './ShareSheetErrorList';
 
 const Container = styled(FlexColumn)({
   padding: 20,
@@ -65,19 +61,22 @@ const InfoText = styled(Text)({
 });
 
 type Props = {
-  onHide: () => mixed,
-  file: ?string,
-  logger: Logger,
+  onHide: () => any;
+  file: string | null | undefined;
+  logger: Logger;
 };
 
 type State = {
-  errorArray: Array<Error>,
-  result: ?{
-    success: boolean,
-    error: ?Error,
-  },
-  statusUpdate: ?string,
-  runInBackground: boolean,
+  errorArray: Array<Error>;
+  result:
+    | {
+        success: boolean;
+        error: Error | null | undefined;
+      }
+    | null
+    | undefined;
+  statusUpdate: string | null | undefined;
+  runInBackground: boolean;
 };
 
 export default class ShareSheetExportFile extends Component<Props, State> {
@@ -112,8 +111,6 @@ export default class ShareSheetExportFile extends Component<Props, State> {
     const mark = 'shareSheetExportFile';
     performance.mark(mark);
     try {
-      // Flow doesn't allow us to check for this earlier because `performance` is untyped
-      // and could presumably do anything.
       if (!this.props.file) {
         return;
       }
@@ -134,8 +131,7 @@ export default class ShareSheetExportFile extends Component<Props, State> {
       );
       this.context.store.dispatch(unsetShare());
       if (this.state.runInBackground) {
-        new window.Notification('Sharable Flipper trace created', {
-          //$FlowFixMe: Already checked that props.file exists
+        new Notification('Sharable Flipper trace created', {
           body: `Flipper trace exported to the ${this.props.file}`,
           requireInteraction: true,
         });
@@ -187,7 +183,7 @@ export default class ShareSheetExportFile extends Component<Props, State> {
           <Container>
             <Title bold>Error</Title>
             <ErrorMessage code>
-              {error?.message || 'File could not be saved.'}
+              {(error && error.message) || 'File could not be saved.'}
             </ErrorMessage>
             <FlexRow>
               <Spacer />