LanguageState should be owned by TranslateManager

components/translate/core/browser/translate_manager.h

Index: components/translate/core/browser/translate_manager.h
diff --git a/components/translate/core/browser/translate_manager.h b/components/translate/core/browser/translate_manager.h
index c29b167775b106075f813a3bf79558a8dd437e2a..cbbfb4bff51b9b4683f6e52b75b2e5bcaa2abac1 100644
--- a/components/translate/core/browser/translate_manager.h
+++ b/components/translate/core/browser/translate_manager.h
@@ -13,6 +13,7 @@
 #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;
@@ -88,6 +89,8 @@ class TranslateManager {
   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,
@@ -108,6 +111,8 @@ class TranslateManager {
   TranslateClient* translate_client_;  // Weak.
   TranslateDriver* translate_driver_;  // Weak.
 
+  LanguageState language_state_;
+
   base::WeakPtrFactory<TranslateManager> weak_method_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(TranslateManager);