Fixed failure to add language in language settings page.

chrome/browser/resources/options2/language_list.js

Index: chrome/browser/resources/options2/language_list.js
diff --git a/chrome/browser/resources/options2/language_list.js b/chrome/browser/resources/options2/language_list.js
index 17fc9863cb57454a2b44cc55f75513052b5788da..72e5f214704110ae5fdf26c6bfbaa8675aaa7bcc 100644
--- a/chrome/browser/resources/options2/language_list.js
+++ b/chrome/browser/resources/options2/language_list.js
@@ -70,7 +70,7 @@ cr.define('options', function() {
       this.languageCodeToLanguageInfo_ = {};
       var languageList = templateData.languageList;
       for (var i = 0; i < languageList.length; i++) {
-        this.languageCodeToLanguageInfo_[languageCode] = languageList[i];
+        this.languageCodeToLanguageInfo_[language.code] = languageList[i];

[csilv, 2012/04/27 21:29:48]

I'm confused by this change, can you explain it?

[Danh Nguyen, 2012/04/27 21:38:52]

The parameter languageCode is a language code of the language to be looked up. 
It doesn't change in this function's scope.  Meanwhile, language.code is the
language code of the current language when we iterate the language list in the
for loop.  So it changes inside the loop.  Without this change, the mapping from
language code to language info will have only one entry, indexed by
languageCode, which is incorrect.

       }
     }