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

Issue 615201 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Feb 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

Syntax highlighting for mojom files

Project Member Reported by dsansome@chromium.org, May 26 2016

Issue description

Status: Available (was: Untriaged)

Comment 2 by emso@chromium.org, Sep 15 2016

Labels: Lang Visualization
Cc: roc...@chromium.org jam@chromium.org yzshen@chromium.org
+some mojo people
Project Member

Comment 4 by sheriffbot@chromium.org, Sep 18 2017

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available. If you change it back, also remove the "Hotlist-Recharge-Cold" label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 5 by emso@chromium.org, Sep 20 2017

Status: Available (was: Untriaged)

Comment 6 by awdf@chromium.org, Jan 17 2018

+1 for this.

Could the fix be as simple as to make codesearch treat .mojom files like .cc files wrt syntax highlighting? Even if it's not perfect this would be a net improvement - I have my vim configured to display mojom files with c++ syntax highlighting and haven't noticed any oddness yet. 

Owner: aga...@chromium.org
Status: Started (was: Available)
I think cl/185202806 is all this will take!

Comment 8 by aga...@chromium.org, Feb 12 2018

Status: Fixed (was: Started)
The change above has been submitted; it will probably be a week or more until it deploys. I'm marking this as fixed, then snoozing the email thread for this bug for a couple weeks. When it pops back into my inbox, I'll double check and mark this as verified.

Sign in to add a comment