| Index: chrome/android/javatests/src/org/chromium/chrome/browser/translate/TranslateOptionsTest.java
|
| diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/translate/TranslateOptionsTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/translate/TranslateOptionsTest.java
|
| index d908fe7cb8a2899c360ddd2bbb7a342134c20939..a31a9caadf5f2a830530a228feb8d082ad7e66df 100644
|
| --- a/chrome/android/javatests/src/org/chromium/chrome/browser/translate/TranslateOptionsTest.java
|
| +++ b/chrome/android/javatests/src/org/chromium/chrome/browser/translate/TranslateOptionsTest.java
|
| @@ -10,29 +10,24 @@ import android.test.AndroidTestCase;
|
| import org.chromium.base.test.util.Feature;
|
| import org.chromium.chrome.browser.infobar.TranslateOptions;
|
|
|
| -import java.util.ArrayList;
|
| -
|
| /**
|
| * Test for TranslateOptions.
|
| */
|
| public class TranslateOptionsTest extends AndroidTestCase {
|
| private static final boolean ALWAYS_TRANSLATE = true;
|
| - private ArrayList<TranslateOptions.TranslateLanguagePair> mLanguages = null;
|
| + private static final String[] LANGUAGES = {"English", "Spanish", "French"};
|
| + private static final String[] CODES = {"en", "es", "fr"};
|
|
|
| @Override
|
| public void setUp() throws Exception {
|
| super.setUp();
|
| - mLanguages = new ArrayList<TranslateOptions.TranslateLanguagePair>();
|
| - mLanguages.add(new TranslateOptions.TranslateLanguagePair("en", "English"));
|
| - mLanguages.add(new TranslateOptions.TranslateLanguagePair("es", "Spanish"));
|
| - mLanguages.add(new TranslateOptions.TranslateLanguagePair("fr", "French"));
|
| }
|
|
|
| @SmallTest
|
| @Feature({"Translate"})
|
| public void testNoChanges() {
|
| TranslateOptions options =
|
| - new TranslateOptions("en", "es", mLanguages, ALWAYS_TRANSLATE, false);
|
| + TranslateOptions.create("en", "es", LANGUAGES, CODES, ALWAYS_TRANSLATE, false);
|
| assertEquals("English", options.sourceLanguageName());
|
| assertEquals("Spanish", options.targetLanguageName());
|
| assertEquals("en", options.sourceLanguageCode());
|
| @@ -47,7 +42,7 @@ public class TranslateOptionsTest extends AndroidTestCase {
|
| @Feature({"Translate"})
|
| public void testBasicLanguageChanges() {
|
| TranslateOptions options =
|
| - new TranslateOptions("en", "es", mLanguages, !ALWAYS_TRANSLATE, true);
|
| + TranslateOptions.create("en", "es", LANGUAGES, CODES, !ALWAYS_TRANSLATE, true);
|
| options.setTargetLanguage("fr");
|
| options.setSourceLanguage("en");
|
| assertEquals("English", options.sourceLanguageName());
|
| @@ -68,7 +63,7 @@ public class TranslateOptionsTest extends AndroidTestCase {
|
| @Feature({"Translate"})
|
| public void testInvalidLanguageChanges() {
|
| TranslateOptions options =
|
| - new TranslateOptions("en", "es", mLanguages, ALWAYS_TRANSLATE, false);
|
| + TranslateOptions.create("en", "es", LANGUAGES, CODES, ALWAYS_TRANSLATE, false);
|
|
|
| // Same target language as source
|
| assertFalse(options.setTargetLanguage("en"));
|
| @@ -91,7 +86,7 @@ public class TranslateOptionsTest extends AndroidTestCase {
|
| @Feature({"Translate"})
|
| public void testBasicOptionsChanges() {
|
| TranslateOptions options =
|
| - new TranslateOptions("en", "es", mLanguages, !ALWAYS_TRANSLATE, false);
|
| + TranslateOptions.create("en", "es", LANGUAGES, CODES, !ALWAYS_TRANSLATE, false);
|
| assertFalse(options.optionsChanged());
|
| options.toggleNeverTranslateDomainState(true);
|
| assertTrue(options.neverTranslateDomainState());
|
| @@ -116,7 +111,7 @@ public class TranslateOptionsTest extends AndroidTestCase {
|
| @Feature({"Translate"})
|
| public void testInvalidOptionsChanges() {
|
| TranslateOptions options =
|
| - new TranslateOptions("en", "es", mLanguages, ALWAYS_TRANSLATE, false);
|
| + TranslateOptions.create("en", "es", LANGUAGES, CODES, ALWAYS_TRANSLATE, false);
|
|
|
| // Never translate language should not work, but never translate domain
|
| // should
|
|
|