DevTools: Give autocomplete suggestions even after brackets

third_party/WebKit/Source/devtools/front_end/components/ExecutionContextSelector.js

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 /**
  * @constructor
  * @implements {WebInspector.TargetManager.Observer}
  * @param {!WebInspector.TargetManager} targetManager
  * @param {!WebInspector.Context} context
  */
@@ skipping 186 matching lines @@
  */
 WebInspector.ExecutionContextSelector.completionsForTextPromptInCurrentContext = function(proxyElement, text, cursorOffset, wordRange, force, completionsReadyCallback)
 {
     var executionContext = WebInspector.context.flavor(WebInspector.ExecutionContext);
     if (!executionContext) {
         completionsReadyCallback([]);
         return;
     }
 
     // Pass less stop characters to rangeOfWord so the range will be a more complete expression.
-    var expressionRange = wordRange.startContainer.rangeOfWord(wordRange.startOffset, " =:({;,!+-*/&|^<>", proxyElement, "backward");
+    var expressionRange = wordRange.startContainer.rangeOfWord(wordRange.startOffset, " =:({;,!+-*/&|^<>`", proxyElement, "backward");

[lushnikov, 2016/07/20 23:37:54]

did you test this new apostrophe?

[einbinder, 2016/07/22 23:26:46]

Done.

     var expressionString = expressionRange.toString();
 
-    // The "[" is also a stop character, except when it's the last character of the expression.
-    var pos = expressionString.lastIndexOf("[", expressionString.length - 2);
-    if (pos !== -1)
-        expressionString = expressionString.substr(pos + 1);
+    var bracketCount = 0;
+    for (var i = expressionString.length - 1; i >= 0; i--) {

[lushnikov, 2016/07/20 23:37:54]

1. let's extract "i" outside of the loop and name it with a full word ("index"?)
2. I'd convert this into a while loop:

var bracketCount = 0;
var index = expressionString.length;
while (bracketCount >= 0 && index >= 0) {
    --index;
    ...
}

[einbinder, 2016/07/22 23:26:46]

Done.

+        switch (expressionString.charAt(i)) {

[lushnikov, 2016/07/20 23:37:54]

converting this into if statement would be easier to read:

if (char === "[" and index < expressionString.length < 1)
    --bracketCount;
else if (char === "]")
    ++bracketCount;

[einbinder, 2016/07/22 23:26:45]

Done.

+        case "]":
+            bracketCount++;
+            break;
+        case "[":
+            // Allow an open bracket at the end for property completion

[lushnikov, 2016/07/20 23:37:54]

nit: comments should finish with "."

[einbinder, 2016/07/22 23:26:45]

Done.

+            if (i < expressionString.length - 1)
+                bracketCount--;
+            break;
+        }
+        if (bracketCount < 0)
+            break;
+    }
+    expressionString = expressionString.substr(i + 1);
 
     var prefix = wordRange.toString();
     executionContext.completionsForExpression(expressionString, text, cursorOffset, prefix, force, completionsReadyCallback);
 }