Modularize Compact Language Detector 2 (CLD2) data sources

chrome/browser/translate/chrome_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 CHROME_BROWSER_TRANSLATE_CHROME_TRANSLATE_CLIENT_H_
 #define CHROME_BROWSER_TRANSLATE_CHROME_TRANSLATE_CLIENT_H_
 
 #include <string>
 
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/weak_ptr.h"
 #include "chrome/browser/ui/translate/translate_bubble_model.h"
+#include "components/translate/content/browser/browser_cld_data_provider.h"
 #include "components/translate/content/browser/content_translate_driver.h"
 #include "components/translate/core/browser/translate_client.h"
 #include "components/translate/core/browser/translate_step.h"
 #include "components/translate/core/common/translate_errors.h"
 #include "content/public/browser/web_contents_observer.h"
 #include "content/public/browser/web_contents_user_data.h"
 
-#if defined(CLD2_DYNAMIC_MODE)
-#include "base/basictypes.h"
-#include "base/lazy_instance.h"
-#include "base/synchronization/lock.h"
-#include "base/task_runner.h"
-#endif
-
 namespace base {
 class File;
 }  // namespace base
 
 namespace content {
 class BrowserContext;
 class WebContents;
 }  // namespace content
 
 namespace test {
@@ skipping 84 matching lines @@
 
   // Initiates translation once the page is finished loading.
   void InitiateTranslation(const std::string& page_lang, int attempt);
   void OnLanguageDetermined(const LanguageDetectionDetails& details,
                             bool page_needs_translation);
   void OnPageTranslated(int32 page_id,
                         const std::string& original_lang,
                         const std::string& translated_lang,
                         TranslateErrors::Type error_type);
 
-#if defined(CLD2_DYNAMIC_MODE)
-  // Called when we receive ChromeViewHostMsg_NeedCLDData from a renderer.
-  // If we have already cached the data, responds immediately; else, enqueues
-  // a HandleCLDDataRequest on the blocking pool to cache the data.
-  // Acquires and releases s_file_lock_ in a non-blocking manner; queries
-  // handled while the file is being cached will gracefully and immediately
-  // fail.
-  // It is up to the originator of the message to poll again later if required;
-  // no "negative response" will be generated.
-  void OnCLDDataRequested();
-
-  // Invoked on the blocking pool in order to cache the data. When successful,
-  // immediately responds to the request that initiated OnCLDDataRequested.
-  // Holds s_file_lock_ while the file is being cached.
-  static void HandleCLDDataRequest();
-
-  // If the CLD data is ready, send it to the renderer. Briefly checks the lock.
-  void MaybeSendCLDDataAvailable();
-
-  // Sends the renderer a response containing the data file handle. No locking.
-  void SendCLDDataAvailable(const base::File* handle,
-                            const uint64 data_offset,
-                            const uint64 data_length);
-
-  // The data file,  cached as long as the process stays alive.
-  // We also track the offset at which the data starts, and its length.
-  static base::File* s_cached_file_;    // guarded by file_lock_
-  static uint64 s_cached_data_offset_;  // guarded by file_lock_
-  static uint64 s_cached_data_length_;  // guarded by file_lock_
-
-  // Guards s_cached_file_
-  static base::LazyInstance<base::Lock> s_file_lock_;
-
-#endif
-
   // Shows the translate bubble.
   void ShowBubble(translate::TranslateStep step,
                   TranslateErrors::Type error_type);
 
   // Max number of attempts before checking if a page has been reloaded.
   int max_reload_check_attempts_;
 
   ContentTranslateDriver translate_driver_;
   scoped_ptr<TranslateManager> translate_manager_;
 
-  // Necessary for binding the callback to HandleCLDDataRequest on the blocking

[Takashi Toyoshima, 2014/06/19 10:20:34]

Why do you remove these comments for WeakPtrFactory?

[Andrew Hayden (chromium.org), 2014/06/19 13:40:19]

Because this factory was here before HandleCLDDataRequest got added, and because
HandleCLDDataRequest no longer exists. I haven't looked into what else is using
it, but it is used in other parts of the code.

-  // pool and for delaying translation initialization until the page has
-  // finished loading on a reload.
+  // Provides CLD data for this process.
+  scoped_ptr<content::BrowserCldDataProvider> cld_data_provider_;
+
   base::WeakPtrFactory<ChromeTranslateClient> weak_pointer_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(ChromeTranslateClient);
 };
 
 #endif  // CHROME_BROWSER_TRANSLATE_CHROME_TRANSLATE_CLIENT_H_