LanguageState should be owned by TranslateManager

components/translate/core/browser/translate_manager.h

 // Copyright 2014 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.
 
 #ifndef COMPONENTS_TRANSLATE_CORE_BROWSER_TRANSLATE_MANAGER_H_
 #define COMPONENTS_TRANSLATE_CORE_BROWSER_TRANSLATE_MANAGER_H_
 
 #include <map>
 #include <string>
 #include <vector>
 
 #include "base/callback_list.h"
 #include "base/gtest_prod_util.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/weak_ptr.h"
+#include "components/translate/core/browser/language_state.h"
 #include "components/translate/core/common/translate_errors.h"
 
 class GURL;
 class PrefService;
 class TranslateClient;
 class TranslateDriver;
 class TranslatePrefs;
 struct TranslateErrorDetails;
 
 // The TranslateManager class is responsible for showing an info-bar when a page
@@ skipping 55 matching lines @@
   // Callback types for translate errors.
   typedef base::Callback<void(const TranslateErrorDetails&)>
       TranslateErrorCallback;
   typedef base::CallbackList<void(const TranslateErrorDetails&)>
       TranslateErrorCallbackList;
 
   // Registers a callback for translate errors.
   static scoped_ptr<TranslateErrorCallbackList::Subscription>
       RegisterTranslateErrorCallback(const TranslateErrorCallback& callback);
 
+  LanguageState& GetLanguageState();

[droger, 2014/05/21 09:13:25]

Can you add a comment for this method?

[nshaik, 2014/05/21 15:26:15]

Done.

+
  private:
   // Sends a translation request to the TranslateDriver.
   void DoTranslatePage(const std::string& translate_script,
                        const std::string& source_lang,
                        const std::string& target_lang);
 
   // Called when the Translate script has been fetched.
   // Initiates the translation.
   void OnTranslateScriptFetchComplete(int page_id,
                                       const std::string& source_lang,
                                       const std::string& target_lang,
                                       bool success,
                                       const std::string& data);
 
   // Preference name for the Accept-Languages HTTP header.
   std::string accept_languages_pref_name_;
 
   TranslateClient* translate_client_;  // Weak.
   TranslateDriver* translate_driver_;  // Weak.
 
+  LanguageState language_state_;
+
   base::WeakPtrFactory<TranslateManager> weak_method_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(TranslateManager);
 };
 
 #endif  // COMPONENTS_TRANSLATE_CORE_BROWSER_TRANSLATE_MANAGER_H_