Move LanguageState to the translate component

components/translate/core/browser/translate_client.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_CLIENT_H_
+#define COMPONENTS_TRANSLATE_CORE_BROWSER_TRANSLATE_CLIENT_H_
+
+#include "components/translate/core/browser/language_state.h"
+
+class TranslateClient {

[blundell, 2014/01/17 15:09:39]

This needs a class-level comment.

[droger, 2014/01/17 16:14:21]

Done.

+ public:
+  TranslateClient();
+  virtual ~TranslateClient();
+
+  // Gets the language state.
+  LanguageState& GetLanguageState();
+
+ protected:
+  LanguageState language_state_;

[blundell, 2014/01/17 15:09:39]

This should just be in the subclasses I think.

[droger, 2014/01/17 16:14:21]

Done.

+
+ private:
+  // Methods called by LanguageState.

[blundell, 2014/01/17 15:09:39]

Why would these methods be private?

[droger, 2014/01/17 16:14:21]

As discussed offline, it was that nobody can call them, except LanguageState
which is a friend.
Everything is now public, as it's not a goal for components to have a private
and a public API.

+  friend class LanguageState;
+  virtual bool IsLinkNavigation() = 0;
+  virtual void OnTranslateEnabledChanged() = 0;
+  virtual void OnIsPageTranslatedChanged() = 0;
+
+  DISALLOW_COPY_AND_ASSIGN(TranslateClient);
+};
+
+#endif  // COMPONENTS_TRANSLATE_CORE_BROWSER_TRANSLATE_CLIENT_H_