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;
 
+namespace content {
+struct LoadCommittedDetails;
+class NavigationController;
+class WebContents;
+}

[droger, 2014/05/20 07:43:07]

Remove all these declarations.

[nshaik, 2014/05/20 22:33:53]

Done.

+
 // The TranslateManager class is responsible for showing an info-bar when a page
 // in a language different than the user language is loaded.  It triggers the
 // page translation the user requests.
 
 class TranslateManager {
  public:
   // |translate_client| is expected to outlive the TranslateManager.
   // |accept_language_pref_name| is the path for the preference for the
   // accept-languages.
   TranslateManager(TranslateClient* translate_client,
@@ skipping 46 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();
+
  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_