New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 725484 link

Starred by 1 user

Issue metadata

Status: Duplicate
Merged: issue 724586
Owner:
Closed: Nov 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 2
Type: Bug



Sign in to add a comment

Console - regex in a class which extends an object is highlighted like a comment

Reported by hakerh403@gmail.com, May 23 2017

Issue description

UserAgent: Mozilla/5.0 (Windows NT 6.3; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/58.0.3029.110 Safari/537.36

Steps to reproduce the problem:
Test case:
    +class extends{1:{}/ /\//}{};

What is the expected behavior?
Regex should be highlighted like a regex.

What went wrong?
It is highlighted like a comment.

Did this work before? No 

Chrome version: 58.0.3029.110  Channel: stable
OS Version: 6.3
Flash Version: No flash at all
 

Comment 1 by hakerh403@gmail.com, May 23 2017

Similar issue:
    (a={}/ /\//)=>a;

Comment 2 by hakerh403@gmail.com, May 23 2017

Another one:
    {}/./gym?
    {}/ /\//:
    {}/ /\//;

To generalize, every time when division is placed after an object, it is interpreted like start of a regex.
Labels: Needs-Triage-M58
Cc: mmanchala@chromium.org
 Issue 725482  has been merged into this issue.
Labels: Needs-Feedback
hakerh403@ : Could you please provide us any sample test file or html,actual and expected screen shots to triage the issue from test team end.

Thanks.

Comment 6 by hakerh403@gmail.com, May 24 2017

I am posting sample test JS file, actual results and what are expected results.
1.zip
86.7 KB Download
Project Member

Comment 7 by sheriffbot@chromium.org, May 24 2017

Labels: -Needs-Feedback
Thank you for providing more feedback. Adding requester "mmanchala@chromium.org" to the cc list and removing "Needs-Feedback" label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Cc: kavvaru@chromium.org
 Issue 725466  has been merged into this issue.
Labels: TE-NeedsTriageHelp
Owner: l...@chromium.org
Status: Assigned (was: Unconfirmed)

Comment 11 by l...@chromium.org, Sep 19 2017

Thanks for the report.  DevTools currently uses CodeMirror for syntax highlighting.  It looks like the fix for this belongs with CodeMirror.  Could you please file an issue on  their Github ( https://github.com/codemirror/CodeMirror/issues )?  Once it is fixed there, the improved highlighting will be included when we do another CodeMirror roll.
Reported: https://github.com/codemirror/CodeMirror/issues/4981
I've also reported examples from Chromium  issue 724586 , so you may merge that issue into this one if you want.

Comment 13 by l...@chromium.org, Nov 15 2017

Mergedinto: 724586
Status: Duplicate (was: Assigned)

Sign in to add a comment