Change how QuickEdit helper function is passed from CodeHints to QuickEdit:

    It now gets set directly as brackets._jsCodeHintsHelper = quickEditHelpe;.

as suggested.

Author: jeffkenton

src/editor/EditorManager.js

@@ -94,12 +94,6 @@ define(function (require, exports, module) {
      */
     var _inlineEditProviders = [];
 
-    /**
-     * QuickEdit helper function.
-     * @type {function}
-     */
-    var _quickEditHelper = null;
-    
     /**
      * Registered inline documentation widget providers. See {@link #registerInlineDocsProvider()}.
      * @type {Array.<function(...)>}
@@ -221,22 +215,6 @@ define(function (require, exports, module) {
         _inlineEditProviders.push(provider);
     }
 
-    /**
-     * Registers a helper for QuickEdit to provide a hook into Tern's jump-to-defintion.
-     * 
-     * @param {function}
-     */
-    function registerQuickEditHelper(helper) {
-        _quickEditHelper = helper;
-    }
-
-    /**
-     * Return QuickEdit helper.
-     */
-    function getQuickEditHelper() {
-        return _quickEditHelper;
-    }
-
     /**
      * Registers a new inline docs provider. When Quick Docs is invoked each registered provider is
      * asked if it wants to provide inline docs given the current editor and cursor location.
@@ -754,8 +732,6 @@ define(function (require, exports, module) {
     exports.getFocusedInlineWidget = getFocusedInlineWidget;
     exports.resizeEditor = resizeEditor;
     exports.registerInlineEditProvider = registerInlineEditProvider;
-    exports.registerQuickEditHelper = registerQuickEditHelper;
-    exports.getQuickEditHelper = getQuickEditHelper;
     exports.registerInlineDocsProvider = registerInlineDocsProvider;
     exports.getInlineEditors = getInlineEditors;
     exports.closeInlineWidget = closeInlineWidget;

src/extensions/default/JavaScriptCodeHints/main.js

@@ -587,7 +587,7 @@ define(function (require, exports, module) {
         CommandManager.register(Strings.CMD_JUMPTO_DEFINITION, JUMPTO_DEFINITION, handleJumpToDefinition);
 
         // Register quickEditHelper.
-        EditorManager.registerQuickEditHelper(quickEditHelper);
+        brackets._jsCodeHintsHelper = quickEditHelper;
 
         // Add the menu item
         var menu = Menus.getMenu(Menus.AppMenuBar.NAVIGATE_MENU);

src/extensions/default/JavaScriptQuickEdit/main.js

@@ -105,7 +105,7 @@ define(function (require, exports, module) {
      */
     function _createInlineEditor(hostEditor, functionName) {
         // Use Tern jump-to-definition helper, if it's available, to find InlineEditor target.
-        var helper = EditorManager.getQuickEditHelper();
+        var helper = brackets._jsCodeHintsHelper;
         if (helper === null) {
             return null;
         }