Back out Electron 7.1.1/7.1.2 upgrades

Summary:
Original commit changeset: eddfbb3571c7
Original commit changeset: f7300f527b65

Pains me to do this, but probably our best chance right now to get a release out.

Reviewed By: cekkaewnumchai

Differential Revision: D18687890

fbshipit-source-id: 05d13aca71457ca6dd19f5b45d58158ae223e66f

package.json

@@ -41,7 +41,7 @@
     ]
   },
   "resolutions": {
-    "@jest-runner/electron/electron": "7.1.2",
+    "@jest-runner/electron/electron": "5.0.11",
     "adbkit/adbkit-logcat": "2",
     "@types/react": "16.9.11",
     "@types/react-dom": "16.9.2"
@@ -60,7 +60,7 @@
     "clearMocks": true
   },
   "devDependencies": {
-    "@jest-runner/electron": "^2.0.2",
+    "@jest-runner/electron": "^2.0.1",
     "@testing-library/react": "^9.3.0",
     "@types/algoliasearch": "^3.30.19",
     "@types/deep-equal": "^1.0.1",
@@ -85,8 +85,8 @@
     "@typescript-eslint/parser": "^2.1.1-alpha.4",
     "babel-code-frame": "^6.26.0",
     "babel-eslint": "^10.0.1",
-    "electron": "7.1.2",
-    "electron-builder": "22.1.0",
+    "electron": "^5.0.11",
+    "electron-builder": "^20.38.5",
     "eslint": "^6.5.1",
     "eslint-config-fbjs": "^3.1.1",
     "eslint-plugin-babel": "^5.3.0",
@@ -185,6 +185,8 @@
   },
   "greenkeeper": {
     "ignore": [
+      "electron",
+      "electron-builder",
       "emotion",
       "react-emotion",
       "tmp"

src/MenuBar.tsx

@@ -108,7 +108,7 @@ export function setupMenuBar(
       const menu = applicationMenu.items.find(
         menuItem => menuItem.label === topLevelMenu,
       );
-      if (menu && menu.submenu) {
+      if (menu) {
         const menuItem = menu.submenu.items.find(
           menuItem => menuItem.label === label,
         );
@@ -270,7 +270,7 @@ function getTemplate(
         {
           label: 'Select All',
           accelerator: 'CmdOrCtrl+A',
-          role: 'selectAll',
+          role: 'selectall',
         },
       ],
     },
@@ -384,7 +384,7 @@ function getTemplate(
   ];
 
   if (process.platform === 'darwin') {
-    const name = app.name;
+    const name = app.getName();
     template.unshift({
       label: name,
       submenu: [
@@ -411,7 +411,7 @@ function getTemplate(
         {
           label: 'Hide Others',
           accelerator: 'Command+Shift+H',
-          role: 'hideOthers',
+          role: 'hideothers',
         },
         {
           label: 'Show All',

src/chrome/settings/configFields.tsx

@@ -7,7 +7,6 @@
  * @format
  */
 
-import electron from 'electron';
 import {FlexColumn, styled, Text, FlexRow, Input, colors, Glyph} from 'flipper';
 import React, {useState} from 'react';
 import {promises as fs} from 'fs';
@@ -88,18 +87,16 @@ export function FilePathConfigField(props: {
       />
       <FlexColumn
         onClick={() =>
-          remote.dialog
-            .showOpenDialog({
+          remote.dialog.showOpenDialog(
+            {
               properties: ['openDirectory', 'showHiddenFiles'],
               defaultPath: path.resolve('/'),
-            })
-            .then((result: electron.SaveDialogReturnValue) => {
-              if (result.filePath) {
-                const path: string = result.filePath.toString();
-                setValue(path);
-                props.onChange(path);
-              }
-            })
+            },
+            (paths: Array<string> | undefined) => {
+              paths && setValue(paths[0]);
+              paths && props.onChange(paths[0]);
+            },
+          )
         }>
         <CenteredGlyph name="dots-3-circle" variant="outline" />
       </FlexColumn>

src/dispatcher/application.tsx

@@ -7,7 +7,7 @@
  * @format
  */
 
-import {remote, ipcRenderer, IpcRendererEvent} from 'electron';
+import {remote, ipcRenderer} from 'electron';
 import {toggleAction} from '../reducers/application';
 import {setStaticView} from '../reducers/connections';
 import {Store} from '../reducers/index.js';
@@ -57,7 +57,7 @@ export default (store: Store, logger: Logger) => {
 
   ipcRenderer.on(
     'flipper-protocol-handler',
-    (_event: IpcRendererEvent, query: string) => {
+    (_event: string, query: string) => {
       const uri = new URL(query);
       if (query.startsWith('flipper://import')) {
         const {search} = new URL(query);
@@ -102,14 +102,11 @@ export default (store: Store, logger: Logger) => {
     },
   );
 
-  ipcRenderer.on(
-    'open-flipper-file',
-    (_event: IpcRendererEvent, url: string) => {
+  ipcRenderer.on('open-flipper-file', (_event: string, url: string) => {
     tryCatchReportPlatformFailures(() => {
       return importFileToStore(url, store);
     }, `${IMPORT_FLIPPER_TRACE_EVENT}:Deeplink`);
-    },
-  );
+  });
 
   if (process.env.FLIPPER_PORTS) {
     const portOverrides = parseFlipperPorts(process.env.FLIPPER_PORTS);

src/dispatcher/notifications.tsx

@@ -12,7 +12,7 @@ import {Logger} from '../fb-interfaces/Logger';
 import {PluginNotification} from '../reducers/notifications';
 import {FlipperPlugin, FlipperDevicePlugin} from '../plugin';
 import isHeadless from '../utils/isHeadless';
-import {ipcRenderer, IpcRendererEvent} from 'electron';
+import {ipcRenderer} from 'electron';
 import {selectPlugin} from '../reducers/connections';
 import {
   setActiveNotifications,
@@ -37,7 +37,6 @@ export default (store: Store, logger: Logger) => {
   ipcRenderer.on(
     'notificationEvent',
     (
-      _event: IpcRendererEvent,
       _e: Error,
       eventName: NotificationEvents,
       pluginNotification: PluginNotification,

src/utils/exportData.tsx

@@ -690,9 +690,8 @@ export function showOpenDialog(store: Store) {
     properties: ['openFile'],
     filters: [{extensions: ['flipper', 'json', 'txt'], name: 'Flipper files'}],
   };
-  remote.dialog.showOpenDialog(options).then(result => {
-    const filePaths = result.filePaths;
-    if (filePaths.length > 0) {
+  remote.dialog.showOpenDialog(options, (filePaths?: Array<string>) => {
+    if (filePaths !== undefined && filePaths.length > 0) {
       tryCatchReportPlatformFailures(() => {
         importFileToStore(filePaths[0], store);
       }, `${IMPORT_FLIPPER_TRACE_EVENT}:UI`);
@@ -701,17 +700,14 @@ export function showOpenDialog(store: Store) {
 }
 
 export function startFileExport(dispatch: Store['dispatch']) {
-  electron.remote.dialog
-    .showSaveDialog(
+  electron.remote.dialog.showSaveDialog(
     // @ts-ignore This appears to work but isn't allowed by the types
     null,
     {
       title: 'FlipperExport',
       defaultPath: path.join(os.homedir(), 'FlipperExport.flipper'),
     },
-    )
-    .then(async (result: electron.SaveDialogReturnValue) => {
-      const file = result.filePath;
+    async (file: string) => {
       if (!file) {
         return;
       }
@@ -722,7 +718,8 @@ export function startFileExport(dispatch: Store['dispatch']) {
           closeOnFinish: false,
         }),
       );
-    });
+    },
+  );
 }
 
 export function startLinkExport(dispatch: Store['dispatch']) {