From Bugzilla Helper: User-Agent: Mozilla/5.0 (Windows; U; WinNT4.0; en-US; rv:0.9.9) Gecko/20020311 BuildID: 2002031104 After mis-typing a URL so that it began with "chome://" instead of "chrome://", the error message reported was "c is not a registered protocol"; i.e. only the first character of the incorrect protocol was reported. Reproducible: Always Steps to Reproduce: Type 'chome://' in the URL bar (or any non-registered protocol). Actual Results: The error message shows only the first character of the incorrect protocol. Expected Results: The full protocol name should be displayed.
*** Bug 131285 has been marked as a duplicate of this bug. ***
qa to me. Sigh. I really wanted error messages to be robust for mozilla 1.0, but there are so many of these getting through... I've seen this as well in 0.9.9.
appstrings.properties appears to be using a lower-case %s throughout when - at least for this instance - it should be using uppercase %S. Most of the other strings are processed in nsWebShell which calls snprintf for itself, but in this instance docshell is calling nsIStringBundle::FormatStringFromName which expects only unichar strings. Patch follows.
We're in localization lockdown, but it might be possible to sneak this into 1.0, so I'll mark it as such for the time being.
Lining this up to go in 1.0 if possible. Waiting on the sr but pushing it through as quickly as possible for consideration. Summary - very low risk, visible impact
Comment on attachment 76772 [details] [diff] [review] Patch uses %S email@example.com
Comment on attachment 76772 [details] [diff] [review] Patch uses %S a=asa (on behalf of drivers) for checkin to the 1.0 trunk
Lori, can you add your approval as far as localization is concerned? Nominate for adt1.0.0+. Low risk, big impact.
a=UI adding impact status. This bug prevents users from figuring out what to do when they make a typo in the URL field. cc'ing mcarlson for localization approval
adt1.0.0+ (on ADT's behalf) approval for checkin to 1.0, pending L10N approval. Changing ADT3
meant "+" ... mia culpa.
Can you please check in this change today. l10n approved for check in today. thanks
1.0 is branching at this moment. I have asked for re-approval and assuming it is given the checkin should go in when the tree has reopened (which will be tomorrow for me)
Fix checked into trunk, still waiting for branch approval
Fix checked into branch
adding fixed1.0.0 keyword (branch resolution). This bug has comments saying it was fixed on the 1.0 branch and a bonsai checkin comment that agrees. To verify the bug has been fixed on the 1.0 branch please replace the fixed1.0.0 keyword with verified1.0.0.
*** Bug 155257 has been marked as a duplicate of this bug. ***
VERIFIED/branch - Netscape 7.0 VERIFIED/trunk - via Mozilla 1.1 allplats