Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(289)

Unified Diff: components/translate/core/browser/translate_ui_delegate.h

Issue 229363002: Componentize TranslateUIDelegate (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/translate/core/browser/translate_ui_delegate.h
diff --git a/chrome/browser/translate/translate_ui_delegate.h b/components/translate/core/browser/translate_ui_delegate.h
similarity index 85%
rename from chrome/browser/translate/translate_ui_delegate.h
rename to components/translate/core/browser/translate_ui_delegate.h
index 37f4c67f2c1addf0f7f53e1136d6ccefb967be8e..6cee1108ec67d098c37f108207cf44e53e32b50d 100644
--- a/chrome/browser/translate/translate_ui_delegate.h
+++ b/components/translate/core/browser/translate_ui_delegate.h
@@ -2,8 +2,8 @@
// 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_UI_DELEGATE_H_
-#define CHROME_BROWSER_TRANSLATE_TRANSLATE_UI_DELEGATE_H_
+#ifndef COMPONENTS_TRANSLATE_CORE_BROWSER_TRANSLATE_UI_DELEGATE_H_
+#define COMPONENTS_TRANSLATE_CORE_BROWSER_TRANSLATE_UI_DELEGATE_H_
#include <string>
#include <vector>
@@ -11,15 +11,14 @@
#include "base/logging.h"
#include "base/memory/scoped_ptr.h"
#include "base/strings/string16.h"
-#include "chrome/common/chrome_constants.h"
#include "components/translate/core/common/translate_errors.h"
+class LanguageState;
+class TranslateClient;
+class TranslateDriver;
+class TranslateManager;
class TranslatePrefs;
-namespace content {
-class WebContents;
-} // namespace content
-
// The TranslateUIDelegate is a generic delegate for UI which offers Translate
// feature to the user.
class TranslateUIDelegate {
@@ -28,16 +27,18 @@ class TranslateUIDelegate {
NO_INDEX = -1,
};
- TranslateUIDelegate(content::WebContents* web_contents,
+ TranslateUIDelegate(TranslateClient* translate_client,
+ TranslateManager* translate_manager,
const std::string& original_language,
const std::string& target_language);
virtual ~TranslateUIDelegate();
- content::WebContents* web_contents() { return web_contents_; }
-
// Handles when an error message is shown.
void OnErrorShown(TranslateErrors::Type error_type);
+ // Returns the LanguageState associated with this object.
+ const LanguageState& GetLanguageState();
+
// Returns the number of languages supported.
size_t GetNumberOfLanguages() const;
@@ -99,7 +100,9 @@ class TranslateUIDelegate {
// associated with the current page.
std::string GetPageHost();
- content::WebContents* web_contents_;
+ TranslateClient* translate_client_;
+ TranslateDriver* translate_driver_;
+ TranslateManager* translate_manager_;
typedef std::pair<std::string, base::string16> LanguageNamePair;
@@ -128,4 +131,4 @@ class TranslateUIDelegate {
DISALLOW_COPY_AND_ASSIGN(TranslateUIDelegate);
};
-#endif // CHROME_BROWSER_TRANSLATE_TRANSLATE_UI_DELEGATE_H_
+#endif // COMPONENTS_TRANSLATE_CORE_BROWSER_TRANSLATE_UI_DELEGATE_H_

Powered by Google App Engine
This is Rietveld 408576698