Closed
Bug 140141
Opened 23 years ago
Closed 23 years ago
Switching language/content packs is broken (once again)
Categories
(Core :: Internationalization: Localization, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: kairo, Assigned: rchen)
Details
(Keywords: intl, regression)
Attachments
(1 file)
1.84 KB,
patch
|
Biesinger
:
review+
alecf
:
superreview+
|
Details | Diff | Splinter Review |
The checkin of http://bugzilla.mozilla.org/attachment.cgi?id=77717&action=view
of bug 119923 broke pref-contentpacks.xul once again, as .selectLocale moved
from nsIChromeRegistry to nsIXULChromeRegistry, and this file was forgotten in
the patch. Adding alecf to CC as it was his work.
I have a patch, attaching it in a minute.
Reporter | ||
Comment 1•23 years ago
|
||
Reporter | ||
Comment 2•23 years ago
|
||
can someone provide r= and sr= here? thanks!
(BTW, this is trunk only, as the mentioned is not in the branch)
Comment 3•23 years ago
|
||
Comment on attachment 81040 [details] [diff] [review]
patch: change nsIChromeRegistry to nsIXULChromeRegistry
argh.
sr=alecf
Attachment #81040 -
Flags: superreview+
Updated•23 years ago
|
Attachment #81040 -
Flags: review+
Comment 4•23 years ago
|
||
Comment on attachment 81040 [details] [diff] [review]
patch: change nsIChromeRegistry to nsIXULChromeRegistry
r=biesi
Comment 5•23 years ago
|
||
Checking in resources/content/pref-contentpacks.xul;
/cvsroot/mozilla/extensions/content-packs/resources/content/pref-contentpacks.xul,v
<-- pref-contentpacks.xul
new revision: 1.18; previous revision: 1.17
done
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•