Bug 92623 - Web Inspector: font names that start with a @ in CSS font-face are not properly handled.
Summary: Web Inspector: font names that start with a @ in CSS font-face are not proper...
Status: RESOLVED WORKSFORME
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks: 116924
  Show dependency treegraph
 
Reported: 2012-07-30 02:20 PDT by Alexander Pavlov (apavlov)
Modified: 2016-12-04 16:24 PST (History)
14 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Alexander Pavlov (apavlov) 2012-07-30 02:20:07 PDT
What steps will reproduce the problem?
1. Open Inspector on http://blogs.msdn.com/b/oldnewthing/archive/2012/07/19/10331400.aspx
2. Inspect the element with the "font-face: SimSun" style.
3. Check the styles sidebar.
4. Attempt to edit the font-face and change the name.
5. Inspect the element with the "font-face: @SimSun" style.
6. Check the styles sidebar.
7. Attempt to edit the font-face and set it to the correct name.

What is the expected result?

The second element's font-face is displayed as @SimSun and both entries can be edited the same way.

What happens instead?

Font-face is displayed as "@SimSunn", though in the Elements view it is correct.  If you attempt to put a @ in a font name you are editing, Inspector crosses the line out and says the value is invalid.  You must escape the @ with a \ (as seen in the Elements view) for it to be accepted.  Editing the value will also add extra "n" on the end.

Upstreaming http://code.google.com/p/chromium/issues/detail?id=138077
Comment 1 Alexander Pavlov (apavlov) 2012-07-30 04:59:51 PDT
void CSSParser::parseEscape(UChar*& result) rewrites the input string, based on which CSSParser computes the actual (source) property name and value. According to zherczeg who rewrote the lexer earlier this year, this is a huge refactoring and doesn't seem reasonable at the moment.
Comment 2 Radar WebKit Bug Importer 2014-12-01 14:36:10 PST
<rdar://problem/19107002>
Comment 3 Blaze Burg 2016-12-04 16:24:01 PST
Unable to repro. Closing.