Title: [181819] trunk/Source/WebInspectorUI
Revision
181819
Author
nvasil...@apple.com
Date
2015-03-20 16:38:03 -0700 (Fri, 20 Mar 2015)

Log Message

Web Inspector: Fast typing lags in the Styles sidebar or Console
https://bugs.webkit.org/show_bug.cgi?id=142919

Asynchronous autocomplete causes a race condition in CodeMirror,
which results in skipped characters in while typing. Completing immediately
fixes the issues.

Reviewed by Timothy Hatcher.

* UserInterface/Controllers/CodeMirrorCompletionController.js:
(WebInspector.CodeMirrorCompletionController.prototype.hideCompletions):
(WebInspector.CodeMirrorCompletionController.prototype._completeAtCurrentPosition):
(WebInspector.CodeMirrorCompletionController.prototype.):

Modified Paths

Diff

Modified: trunk/Source/WebInspectorUI/ChangeLog (181818 => 181819)


--- trunk/Source/WebInspectorUI/ChangeLog	2015-03-20 23:37:51 UTC (rev 181818)
+++ trunk/Source/WebInspectorUI/ChangeLog	2015-03-20 23:38:03 UTC (rev 181819)
@@ -1,3 +1,19 @@
+2015-03-20  Nikita Vasilyev  <nvasil...@apple.com>
+
+        Web Inspector: Fast typing lags in the Styles sidebar or Console
+        https://bugs.webkit.org/show_bug.cgi?id=142919
+
+        Asynchronous autocomplete causes a race condition in CodeMirror,
+        which results in skipped characters in while typing. Completing immediately
+        fixes the issues.
+
+        Reviewed by Timothy Hatcher.
+
+        * UserInterface/Controllers/CodeMirrorCompletionController.js:
+        (WebInspector.CodeMirrorCompletionController.prototype.hideCompletions):
+        (WebInspector.CodeMirrorCompletionController.prototype._completeAtCurrentPosition):
+        (WebInspector.CodeMirrorCompletionController.prototype.):
+
 2015-03-19  Joseph Pecoraro  <pecor...@apple.com>
 
         Web Inspector: Adopt ES6 Class Syntax for all Model Objects

Modified: trunk/Source/WebInspectorUI/UserInterface/Controllers/CodeMirrorCompletionController.js (181818 => 181819)


--- trunk/Source/WebInspectorUI/UserInterface/Controllers/CodeMirrorCompletionController.js	2015-03-20 23:37:51 UTC (rev 181818)
+++ trunk/Source/WebInspectorUI/UserInterface/Controllers/CodeMirrorCompletionController.js	2015-03-20 23:38:03 UTC (rev 181819)
@@ -169,11 +169,6 @@
         this._implicitSuffix = "";
         this._forced = false;
 
-        if (this._completionDelayTimeout) {
-            clearTimeout(this._completionDelayTimeout);
-            delete this._completionDelayTimeout;
-        }
-
         delete this._currentCompletion;
         delete this._ignoreNextCursorActivity;
     },
@@ -443,11 +438,6 @@
             return;
         }
 
-        if (this._completionDelayTimeout) {
-            clearTimeout(this._completionDelayTimeout);
-            delete this._completionDelayTimeout;
-        }
-
         this._removeCompletionHint(true, true);
 
         var cursor = this._codeMirror.getCursor();
@@ -787,15 +777,7 @@
         if (change.origin === "+delete" && !this._hasPendingCompletion())
             return;
 
-        if (this._completionDelayTimeout) {
-            clearTimeout(this._completionDelayTimeout);
-            delete this._completionDelayTimeout;
-        }
-
-        if (this._hasPendingCompletion())
-            this._completeAtCurrentPosition(false);
-        else
-            this._completionDelayTimeout = setTimeout(this._completeAtCurrentPosition.bind(this, false), WebInspector.CodeMirrorCompletionController.CompletionTypingDelay);
+        this._completeAtCurrentPosition(false);
     },
 
     _handleCursorActivity: function(codeMirror)
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to