[Webkit-unassigned] [Bug 26350] Make WebInspector's console evaluation/completion asynchronous.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sun Jun 14 00:33:33 PDT 2009


pfeldman at chromium.org changed:

           What    |Removed                     |Added
  Attachment #31232|0                           |1
        is obsolete|                            |
  Attachment #31254|                            |review?
               Flag|                            |

------- Comment #5 from pfeldman at chromium.org  2009-06-14 00:33 PDT -------
Created an attachment (id=31254)
 --> (https://bugs.webkit.org/attachment.cgi?id=31254&action=view)

(In reply to comment #4)
> (From update of attachment 31232 [review])
> This can just be written as:
> setTimeout(completionsReadyCallback, 0, results);


> The open brace should be on the next line for these.


> I think it would be cleaner to define a nested function then call setTimeout on
> it, not using an anonymous function inline.


> Same here.


> > -
> > + 
> Extra space added here.

Not sure which one you mean. How do I find exact line number?

> Does this work when case dosen't match? The completions are found
> case-insenitvly and indexOf is case-sensitive.

In fact, I think that completions are also case-sensitive. At least that is
what I've learned from playing with the tool.

> I think a better way to detect instead of looking at all the completion
> prefixes, would be to get the word range that was present when completions was
> made. If the word range matches the current fullWordRange, then it is the right
> one. Otherwise return early when there isn't a match and don't call
> this.clearAutoComplete in that case.

Done. Regardless of case sensitivity I could do it this way.

Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

More information about the webkit-unassigned mailing list