PRExpectEndValue is a typo for PEExpectEndValue

VERIFIED FIXED in mozilla1.9.1b4

Status

()

Core
CSS Parsing and Computation
--
minor
VERIFIED FIXED
9 years ago
9 years ago

People

(Reporter: timeless, Assigned: timeless)

Tracking

({verified1.9.1})

Trunk
mozilla1.9.1b4
verified1.9.1
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment)

(Assignee)

Description

9 years ago
Created attachment 371411 [details] [diff] [review]
correct typo

nsIPersistentProperties.getStringProperty]" nsresult: "0x80004005 (NS_ERROR_FAILURE)" thrown from function formatStringFromName(length=integer:1, params=Array:{0}, aName=string:"PRExpectEndValue")

http://mxr.mozilla.org/mozilla-central/search?string=ExpectEndValue

ExpectEndValue

/layout/style/nsCSSParser.cpp
    * line 1091 -- REPORT_UNEXPECTED_TOKEN(PEExpectEndValue);
    * line 1335 -- REPORT_UNEXPECTED_TOKEN(PRExpectEndValue);
/dom/locales/en-US/chrome/layout/css.properties
    * line 46 -- PEExpectEndValue=Expected end of value but found '%1$S'.

Found 3 matching lines in 2 files 

I noticed this because I've replaced the stringbundle implementation and saw the exception
Attachment #371411 - Flags: review?(dbaron)
(Assignee)

Comment 1

9 years ago
:( the bug number is wrong in the patch. off by one error :)
Attachment #371411 - Flags: superreview+
Attachment #371411 - Flags: review?(dbaron)
Attachment #371411 - Flags: review+
Attachment #371411 - Flags: approval1.9.1+
Comment on attachment 371411 [details] [diff] [review]
correct typo

r+sr+a1.9.1=dbaron
http://hg.mozilla.org/mozilla-central/rev/b427fd627d36
Status: ASSIGNED → RESOLVED
Last Resolved: 9 years ago
OS: Windows XP → All
Hardware: x86 → All
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.2a1
Keywords: checkin-needed
Whiteboard: [checkin-needed on mozilla-1.9.1]
http://hg.mozilla.org/releases/mozilla-1.9.1/rev/17dbbc102f97
Keywords: checkin-needed → fixed1.9.1
Whiteboard: [checkin-needed on mozilla-1.9.1]
Target Milestone: mozilla1.9.2a1 → mozilla1.9.1b4
(Assignee)

Updated

9 years ago
Status: RESOLVED → VERIFIED
Keywords: fixed1.9.1 → verified1.9.1
You need to log in before you can comment on or make changes to this bug.