Editor unresponsive when clicked on commented CSS while live preview is active

In release 43 (OSX Mavericks), the editor becomes unresponsive when you click on a multi-line comment (that comments out a valid class/id) in CSS while having the live preview running. I tried replicating the bug without plugins and it still occurs.

@RaymondLim

@RaymondLim Cannot reproduce unresponsiveness on 64bit Ubuntu 14.04. (on master ece9bbf).

I do see a 'Unable to load live preview page' dialog; but this happens after the index.html of the Getting Started project has rendered, although the Brackets' console says this (full stack trace below), but I'm not sure what this means - other than that would probably be the file that Brackets' errors on.

GET http://127.0.0.1:9222/json  Inspector.js:266
    getDebuggableWindows Inspector.js:266
    connectToURL Inspector.js:348
    _openInterstitialPage LiveDevelopment.js:1125
    _doLaunchAfterServerReady LiveDevelopment.js:1242
    (anonymous function) LiveDevelopment.js:1343
    j jquery-2.1.0.min.js:2
    k.fireWith jquery-2.1.0.min.js:2
    e.(anonymous function) jquery-2.1.0.min.js:2
    (anonymous function) jquery-2.1.0.min.js:2
    j jquery-2.1.0.min.js:2
    k.fireWith jquery-2.1.0.min.js:2
    e.(anonymous function) jquery-2.1.0.min.js:2
    onSuccess StaticServer.js:124
    (anonymous function) StaticServer.js:149
    j jquery-2.1.0.min.js:2
    k.fireWith jquery-2.1.0.min.js:2
    NodeConnection._receive

@Mark-Simulacrum

@Mark-Simulacrum That's a rather common error message, which is probably not part of the issue.
Apparently nobody knows what it means :D See #8611.

@Mark-Simulacrum

@Mark-Simulacrum I found a more reproducible recipe that works for me on Win. Could you try it?

Create the two files i.html and i.css, with the contents:

<html>
    <head>
        <link rel="stylesheet" href="i.css" type="text/css">
    </head>
    <body>
        Some text.
    </body>
</html>

/* body {
    background-color: black;
    color: white;
} */

Start live preview on i.html and wait
Switch to i.css
Remove */ on line 4, then press Backspace again

Result: The editor becomes unresponsive

@RaymondLim

git bisect points to @RaymondLim's c954fb4.

@marcelgerber

@marcelgerber Yes, your test case does allow me to reproduce the hang. Will test the PR branch soon.

@Mark-Simulacrum

@Mark-Simulacrum @marcelgerber Since you both repro'ed this earlier, would one of you mind confirming the fix before we close it? (I tested it a bit during PR review, but this was a nasty enough bug that it never hurts to get some extra eyes on it). Thanks!

@peterflynn

@peterflynn @RaymondLim I'm sorry to say, but using the files from above recipe, the bug got even worse.
Latest master (03ab969) on Win 8.1

Create the two files i.html and i.css, with the contents:

<html>
    <head>
        <link rel="stylesheet" href="i.css" type="text/css">
    </head>
    <body>
        Some text.
    </body>
</html>

/* body {
    background-color: black;
    color: white;
} */

Start live preview on i.html and wait
Switch to i.css
Click after */

Result: The editor becomes unresponsive

@RaymondLim

Ugh, sorry -- my fault. My last suggestion for simplifying the loop was not quite right. The stopping condition needs to exactly match how TokenUtils decides whether to stop & return false or not.

@RaymondLim thee potential ways to fix that:

Just change the loop to match movePrevToken() properly -- i.e. while (!(ctx.pos.line <= 0 && (ctx.pos.ch <= 0 || ctx.token.start <= 0)))
Same, but wrap that up in a TokenUtils.isAtStart() utility for clarity
Change movePrevToken() to more consistently stop at the start of the stream, instead of stopping one token sooner when it knows that's the last token available. (This seems more maintainable in the long run, but a tad riskier because it would affect other callers... albeit callers that are probably well unit-tested).

@RaymondLim

@RaymondLim I can post a fix using approach 2 above -- just proofed it out.

Looks good now.

@liaujianjie

Closing as fixed. @liaujianjie let us know if you still see problems.

Navigation Menu

Repository navigation

Editor unresponsive when clicked on commented CSS while live preview is active #9002

1 remaining item

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Editor unresponsive when clicked on commented CSS while live preview is active #9002

Description

Activity

Mark-Simulacrum commented on Sep 7, 2014

marcelgerber commented on Sep 7, 2014

marcelgerber commented on Sep 7, 2014

marcelgerber commented on Sep 7, 2014

Mark-Simulacrum commented on Sep 7, 2014

1 remaining item

peterflynn commented on Sep 25, 2014

marcelgerber commented on Sep 25, 2014

peterflynn commented on Sep 25, 2014

peterflynn commented on Sep 25, 2014

marcelgerber commented on Sep 29, 2014

JeffryBooher commented on Sep 29, 2014

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions