Merge branch 'main' into iterable-iterator-pathset

mstange · web-flow · commit d64e75f95bce · 2025-08-25T12:24:31.000-04:00
diff --git a/package.json b/package.json
@@ -62,10 +62,10 @@
     "@codemirror/lang-cpp": "^6.0.3",
     "@codemirror/lang-javascript": "^6.2.4",
     "@codemirror/lang-rust": "^6.0.2",
-    "@codemirror/language": "^6.11.2",
+    "@codemirror/language": "^6.11.3",
     "@codemirror/state": "^6.5.2",
     "@codemirror/view": "^6.38.1",
-    "@firefox-devtools/react-contextmenu": "^5.2.2",
+    "@firefox-devtools/react-contextmenu": "^5.2.3",
     "@fluent/bundle": "^0.19.1",
     "@fluent/langneg": "^0.7.0",
     "@fluent/react": "^0.15.2",
@@ -77,7 +77,7 @@
     "classnames": "^2.5.1",
     "common-tags": "^1.8.2",
     "copy-to-clipboard": "^3.3.3",
-    "core-js": "^3.44.0",
+    "core-js": "^3.45.0",
     "escape-string-regexp": "^4.0.0",
     "gecko-profiler-demangle": "^0.3.3",
     "idb": "^8.0.3",
@@ -89,7 +89,7 @@
     "mixedtuplemap": "^1.0.0",
     "namedtuplemap": "^1.0.0",
     "photon-colors": "^3.3.2",
-    "protobufjs": "^7.5.3",
+    "protobufjs": "^7.5.4",
     "query-string": "^9.2.2",
     "react": "^18.3.1",
     "react-dom": "^18.3.1",
@@ -106,24 +106,24 @@
     "workbox-window": "^7.3.0"
   },
   "devDependencies": {
-    "@babel/cli": "^7.28.0",
-    "@babel/core": "^7.28.0",
+    "@babel/cli": "^7.28.3",
+    "@babel/core": "^7.28.3",
     "@babel/eslint-parser": "^7.28.0",
     "@babel/eslint-plugin": "^7.27.1",
     "@babel/plugin-proposal-class-properties": "^7.18.6",
-    "@babel/preset-env": "^7.28.0",
+    "@babel/preset-env": "^7.28.3",
     "@babel/preset-react": "^7.27.1",
     "@babel/preset-typescript": "^7.27.1",
     "@fetch-mock/jest": "^0.2.16",
-    "@testing-library/dom": "^10.4.0",
+    "@testing-library/dom": "^10.4.1",
     "@testing-library/jest-dom": "^6.6.4",
     "@testing-library/react": "^16.3.0",
     "@types/clamp": "^1.0.3",
     "@types/common-tags": "^1.8.4",
     "@types/jest": "^30.0.0",
     "@types/minimist": "^1.2.5",
     "@types/query-string": "^6.3.0",
-    "@types/react": "^18.3.1",
+    "@types/react": "^18.3.24",
     "@types/react-dom": "^18.3.1",
     "@types/react-splitter-layout": "^4.0.0",
     "@types/react-transition-group": "^4.4.5",
@@ -133,14 +133,14 @@
     "@typescript-eslint/parser": "^8.38.0",
     "alex": "^11.0.1",
     "autoprefixer": "^10.4.21",
-    "babel-jest": "^30.0.4",
+    "babel-jest": "^30.0.5",
     "babel-loader": "^10.0.0",
     "babel-plugin-module-resolver": "^5.0.2",
-    "browserslist": "^4.25.1",
-    "caniuse-lite": "^1.0.30001727",
+    "browserslist": "^4.25.2",
+    "caniuse-lite": "^1.0.30001731",
     "circular-dependency-plugin": "^5.2.1",
     "codecov": "^3.8.3",
-    "copy-webpack-plugin": "^13.0.0",
+    "copy-webpack-plugin": "^13.0.1",
     "cross-env": "^10.0.0",
     "css-loader": "^7.1.2",
     "cssnano": "^7.1.0",
@@ -153,15 +153,15 @@
     "eslint-plugin-jest-dom": "^5.5.0",
     "eslint-plugin-jest-formatting": "^3.1.0",
     "eslint-plugin-react": "^7.37.5",
-    "eslint-plugin-testing-library": "^7.6.0",
+    "eslint-plugin-testing-library": "^7.6.3",
     "espree": "^10.4.0",
-    "fake-indexeddb": "^6.0.1",
+    "fake-indexeddb": "^6.1.0",
     "file-loader": "^6.2.0",
     "glob": "^11.0.3",
-    "html-webpack-plugin": "^5.6.3",
+    "html-webpack-plugin": "^5.6.4",
     "husky": "^4.3.8",
-    "jest": "^30.0.4",
-    "jest-environment-jsdom": "^30.0.4",
+    "jest": "^30.0.5",
+    "jest-environment-jsdom": "^30.0.5",
     "jest-extended": "^6.0.0",
     "json-loader": "^0.5.7",
     "local-web-server": "^5.4.0",
@@ -177,18 +177,18 @@
     "raw-loader": "^4.0.2",
     "rimraf": "^5.0.10",
     "style-loader": "^4.0.0",
-    "stylelint": "^16.21.1",
+    "stylelint": "^16.23.0",
     "stylelint-config-idiomatic-order": "^10.0.0",
-    "stylelint-config-standard": "^38.0.0",
+    "stylelint-config-standard": "^39.0.0",
     "typescript": "^5.8.3",
-    "webpack": "^5.100.2",
+    "webpack": "^5.101.3",
     "webpack-cli": "^6.0.1",
     "webpack-dev-server": "^5.2.2",
     "workbox-webpack-plugin": "^7.3.0",
     "yargs": "^18.0.0"
   },
   "resolutions": {
-    "@types/react-splitter-layout/@types/react": "^18.3.1",
+    "@types/react-splitter-layout/@types/react": "^18.3.24",
     "@types/trusted-types": "^2.0.7"
   },
   "jest": {
diff --git a/src/components/app/AppHeader.tsx b/src/components/app/AppHeader.tsx
@@ -22,7 +22,6 @@ export class AppHeader extends React.PureComponent<{}> {
             id="AppHeader--app-header"
             elems={{
               header: (
-                // $FlowExpectError Flow doesn't know about this fluent rule for react component.
                 <InnerNavigationLink
                   dataSource="none"
                   className="appHeaderLink"
diff --git a/src/components/app/CompareHome.css b/src/components/app/CompareHome.css
@@ -17,7 +17,7 @@
 .compareHomeForm {
   display: grid;
   align-items: center;
-  grid-gap: 1em;
+  gap: 1em;
   grid-template-columns: auto 1fr;
 }
 
diff --git a/src/components/app/Home.tsx b/src/components/app/Home.tsx
@@ -660,7 +660,6 @@ class HomeImpl extends React.PureComponent<HomeProps, HomeState> {
                 id="Home--compare-recordings-info"
                 elems={{
                   a: (
-                    // $FlowExpectError Flow doesn't know about this fluent rule for react component.
                     <InnerNavigationLink dataSource="compare">
                       Compare
                     </InnerNavigationLink>
diff --git a/src/components/app/KeyboardShortcut.css b/src/components/app/KeyboardShortcut.css
@@ -64,7 +64,7 @@
 .appKeyboardShortcutsContent {
   display: grid;
   margin: 40px;
-  grid-gap: 80px;
+  gap: 80px;
   grid-template-columns: 1fr 1fr;
 }
 
diff --git a/src/components/flame-graph/Canvas.css b/src/components/flame-graph/Canvas.css
@@ -18,7 +18,7 @@
 
 .flameGraphCanvasTooltip > .tooltipHeader {
   display: grid;
-  grid-gap: 2px 0;
+  gap: 2px 0;
   grid-template-columns: min-content auto;
 }
 
diff --git a/src/components/js-tracer/Chart.tsx b/src/components/js-tracer/Chart.tsx
@@ -104,7 +104,6 @@ class JsTracerExpensiveChartImpl extends React.PureComponent<Props> {
         chartProps={{
           jsTracerTimingRows,
           jsTracerTable,
-          // $FlowFixMe Error introduced by upgrading to v0.96.0. See issue #1936.
           updatePreviewSelection,
           rangeStart: timeRange.start,
           rangeEnd: timeRange.end,
diff --git a/src/components/marker-chart/index.tsx b/src/components/marker-chart/index.tsx
@@ -153,7 +153,6 @@ class MarkerChartImpl extends React.PureComponent<Props> {
                 getMarker,
                 getMarkerLabel,
                 markerListLength,
-                // $FlowFixMe Error introduced by upgrading to v0.96.0. See issue #1936.
                 updatePreviewSelection,
                 changeMouseTimePosition,
                 changeRightClickedMarker,
diff --git a/src/components/shared/CodeView.css b/src/components/shared/CodeView.css
@@ -128,9 +128,9 @@
 .cm-content {
   font-family: ui-monospace, 'Roboto Mono', monospace;
   hyphens: none;
+  overflow-wrap: normal;
   word-break: normal;
   word-spacing: normal;
-  word-wrap: normal;
 }
 
 /**
diff --git a/src/components/sidebar/sidebar.css b/src/components/sidebar/sidebar.css
@@ -21,7 +21,7 @@
   display: grid;
   align-content: start; /* the grid isn't vertically stretched */
   align-items: center;
-  grid-gap: 2px 5px;
+  gap: 2px 5px;
   grid-template-columns:
     /* Label */
     1fr
@@ -51,7 +51,7 @@
 
 .sidebar-title-label {
   display: grid;
-  grid-gap: 5px;
+  gap: 5px;
   grid-template-columns: min-content 1fr;
 }
 
diff --git a/src/components/stack-chart/index.tsx b/src/components/stack-chart/index.tsx
@@ -266,7 +266,6 @@ class StackChartImpl extends React.PureComponent<Props> {
                   combinedTimingRows,
                   sameWidthsIndexToTimestampMap,
                   getMarker,
-                  // $FlowFixMe Error introduced by upgrading to v0.96.0. See issue #1936.
                   updatePreviewSelection,
                   changeMouseTimePosition,
                   rangeStart: timeRange.start,
diff --git a/src/components/tooltip/CallNode.css b/src/components/tooltip/CallNode.css
@@ -7,7 +7,7 @@
   align-items: center;
   padding-bottom: 10px;
   border-top: 1px solid var(--grey-40);
-  grid-gap: 4px;
+  gap: 4px;
   grid-template-columns: repeat(4, min-content);
 }
 
diff --git a/src/components/tooltip/NetworkMarker.css b/src/components/tooltip/NetworkMarker.css
@@ -11,7 +11,7 @@
   padding-top: 4px;
   border-top: 1px solid var(--grey-40);
   margin-top: 4px;
-  grid-gap: 2px 5px;
+  gap: 2px 5px;
 
   /* When changing the number of columns, don't forget to change the span values
    * in grid-column-start properties. */
diff --git a/src/components/tooltip/Tooltip.css b/src/components/tooltip/Tooltip.css
@@ -66,7 +66,7 @@
 
 .tooltipDetails {
   display: grid;
-  grid-gap: 2px 5px;
+  gap: 2px 5px;
 
   /* Make the right column "definitely" sized (ignore its intrinsic size) by setting a min-width of 0.
    * This stops long contents from overflowing the tooltip. */
@@ -129,7 +129,7 @@
 .sidebar .tooltipDetails {
   /* Force the details to the max-width, even if they have long pieces of text. */
   overflow: hidden;
-  grid-gap: 2px 0;
+  gap: 2px 0;
 
   /* Don't let the width overflow for long pieces of text */
   grid-template-columns: minmax(0, 1fr);
diff --git a/src/components/tooltip/TrackPower.tsx b/src/components/tooltip/TrackPower.tsx
@@ -142,7 +142,6 @@ class TooltipTrackPowerImpl extends React.PureComponent<Props> {
     );
 
     return (
-      // $FlowExpectError our version of Flow doesn't understand Fragments very well.
       <>
         {this._formatPowerValue(
           powerSumForPreviewRange,
diff --git a/src/profile-logic/merge-compare.ts b/src/profile-logic/merge-compare.ts
@@ -1363,9 +1363,6 @@ function mergeMarkers(
           );
         }
 
-        // Flow doesn't know well how to handle the spread operator with our
-        // MarkerPayload type.
-        // $FlowExpectError
         newMarkerTable.data.push({
           ...oldData,
           cause: {
diff --git a/src/test/components/AppLocalizationProvider.test.tsx b/src/test/components/AppLocalizationProvider.test.tsx
@@ -70,7 +70,6 @@ describe('AppLocalizationProvider', () => {
       await screen.findByText(translatedText('en-US'))
     ).toBeInTheDocument();
     expect(document.documentElement).toHaveAttribute('lang', 'en-US');
-    // $FlowExpectError Our version of flow doesn't know about document.dir
     expect(document.dir).toBe('ltr');
 
     // Now we're testing the LTR pseudo-localization.
@@ -79,7 +78,6 @@ describe('AppLocalizationProvider', () => {
     });
     expect(await screen.findByText('Ŧħīş īş ḗḗƞ-ŬŞ Ŧḗḗẋŧ')).toBeInTheDocument();
     expect(document.documentElement).toHaveAttribute('lang', 'en-US');
-    // $FlowExpectError Our version of flow doesn't know about document.dir
     expect(document.dir).toBe('ltr');
 
     // And now the RTL pseudo-localization.
@@ -88,7 +86,6 @@ describe('AppLocalizationProvider', () => {
     });
     expect(await screen.findByText(/⊥ɥıs ıs ǝu-∩S ⊥ǝxʇ/)).toBeInTheDocument();
     expect(document.documentElement).toHaveAttribute('lang', 'en-US');
-    // $FlowExpectError Our version of flow doesn't know about document.dir
     expect(document.dir).toBe('rtl');
 
     // Back to no pseudo-localization
@@ -99,7 +96,6 @@ describe('AppLocalizationProvider', () => {
       await screen.findByText(translatedText('en-US'))
     ).toBeInTheDocument();
     expect(document.documentElement).toHaveAttribute('lang', 'en-US');
-    // $FlowExpectError Our version of flow doesn't know about document.dir
     expect(document.dir).toBe('ltr');
   });
 
@@ -136,7 +132,6 @@ describe('AppLocalizationProvider', () => {
       await screen.findByText(translatedText('en-US'))
     ).toBeInTheDocument();
     expect(document.documentElement).toHaveAttribute('lang', 'en-US');
-    // $FlowExpectError Our version of flow doesn't know about document.dir
     expect(document.dir).toBe('ltr');
 
     // Switch to german
@@ -146,7 +141,6 @@ describe('AppLocalizationProvider', () => {
     });
     expect(await screen.findByText(translatedText('de'))).toBeInTheDocument();
     expect(document.documentElement).toHaveAttribute('lang', 'de');
-    // $FlowExpectError Our version of flow doesn't know about document.dir
     expect(document.dir).toBe('ltr');
   });
 
diff --git a/src/test/components/BeforeUnloadManager.test.tsx b/src/test/components/BeforeUnloadManager.test.tsx
@@ -38,10 +38,8 @@ describe('app/BeforeUnloadManager', () => {
     const event = new Event('beforeunload');
 
     /* Because of the current jsdom implementation, we need to mock
-       preventDefault(). To prevent a Flow error when assigning jest.fn()
-       to event.preventDefault (not writeable), we add the following line.
+       preventDefault().
     */
-    // $FlowExpectError
     event.preventDefault = jest.fn();
 
     fireEvent(window as any, event);
diff --git a/src/test/components/FooterLinks.test.tsx b/src/test/components/FooterLinks.test.tsx
@@ -19,7 +19,6 @@ beforeEach(() => {
     .get(fetchUrlRe, ({ url }: { url: string }) => {
       const matchUrlResult = fetchUrlRe.exec(url);
       if (matchUrlResult) {
-        // $FlowExpectError Our Flow doesn't know about named groups.
         const { language } = matchUrlResult.groups!;
         const path = `locales/${language}/app.ftl`;
         if (fs.existsSync(path)) {
diff --git a/src/test/components/GlobalTrack.test.tsx b/src/test/components/GlobalTrack.test.tsx
@@ -180,7 +180,6 @@ describe('timeline/GlobalTrack', function () {
   it('displays the private browsing status of a fission thread', () => {
     setup(PRIVATE_TRACK_INDEX);
     const track = screen.getByText('Private');
-    // $FlowExpectError The parent element is an HTML Element but Flow doesn't know that.
     expect(ensureExists(track.parentElement).title).toBe(stripIndent`
       Private
       Thread: "Private" (6666)
@@ -192,7 +191,6 @@ describe('timeline/GlobalTrack', function () {
   it('displays the container status of a fission thread', () => {
     setup(CONTAINER_TRACK_INDEX);
     const track = screen.getByText('InContainer');
-    // $FlowExpectError The parent element is an HTML Element but Flow doesn't know that.
     expect(ensureExists(track.parentElement).title).toBe(stripIndent`
       InContainer
       Thread: "InContainer" (7777)
diff --git a/src/test/components/MenuButtons.test.tsx b/src/test/components/MenuButtons.test.tsx
@@ -416,7 +416,6 @@ describe('app/MenuButtons', function () {
 
     it('matches the snapshot for a compression error', async () => {
       const { compress } = require('firefox-profiler/utils/gz');
-      // $FlowExpectError Flow doesn't know about Jest mocks
       compress.mockRejectedValue(new Error('Compression error'));
       jest.spyOn(console, 'error').mockImplementation(() => {});
       const { getPanel, openPublishPanel } = setupForPublish();
diff --git a/src/test/components/ServiceWorkerManager.test.tsx b/src/test/components/ServiceWorkerManager.test.tsx
@@ -43,7 +43,6 @@ describe('app/ServiceWorkerManager', () => {
 
     // It seems node v8 doesn't let us change the value unless we delete it before.
     delete (window as any).location;
-    // $FlowExpectError because the value we pass isn't a proper Location object.
     Object.defineProperty(window, 'location', {
       value: { reload: jest.fn() },
       writable: true,
diff --git a/src/test/components/Viewport.test.tsx b/src/test/components/Viewport.test.tsx
@@ -623,8 +623,6 @@ function setup(profileOverrides: MixedObject = {}) {
 
   // Hook up a dummy chart with a viewport.
   const DummyChart = jest.fn((_props) => <div id="dummy-chart" />);
-  // Flow's internal structures for React don't recognize our mock function as a
-  // valid stateless component. $FlowExpectError
   const ChartWithViewport = withChartViewport(DummyChart);
 
   type OwnProps = {
diff --git a/src/test/fixtures/mocks/indexeddb.ts b/src/test/fixtures/mocks/indexeddb.ts
@@ -9,9 +9,8 @@ import { IDBFactory } from 'fake-indexeddb';
 
 function resetIndexedDb() {
   // This is the recommended way to reset the IDB state between test runs, but
-  // neither flow nor eslint like that we assign to indexedDB directly, for
-  // different reasons.
-  /* $FlowExpectError */ /* eslint-disable-next-line no-global-assign */
+  // eslint does not like that we assign to indexedDB directly.
+  /* eslint-disable-next-line no-global-assign */
   indexedDB = new IDBFactory();
 }
 
diff --git a/src/test/fixtures/mocks/intersection-observer.ts b/src/test/fixtures/mocks/intersection-observer.ts
diff --git a/src/test/fixtures/mocks/window-navigation.ts b/src/test/fixtures/mocks/window-navigation.ts
diff --git a/src/test/integration/symbolicator-cli/symbolicator-cli.test.ts b/src/test/integration/symbolicator-cli/symbolicator-cli.test.ts
diff --git a/src/test/unit/process-profile.test.ts b/src/test/unit/process-profile.test.ts
diff --git a/src/utils/types.ts b/src/utils/types.ts
diff --git a/yarn.lock b/yarn.lock

Original file line number	Diff line number	Diff line change
`@@ -17,7 +17,7 @@`
`17`	`17`	`.compareHomeForm {`
`18`	`18`	`display: grid;`
`19`	`19`	`align-items: center;`
`20`		`- grid-gap: 1em;`
	`20`	`+ gap: 1em;`
`21`	`21`	`grid-template-columns: auto 1fr;`
`22`	`22`	`}`
`23`	`23`
Original file line number	Diff line number	Diff line change
`@@ -64,7 +64,7 @@`
`64`	`64`	`.appKeyboardShortcutsContent {`
`65`	`65`	`display: grid;`
`66`	`66`	`margin: 40px;`
`67`		`- grid-gap: 80px;`
	`67`	`+ gap: 80px;`
`68`	`68`	`grid-template-columns: 1fr 1fr;`
`69`	`69`	`}`
`70`	`70`
Original file line number	Diff line number	Diff line change
`@@ -18,7 +18,7 @@`
`18`	`18`
`19`	`19`	`.flameGraphCanvasTooltip > .tooltipHeader {`
`20`	`20`	`display: grid;`
`21`		`- grid-gap: 2px 0;`
	`21`	`+ gap: 2px 0;`
`22`	`22`	`grid-template-columns: min-content auto;`
`23`	`23`	`}`
`24`	`24`