Replace PR_MIN/PR_MAX with NS_MIN/NS_MAX in /dom

RESOLVED FIXED in mozilla1.9.3a2

Status

()

--
minor
RESOLVED FIXED
9 years ago
9 years ago

People

(Reporter: lusian, Assigned: lusian)

Tracking

Trunk
mozilla1.9.3a2
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

9 years ago
User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.3a1pre) Gecko/20091120 Minefield/3.7a1pre (.NET CLR 3.5.30729)
Build Identifier: 

Please see Bug #512106.

Reproducible: Always
(Assignee)

Updated

9 years ago
Blocks: 518502
OS: Windows XP → All
Hardware: x86 → All
Version: unspecified → Trunk
(Assignee)

Comment 1

9 years ago
Created attachment 413817 [details] [diff] [review]
Search&Replace, 0
Attachment #413817 - Flags: review?(peterv)
Assignee: nobody → lusian
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
diff --git a/dom/src/json/test/unit/json2.js b/dom/src/json/test/unit/json2.js
--- a/dom/src/json/test/unit/json2.js
+++ b/dom/src/json/test/unit/json2.js
@@ -340,16 +340,19 @@ if (!this.crockfordJSON) {
 // Join all of the member texts together, separated with commas,
 // and wrap them in braces.
 
             v = partial.length === 0 ? '{}' :
                 gap ? '{\n' + gap + partial.join(',\n' + gap) + '\n' +
                         mind + '}' : '{' + partial.join(',') + '}';
             gap = mind;
             return v;
+
+	default:
+	    throw new Error("str: Invalid type " + value);

Drive-by comment: was this intentional?
(Assignee)

Comment 3

9 years ago
I was trying to make xpcshell-tests strict legal, and json2.js caused failures on the JavaScript strict mode (TypeError: function str does not always return a value).  I added the default case after reading the review commment: https://bugzilla.mozilla.org/show_bug.cgi?id=524781#c14
Comment on attachment 413817 [details] [diff] [review]
Search&Replace, 0

Please remove the json2.js change if it's not related to this bug.
Attachment #413817 - Flags: review?(peterv) → review+
(Assignee)

Comment 5

9 years ago
Created attachment 426689 [details] [diff] [review]
updated, chekin please

Sorry for json2; it shouldn't be there.
Attachment #413817 - Attachment is obsolete: true
(Assignee)

Updated

9 years ago
Keywords: checkin-needed
(Assignee)

Updated

9 years ago
Attachment #426689 - Attachment is patch: true
Attachment #426689 - Attachment mime type: application/octet-stream → text/plain
http://hg.mozilla.org/mozilla-central/rev/f8a43ae32718

Thanks!
Status: ASSIGNED → RESOLVED
Last Resolved: 9 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.3a2
You need to log in before you can comment on or make changes to this bug.