style="Foo: bar'" incorrectly diagnosed ('%1$S' instead of "'")

RESOLVED FIXED in mozilla12

Status

()

Core
CSS Parsing and Computation
P4
normal
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: Philip TAYLOR, Assigned: dbaron)

Tracking

unspecified
mozilla12
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

5 years ago
User Agent: Mozilla/5.0 (Windows NT 5.1; rv:9.0.1) Gecko/20111221 Firefox/9.0.1 SeaMonkey/2.6.1
Build ID: 20111221001913

Steps to reproduce:

A source file includes the line :

<DIV style="width: 500px; border: none; font: 18px/1.0 'Palatino
Linotype'; color: #000066; text-decoration: none'" id="Text">


Actual results:

The error console reports :

> Warning: Expected ';' to terminate declaration but found '%1$S'. 
> Declaration dropped.
> Source File:
> http://porphyrogenitus.org/Etheridge/Multifolio/Index-11.html
> Line: 0



Expected results:

The error console should report :

> Warning: Expected ';' or '"' to terminate declaration but found "'". 
> Declaration dropped.
> Source File:
> http://porphyrogenitus.org/Etheridge/Multifolio/Index-11.html
> Line: 0

Updated

5 years ago
Component: General → Style System (CSS)
QA Contact: general → style-system
(Assignee)

Comment 1

5 years ago
Created attachment 590511 [details]
testcase
(Assignee)

Comment 2

5 years ago
Created attachment 590512 [details] [diff] [review]
patch
Attachment #590512 - Flags: review?(bzbarsky)
Comment on attachment 590512 [details] [diff] [review]
patch

r=me
Attachment #590512 - Flags: review?(bzbarsky) → review+
Assignee: nobody → dbaron
Status: UNCONFIRMED → NEW
Ever confirmed: true
(Assignee)

Comment 4

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/4b3f1baa77fc
OS: Windows XP → All
Priority: -- → P4
Hardware: x86 → All
Target Milestone: --- → mozilla11
https://hg.mozilla.org/mozilla-central/rev/4b3f1baa77fc
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: mozilla11 → mozilla12
You need to log in before you can comment on or make changes to this bug.