refactor loadSourcText (firefox-devtools#3853)

Author: codehag

src/actions/index.js

@@ -11,6 +11,7 @@ import * as ast from "./ast";
 import * as coverage from "./coverage";
 import * as projectTextSearch from "./project-text-search";
 import * as sourceSearch from "./source-search";
+import * as loadSourceText from "./sources/loadSourceText";
 
 export default Object.assign(
   {},
@@ -24,5 +25,6 @@ export default Object.assign(
   ast,
   coverage,
   projectTextSearch,
-  sourceSearch
+  sourceSearch,
+  loadSourceText
 );

src/actions/sources.js

@@ -13,12 +13,13 @@ import { PROMISE } from "../utils/redux/middleware/promise";
 import assert from "../utils/assert";
 import { remapBreakpoints } from "./breakpoints";
 
-import { setEmptyLines, setSymbols, setOutOfScopeLocations } from "./ast";
+import { setEmptyLines, setOutOfScopeLocations } from "./ast";
 import { syncBreakpoint } from "./breakpoints";
 import { searchSource } from "./project-text-search";
 
 import { getPrettySourceURL, isLoaded } from "../utils/source";
 import { createPrettySource } from "./sources/createPrettySource";
+import { loadSourceText } from "./sources/loadSourceText";
 
 import { prefs } from "../utils/prefs";
 import { removeDocument } from "../utils/editor";
@@ -356,40 +357,6 @@ export function toggleBlackBox(source: Source) {
   };
 }
 
-/**
- * @memberof actions/sources
- * @static
- */
-export function loadSourceText(source: Source) {
-  return async ({ dispatch, getState, client, sourceMaps }: ThunkArgs) => {
-    // Fetch the source text only once.
-    if (source.text) {
-      return Promise.resolve(source);
-    }
-
-    await dispatch({
-      type: "LOAD_SOURCE_TEXT",
-      source: source,
-      [PROMISE]: (async function() {
-        if (sourceMaps.isOriginalId(source.id)) {
-          return await sourceMaps.getOriginalSourceText(source);
-        }
-
-        const response = await client.sourceContents(source.id);
-
-        return {
-          id: source.id,
-          text: response.source,
-          contentType: response.contentType || "text/javascript"
-        };
-      })()
-    });
-
-    await dispatch(setSymbols(source.id));
-    await dispatch(setEmptyLines(source.id));
-  };
-}
-
 /**
   Load the text for all the avaliable sources
  * @memberof actions/sources

src/actions/sources/loadSourceText.js

@@ -0,0 +1,38 @@
+// @flow
+import { PROMISE } from "../../utils/redux/middleware/promise";
+import { setEmptyLines, setSymbols } from "../ast";
+import type { Source } from "../../types";
+import type { ThunkArgs } from "../types";
+/**
+ * @memberof actions/sources
+ * @static
+ */
+export function loadSourceText(source: Source) {
+  return async ({ dispatch, getState, client, sourceMaps }: ThunkArgs) => {
+    // Fetch the source text only once.
+    if (source.text) {
+      return Promise.resolve(source);
+    }
+
+    await dispatch({
+      type: "LOAD_SOURCE_TEXT",
+      source: source,
+      [PROMISE]: (async function() {
+        if (sourceMaps.isOriginalId(source.id)) {
+          return await sourceMaps.getOriginalSourceText(source);
+        }
+
+        const response = await client.sourceContents(source.id);
+
+        return {
+          id: source.id,
+          text: response.source,
+          contentType: response.contentType || "text/javascript"
+        };
+      })()
+    });
+
+    await dispatch(setSymbols(source.id));
+    await dispatch(setEmptyLines(source.id));
+  };
+}