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

Side by Side Diff: trunk/src/chrome/browser/translate/translate_tab_helper.h

Issue 296003014: Revert 272217 "LanguageState should be owned by TranslateManager" (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: 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 | Annotate | Revision Log
OLDNEW
1 // Copyright 2011 The Chromium Authors. All rights reserved. 1 // Copyright 2011 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 CHROME_BROWSER_TRANSLATE_TRANSLATE_TAB_HELPER_H_ 5 #ifndef CHROME_BROWSER_TRANSLATE_TRANSLATE_TAB_HELPER_H_
6 #define CHROME_BROWSER_TRANSLATE_TRANSLATE_TAB_HELPER_H_ 6 #define CHROME_BROWSER_TRANSLATE_TRANSLATE_TAB_HELPER_H_
7 7
8 #include <string> 8 #include <string>
9 9
10 #include "base/memory/scoped_ptr.h" 10 #include "base/memory/scoped_ptr.h"
(...skipping 20 matching lines...) Expand all
31 namespace content { 31 namespace content {
32 class BrowserContext; 32 class BrowserContext;
33 class WebContents; 33 class WebContents;
34 } // namespace content 34 } // namespace content
35 35
36 namespace test { 36 namespace test {
37 class ScopedCLDDynamicDataHarness; 37 class ScopedCLDDynamicDataHarness;
38 } // namespace test 38 } // namespace test
39 39
40 struct LanguageDetectionDetails; 40 struct LanguageDetectionDetails;
41 class LanguageState;
42 class PrefService; 41 class PrefService;
43 class TranslateAcceptLanguages; 42 class TranslateAcceptLanguages;
44 class TranslatePrefs; 43 class TranslatePrefs;
45 class TranslateManager; 44 class TranslateManager;
46 45
47 class TranslateTabHelper 46 class TranslateTabHelper
48 : public TranslateClient, 47 : public TranslateClient,
49 public content::WebContentsObserver, 48 public content::WebContentsObserver,
50 public content::WebContentsUserData<TranslateTabHelper> { 49 public content::WebContentsUserData<TranslateTabHelper> {
51 public: 50 public:
(...skipping 120 matching lines...) Expand 10 before | Expand all | Expand 10 after
172 171
173 // Necessary for binding the callback to HandleCLDDataRequest on the blocking 172 // Necessary for binding the callback to HandleCLDDataRequest on the blocking
174 // pool and for delaying translation initialization until the page has 173 // pool and for delaying translation initialization until the page has
175 // finished loading on a reload. 174 // finished loading on a reload.
176 base::WeakPtrFactory<TranslateTabHelper> weak_pointer_factory_; 175 base::WeakPtrFactory<TranslateTabHelper> weak_pointer_factory_;
177 176
178 DISALLOW_COPY_AND_ASSIGN(TranslateTabHelper); 177 DISALLOW_COPY_AND_ASSIGN(TranslateTabHelper);
179 }; 178 };
180 179
181 #endif // CHROME_BROWSER_TRANSLATE_TRANSLATE_TAB_HELPER_H_ 180 #endif // CHROME_BROWSER_TRANSLATE_TRANSLATE_TAB_HELPER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698