LanguageState should be owned by TranslateManager

chrome/browser/translate/translate_tab_helper.h

 // Copyright 2011 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 CHROME_BROWSER_TRANSLATE_TRANSLATE_TAB_HELPER_H_
 #define CHROME_BROWSER_TRANSLATE_TRANSLATE_TAB_HELPER_H_
 
 #include <string>
 
 #include "base/memory/scoped_ptr.h"
@@ skipping 11 matching lines @@
 #include "base/lazy_instance.h"
 #include "base/synchronization/lock.h"
 #include "base/task_runner.h"
 #endif
 
 namespace base {
 class File;
 }
 
 namespace content {
+struct LoadCommittedDetails;
 class BrowserContext;
+class NavigationController;

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

Is this declaration necessary? I don't see where this is referenced.

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

Done.

 class WebContents;
 }
 
 struct LanguageDetectionDetails;
 class PrefService;
 class TranslateAcceptLanguages;
 class TranslatePrefs;
 class TranslateManager;
 
 class TranslateTabHelper
@@ skipping 123 matching lines @@
 
   // Necessary for binding the callback to HandleCLDDataRequest on the blocking
   // pool and for delaying translation initialization until the page has
   // finished loading on a reload.
   base::WeakPtrFactory<TranslateTabHelper> weak_pointer_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(TranslateTabHelper);
 };
 
 #endif  // CHROME_BROWSER_TRANSLATE_TRANSLATE_TAB_HELPER_H_