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

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

Issue 187393005: Make it possible to read CLD data from a file (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address Marcus' and Jochen's comments Created 6 years, 9 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 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"
11 #include "base/memory/weak_ptr.h"
12 #include "base/platform_file.h"
13 #include "base/synchronization/lock.h"
14 #include "base/task_runner.h"
11 #include "chrome/browser/ui/translate/translate_bubble_model.h" 15 #include "chrome/browser/ui/translate/translate_bubble_model.h"
12 #include "components/translate/content/browser/content_translate_driver.h" 16 #include "components/translate/content/browser/content_translate_driver.h"
13 #include "components/translate/core/common/translate_errors.h" 17 #include "components/translate/core/common/translate_errors.h"
14 #include "content/public/browser/web_contents_observer.h" 18 #include "content/public/browser/web_contents_observer.h"
15 #include "content/public/browser/web_contents_user_data.h" 19 #include "content/public/browser/web_contents_user_data.h"
16 20
17 namespace content { 21 namespace content {
18 class BrowserContext; 22 class BrowserContext;
19 class WebContents; 23 class WebContents;
20 } 24 }
(...skipping 64 matching lines...) Expand 10 before | Expand all | Expand 10 after
85 virtual void WebContentsDestroyed( 89 virtual void WebContentsDestroyed(
86 content::WebContents* web_contents) OVERRIDE; 90 content::WebContents* web_contents) OVERRIDE;
87 91
88 void OnLanguageDetermined(const LanguageDetectionDetails& details, 92 void OnLanguageDetermined(const LanguageDetectionDetails& details,
89 bool page_needs_translation); 93 bool page_needs_translation);
90 void OnPageTranslated(int32 page_id, 94 void OnPageTranslated(int32 page_id,
91 const std::string& original_lang, 95 const std::string& original_lang,
92 const std::string& translated_lang, 96 const std::string& translated_lang,
93 TranslateErrors::Type error_type); 97 TranslateErrors::Type error_type);
94 98
99 // Called when we receive ChromeViewHostMsg_NeedCLDData from a renderer.
100 // If we have already cached the data, responds immediately; else, enqueues
101 // a HandleCLDDataRequest on the blocking pool to cache the data.
102 // Acquires and releases s_file_lock_ in a non-blocking manner; queries
103 // handled while the file is being cached will gracefully and immediately
104 // fail.
105 void OnCLDDataRequested();
106
107 // Invoked on the blocking pool in order to cache the data. When successful,
108 // immediately responds to the request that initiated OnCLDDataRequested.
109 // Holds s_file_lock_ while the file is being cached.
110 void HandleCLDDataRequest();
111
112 // Sends the renderer a response containing the data file handle. No locking.
113 void SendCLDDataAvailable(const base::PlatformFile handle);
114
115 // Necessary for binding the callback to HandleCLDDataRequest on the blocking
116 // pool.
117 base::WeakPtrFactory<TranslateTabHelper> weak_pointer_factory_;
118
95 // Shows the translate bubble. 119 // Shows the translate bubble.
96 void ShowBubble(TranslateStep step, TranslateErrors::Type error_type); 120 void ShowBubble(TranslateStep step, TranslateErrors::Type error_type);
97 121
98 ContentTranslateDriver translate_driver_; 122 ContentTranslateDriver translate_driver_;
99 scoped_ptr<TranslateManager> translate_manager_; 123 scoped_ptr<TranslateManager> translate_manager_;
100 124
125 // The data file, cached as long as the process stays alive.
126 static base::PlatformFile s_cached_platform_file_; // guarded by file_lock_
127
128 // Guards s_cached_platform_file_
129 static base::Lock s_file_lock_;
130
101 DISALLOW_COPY_AND_ASSIGN(TranslateTabHelper); 131 DISALLOW_COPY_AND_ASSIGN(TranslateTabHelper);
102 }; 132 };
103 133
104 #endif // CHROME_BROWSER_TRANSLATE_TRANSLATE_TAB_HELPER_H_ 134 #endif // CHROME_BROWSER_TRANSLATE_TRANSLATE_TAB_HELPER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698