Closed
Bug 442830
Opened 16 years ago
Closed 16 years ago
modules/distribution.js: int and bool prefs are not set correctly
Categories
(Firefox :: General, defect)
Firefox
General
Tracking
()
VERIFIED
FIXED
Firefox 3.1a1
People
(Reporter: u217243, Assigned: u217243)
Details
(Keywords: verified1.9.0.2)
Attachments
(1 file)
1.19 KB,
patch
|
hello
:
review+
benjamin
:
review+
mconnor
:
approval1.9.0.2+
|
Details | Diff | Splinter Review |
I have tested a partner build and found that integer and boolean prefs specified in the distribution.ini are not set correctly. Here's the patch.
Updated•16 years ago
|
Component: Release Engineering → General
Product: mozilla.org → Firefox
QA Contact: release → general
Version: other → Trunk
Comment 1•16 years ago
|
||
Dan, could you please review Kohei's patch? Apologies, I thought you were on this bug.
Attachment #327527 -
Flags: review?(thunder)
Updated•16 years ago
|
Assignee: thunder → yoshino
Comment 2•16 years ago
|
||
Comment on attachment 327527 [details] [diff] [review] fix bsmedberg is probably a more legal reviewer for this.
Attachment #327527 -
Flags: review?(benjamin)
Comment 3•16 years ago
|
||
dan wrote it, I would like him to review the changes, then I'll ask for a formal review.
Comment 5•16 years ago
|
||
Comment on attachment 327527 [details] [diff] [review] fix Oops. Yes, looks good.
Attachment #327527 -
Flags: review?(thunder) → review+
Comment 6•16 years ago
|
||
(In reply to comment #4) > ダンさん、レビューお願いします。 パッチをありがとうございます! :-)
I don't have a CVS commit privilege, so please checkin my patch along with an another patch in Bug 436540. よろしくお願いします :)
Updated•16 years ago
|
Attachment #327527 -
Flags: review?(benjamin) → review+
Updated•16 years ago
|
Flags: blocking1.9.0.2+
Updated•16 years ago
|
Attachment #327527 -
Flags: approval1.9.0.2+
Updated•16 years ago
|
Keywords: checkin-needed
Comment 8•16 years ago
|
||
pushed to mozilla-central: http://hg.mozilla.org/index.cgi/mozilla-central/rev/b2233053a941
Comment 9•16 years ago
|
||
CVS HEAD (1.9.0 branch): Checking in browser/components/distribution.js; /cvsroot/mozilla/browser/components/distribution.js,v <-- distribution.js new revision: 1.2; previous revision: 1.1 done
Status: NEW → RESOLVED
Closed: 16 years ago
Keywords: checkin-needed → fixed1.9.0.2
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3.1a1
Comment 10•16 years ago
|
||
Verified fix on Mozilla/5.0 (Windows; U; Windows NT 5.1; ja-JP-mac; rv:1.9.0.2pre) Gecko/2008080705 GranParadiso/3.0.2pre. The code changes are shown in distribution.js
Status: RESOLVED → VERIFIED
Keywords: fixed1.9.0.2 → verified1.9.0.2
You need to log in
before you can comment on or make changes to this bug.
Description
•