Chromium Code Reviews| Index: chrome/browser/spellchecker/spellcheck_service.h |
| diff --git a/chrome/browser/spellchecker/spellcheck_service.h b/chrome/browser/spellchecker/spellcheck_service.h |
| index c7229e8b468b03a55676e5ad7ab141c84c5f89cb..c1a22f33486f92d05bd87a4e5169d104ebf04441 100644 |
| --- a/chrome/browser/spellchecker/spellcheck_service.h |
| +++ b/chrome/browser/spellchecker/spellcheck_service.h |
| @@ -63,20 +63,17 @@ class SpellcheckService : public KeyedService, |
| base::WeakPtr<SpellcheckService> GetWeakPtr(); |
| - // This function computes a vector of strings which are to be displayed in |
| - // the context menu over a text area for changing spell check languages. It |
| - // returns the index of the current spell check language in the vector. |
| + // Computes |languages| to display in the context menu over a text area for |
| + // changing spellcheck languages. Returns the number of the first spellcheck |
| + // |languages| that are enabled. |
| + // |languages| will begin with all enabled spellcheck languages, followed by |
|
please use gerrit instead
2015/06/06 01:42:12
Delete lines 69 to 72. They seem redundant.
Julius
2015/06/12 20:10:14
Done.
|
| + // all other |accept_languages| not currently enabled for spellchecking. |
| + // Therefore, it is only necessary to know how many of the first elements in |
| + // |languages| are enabled dictionary languages. |
| // TODO(port): this should take a vector of base::string16, but the |
| // implementation has some dependencies in l10n util that need porting first. |
| - static int GetSpellCheckLanguages(content::BrowserContext* context, |
| - std::vector<std::string>* languages); |
| - |
| - // Computes a vector of strings which are to be displayed in the context |
| - // menu from |accept_languages| and |dictionary_language|. |
| - static void GetSpellCheckLanguagesFromAcceptLanguages( |
| - const std::vector<std::string>& accept_languages, |
| - const std::string& dictionary_language, |
| - std::vector<std::string>* languages); |
| + static size_t GetSpellCheckLanguages(content::BrowserContext* context, |
| + std::vector<std::string>* languages); |
| // Signals the event attached by AttachTestEvent() to report the specified |
| // event to browser tests. This function is called by this class and its |