641854 - Use nsIStringBundle API for formatting strings

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Mark Finkle (:mfinkle) (use needinfo?)]

Attachment: patch

We missed (or added by mistake) a string formatting call that uses the wrong API. Results in:

Warning: WARN addons.manager: InstallListener threw exception when calling onInstallEnded: TypeError: strings.getFormattedString is not a function
Source File: chrome://browser/content/extensions.js
Line: 937

Don't be fooled, this is an error and could be keeping the add-ons manager from properly clearing updates.

Comment 1

[Mark Finkle (:mfinkle) (use needinfo?)]

Comment on attachment 519421 [details] [diff] [review]
patch

low risk patch that could also fix some UI issues while updating add-ons

Comment 2

[Mark Finkle (:mfinkle) (use needinfo?)]

pushed:
http://hg.mozilla.org/mobile-browser/rev/1f1e7d646564

Comment 3

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Verified Fixed:
Mozilla/5.0 (Android; Linux armv71; rv2.0b13pre) Gecko/20110316 Firefox/4.0b13pre Fennec/4.0b6pre
Device: Droid 2 
OS: Android 2.2


Steps to reproduce : 
1. on build 20110315 go to people.mozilla.com/~nhirata/fennec/addons/
2. install fennec-sample-a.xpi
3. restart when asked
4. when fennec comes back up, go to control panel -> addons manager
5. select update
6. go to the error console and look at the warnings.

Expected: no warnings
Actual: 
Warning: WARN addons.manager: InstallListener threw exception when calling
onInstallEnded: TypeError: strings.getFormattedString is not a function
Source File: chrome://browser/content/extensions.js
Line: 937

Note: the plugin is mfinkle's I just modded it so that it works with the more recent builds of fennec.

Comment 4

[Dão Gottwald [:dao]]

bugspam