Move LanguageState to the translate component

chrome/browser/translate/translate_tab_helper.h

-// Copyright (c) 2011 The Chromium Authors. All rights reserved.
+// 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 "chrome/browser/tab_contents/language_state.h"
+#include "components/translate/content/browser/content_translate_driver.h"
 #include "components/translate/core/common/translate_errors.h"
 #include "content/public/browser/web_contents_observer.h"
 #include "content/public/browser/web_contents_user_data.h"
 
 struct LanguageDetectionDetails;
 
 class TranslateTabHelper
     : public content::WebContentsObserver,
       public content::WebContentsUserData<TranslateTabHelper> {
  public:
   virtual ~TranslateTabHelper();
 
-  LanguageState& language_state() { return language_state_; }
+  // Gets the LanguageState associated with the page.
+  // Information about the language the page is in and has been translated to.

[blundell, 2014/01/17 16:37:36]

This comment reads oddly.

[droger, 2014/01/20 10:37:06]

This line was in the original code (line 40 on the left).
Removed.

+  LanguageState& GetLanguageState();
+
+  // Returns the content driver for translate.
+  ContentTranslateDriver* GetTranslateDriver();

[blundell, 2014/01/17 16:37:36]

I assume that TranslateTabHelper will get merged with TranslateDelegateImpl in
the medium-term?

[droger, 2014/01/20 10:37:06]

If all goes well yes.
I still need to sort out the ownership issues in TranslateManager before being
100% sure, and that's what I'll do next.

 
  private:
   explicit TranslateTabHelper(content::WebContents* web_contents);
   friend class content::WebContentsUserData<TranslateTabHelper>;
 
   // content::WebContentsObserver implementation.
   virtual bool OnMessageReceived(const IPC::Message& message) OVERRIDE;
   virtual void DidNavigateAnyFrame(
       const content::LoadCommittedDetails& details,
       const content::FrameNavigateParams& params) OVERRIDE;
 
   void OnLanguageDetermined(const LanguageDetectionDetails& details,
                             bool page_needs_translation);
   void OnPageTranslated(int32 page_id,
                         const std::string& original_lang,
                         const std::string& translated_lang,
                         TranslateErrors::Type error_type);
 
-  // Information about the language the page is in and has been translated to.
-  LanguageState language_state_;
+  ContentTranslateDriver translate_driver_;
 
   DISALLOW_COPY_AND_ASSIGN(TranslateTabHelper);
 };
 
 #endif  // CHROME_BROWSER_TRANSLATE_TRANSLATE_TAB_HELPER_H_