This line of code is repeated in the three source files printProgress.js, sendProgress.js, and MsgComposeCommands.js: percent = parseInt( (aCurTotalProgress*100)/aMaxTotalProgress + .5 ); This can be replaced with: percent = Math.round( (aCurTotalProgress*100)/aMaxTotalProgress ); which is faster and easier to read.
-> XP Apps
Assignee: asa → jaggernaut
Component: Browser-General → XP Apps
QA Contact: asa → paw
Created attachment 120882 [details] [diff] [review] A file was missing
Attachment #120875 - Attachment is obsolete: true
Attachment #120882 - Flags: superreview?(bryner) → superreview+
Comment on attachment 120882 [details] [diff] [review] A file was missing a=asa (on behalf of drivers) for checkin to 1.4beta.
Attachment #120882 - Flags: approval1.4b? → approval1.4b+
Status: NEW → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.