Enables the user to select multiple languages for spellchecking (UI)

chrome/browser/spellchecker/spellcheck_service_unittest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <vector>
 
 #include "base/files/scoped_temp_dir.h"
 #include "base/message_loop/message_loop.h"
 #include "chrome/browser/spellchecker/spellcheck_custom_dictionary.h"
 #include "chrome/browser/spellchecker/spellcheck_factory.h"
@@ skipping 23 matching lines @@
   TestingProfile profile_;
 };
 
 TEST_F(SpellcheckServiceTest, GetSpellCheckLanguages1) {
   std::vector<std::string> accept_languages;
   accept_languages.push_back("en");
   accept_languages.push_back("en-US");
   std::vector<std::string> languages;
 
   SpellcheckService::GetSpellCheckLanguagesFromAcceptLanguages(
-      accept_languages, "en-US", &languages);
+      accept_languages, &languages);
 
   EXPECT_EQ(1U, languages.size());
   EXPECT_EQ("en-US", languages[0]);
 }
 
 TEST_F(SpellcheckServiceTest, GetSpellCheckLanguages2) {
   std::vector<std::string> accept_languages;
   accept_languages.push_back("en-US");
   accept_languages.push_back("en");
   std::vector<std::string> languages;
 
   SpellcheckService::GetSpellCheckLanguagesFromAcceptLanguages(
-      accept_languages, "en-US", &languages);
+      accept_languages, &languages);
 
   EXPECT_EQ(1U, languages.size());
   EXPECT_EQ("en-US", languages[0]);
 }
 
 TEST_F(SpellcheckServiceTest, GetSpellCheckLanguages3) {
   std::vector<std::string> accept_languages;
   accept_languages.push_back("en");
   accept_languages.push_back("en-US");
   accept_languages.push_back("en-AU");
   std::vector<std::string> languages;
 
   SpellcheckService::GetSpellCheckLanguagesFromAcceptLanguages(
-      accept_languages, "en-US", &languages);
+      accept_languages, &languages);
 
   EXPECT_EQ(2U, languages.size());
 
   std::sort(languages.begin(), languages.end());
   EXPECT_EQ("en-AU", languages[0]);
   EXPECT_EQ("en-US", languages[1]);
 }
 
 TEST_F(SpellcheckServiceTest, GetSpellCheckLanguages4) {
   std::vector<std::string> accept_languages;
   accept_languages.push_back("en");
   accept_languages.push_back("en-US");
   accept_languages.push_back("fr");
   std::vector<std::string> languages;
 
   SpellcheckService::GetSpellCheckLanguagesFromAcceptLanguages(

[groby-ooo-7-16, 2015/03/03 17:54:34]

There don't seem to be any tests for the return value. If we don't need it at
all, let's void it. Otherwise, please add a test :)

-      accept_languages, "en-US", &languages);
+      accept_languages, &languages);
 
   EXPECT_EQ(2U, languages.size());
 
   std::sort(languages.begin(), languages.end());
   EXPECT_EQ("en-US", languages[0]);
   EXPECT_EQ("fr", languages[1]);
 }
 
 TEST_F(SpellcheckServiceTest, GetSpellCheckLanguages5) {
   std::vector<std::string> accept_languages;
   accept_languages.push_back("en");
   accept_languages.push_back("en-JP");  // Will not exist.
   accept_languages.push_back("fr");
   accept_languages.push_back("aa");  // Will not exist.
   std::vector<std::string> languages;
 
   SpellcheckService::GetSpellCheckLanguagesFromAcceptLanguages(
-      accept_languages, "fr", &languages);
+      accept_languages, &languages);
 
   EXPECT_EQ(1U, languages.size());
   EXPECT_EQ("fr", languages[0]);
 }