357143 - code cleanup: use full literal of introType_major_app_and_version and introType_minor_app in updates.js

Status: RESOLVED FIXED

(Toolkit :: Application Update, defect)

Description

[(not reading, please use [e-mail] instead)]

code cleanup:  use full literal of introType_major_app_and_version and introType_minor_app in updates.js

from dan's review of my patch in #329729


>+    if (severity == "major") {
     ...
>+        "introType_" + severity + "_app_and_version", 

This seems silly, there is only one app_and_version string and we know
severity == "major". It'd be much clearer just to use the full
literal "introType_major_app_and_version"

>+    else {
      ...
>+          "introType_" + severity + "_app", [gUpdates.brandName]);

ditto "introType_minor_app"

see http://lxr.mozilla.org/seamonkey/source/toolkit/mozapps/update/content/updates.js#670

Comment 1

[Ryan Jones-Ward [:sciguyryan]]

Attachment: Patch v1

Patch v1.

Code changes as suggested by Seth.

Comment 2

[(not reading, please use [e-mail] instead)]

ryan, thanks for the patch.  

It looks good, but i'll double check by testing it out before I land it.

Comment 3

[(not reading, please use [e-mail] instead)]

Comment on attachment 242676 [details] [diff] [review]
Patch v1

i've reviewed and tested this patch.

r=sspitzer

thanks ryan!

Comment 4

[(not reading, please use [e-mail] instead)]

fixed.

Checking in updates.js;
/cvsroot/mozilla/toolkit/mozapps/update/content/updates.js,v  <--  updates.js
new revision: 1.70; previous revision: 1.69
done