WIP firefox-devtools#4022 - need to get startline of function

findFunctionText (utils/function) already has all information
borian · Sep 17, 2017 · 7101873 · 7101873
1 parent 834deba
commit 7101873
Show file tree

Hide file tree

Showing 3 changed files with 23 additions and 5 deletions.
diff --git a/src/actions/sources.js b/src/actions/sources.js
@@ -208,7 +208,7 @@ export function selectSource(id: string, options: SelectSourceOptions = {}) {
     }
 
     dispatch(addTab(source.toJS(), 0));
-
+    console.log("selectSource ", options.line, options.tabIndex);
     return dispatch({
       type: "SELECT_SOURCE",
       source: source.toJS(),

diff --git a/src/components/Editor/EditorMenu.js b/src/components/Editor/EditorMenu.js
@@ -1,6 +1,8 @@
 import { showMenu } from "devtools-launchpad";
 import { isOriginalId } from "devtools-source-map";
 import { copyToTheClipboard } from "../../utils/clipboard";
+import { findFunctionText } from "../../utils/function";
+import getSymbols from "../../utils/parser/getSymbols";
 
 function getMenuItems(
   event,
@@ -13,7 +15,8 @@ function getMenuItems(
     jumpToMappedLocation,
     toggleBlackBox,
     addExpression,
-    getFunctionText
+    getFunctionText,
+    selectSource
   }
 ) {
   const copySourceLabel = L10N.getStr("copySource");
@@ -101,7 +104,20 @@ function getMenuItems(
     label: copyFunctionLabel,
     accesskey: copyFunctionKey,
     disabled: !functionText,
-    click: () => copyToTheClipboard(functionText)
+    click: () => {
+      copyToTheClipboard(functionText);
+      console.log("responding");
+      console.log(selectedSource.get("id"), sourceLocation);
+
+      console.log("functionText ", functionText, selectedSource.toJS());
+      selectSource(selectedLocation.sourceId, { line: line + 1 });
+
+      // highlightLineRange({
+      //   start: line + 1,
+      //   end: line + 2,
+      //   sourceId: selectedLocation.sourceId
+      // });
+    }
   };
 
   const menuItems = [

diff --git a/src/components/Editor/index.js b/src/components/Editor/index.js
@@ -343,7 +343,8 @@ class Editor extends PureComponent {
       jumpToMappedLocation,
       addExpression,
       toggleBlackBox,
-      getFunctionText
+      getFunctionText,
+      selectSource
     } = this.props;
 
     return EditorMenu({
@@ -356,7 +357,8 @@ class Editor extends PureComponent {
       addExpression,
       toggleBlackBox,
       getFunctionText,
-      onGutterContextMenu: this.onGutterContextMenu
+      onGutterContextMenu: this.onGutterContextMenu,
+      selectSource
     });
   }