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

Side by Side Diff: components/translate/core/browser/translate_driver.h

Issue 290573013: LanguageState should be owned by TranslateManager (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Inlining the DidNavigate function Created 6 years, 7 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef COMPONENTS_TRANSLATE_CORE_BROWSER_TRANSLATE_DRIVER_H_ 5 #ifndef COMPONENTS_TRANSLATE_CORE_BROWSER_TRANSLATE_DRIVER_H_
6 #define COMPONENTS_TRANSLATE_CORE_BROWSER_TRANSLATE_DRIVER_H_ 6 #define COMPONENTS_TRANSLATE_CORE_BROWSER_TRANSLATE_DRIVER_H_
7 7
8 #include <string> 8 #include <string>
9 9
10 class GURL; 10 class GURL;
11 class LanguageState; 11 class LanguageState;
droger 2014/05/20 07:43:07 You can remove this.
nshaik 2014/05/20 22:33:53 Done.
12 12
13 // Interface that allows Translate core code to interact with its driver (i.e., 13 // Interface that allows Translate core code to interact with its driver (i.e.,
14 // obtain information from it and give information to it). A concrete 14 // obtain information from it and give information to it). A concrete
15 // implementation must be provided by the driver. 15 // implementation must be provided by the driver.
16 class TranslateDriver { 16 class TranslateDriver {
17 public: 17 public:
18 // Returns true if the current page was navigated through a link. 18 // Returns true if the current page was navigated through a link.
19 virtual bool IsLinkNavigation() = 0; 19 virtual bool IsLinkNavigation() = 0;
20 20
21 // Called when Translate is enabled or disabled. 21 // Called when Translate is enabled or disabled.
22 virtual void OnTranslateEnabledChanged() = 0; 22 virtual void OnTranslateEnabledChanged() = 0;
23 23
24 // Called when the page is "translated" state of the page changed. 24 // Called when the page is "translated" state of the page changed.
25 virtual void OnIsPageTranslatedChanged() = 0; 25 virtual void OnIsPageTranslatedChanged() = 0;
26 26
27 // Gets the LanguageState associated with the driver.
28 virtual LanguageState& GetLanguageState() = 0;
29
30 // Translates the page contents from |source_lang| to |target_lang|. 27 // Translates the page contents from |source_lang| to |target_lang|.
31 virtual void TranslatePage(const std::string& translate_script, 28 virtual void TranslatePage(const std::string& translate_script,
32 const std::string& source_lang, 29 const std::string& source_lang,
33 const std::string& target_lang) = 0; 30 const std::string& target_lang) = 0;
34 31
35 // Reverts the contents of the page to its original language. 32 // Reverts the contents of the page to its original language.
36 virtual void RevertTranslation() = 0; 33 virtual void RevertTranslation() = 0;
37 34
38 // Returns whether the user is currently operating in off-the-record mode. 35 // Returns whether the user is currently operating in off-the-record mode.
39 virtual bool IsOffTheRecord() = 0; 36 virtual bool IsOffTheRecord() = 0;
(...skipping 13 matching lines...) Expand all
53 50
54 // Returns whether the driver has access to the current page. 51 // Returns whether the driver has access to the current page.
55 virtual bool HasCurrentPage() = 0; 52 virtual bool HasCurrentPage() = 0;
56 53
57 // Returns an int identifying the current page. Should only be called if 54 // Returns an int identifying the current page. Should only be called if
58 // |HasCurrentPage()| is true. 55 // |HasCurrentPage()| is true.
59 virtual int GetCurrentPageID() = 0; 56 virtual int GetCurrentPageID() = 0;
60 }; 57 };
61 58
62 #endif // COMPONENTS_TRANSLATE_CORE_BROWSER_TRANSLATE_DRIVER_H_ 59 #endif // COMPONENTS_TRANSLATE_CORE_BROWSER_TRANSLATE_DRIVER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698