Modularize Compact Language Detector 2 (CLD2) data sources

components/translate/content/browser/data_file_browser_cld_data_provider.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 COMPONENTS_TRANSLATE_CONTENT_BROWSER_DATA_FILE_BROWSER_CLD_DATA_PROVIDER_H_
+#define COMPONENTS_TRANSLATE_CONTENT_BROWSER_DATA_FILE_BROWSER_CLD_DATA_PROVIDER_H_
+
+#include "base/files/file.h"
+#include "base/files/file_path.h"
+#include "base/macros.h"
+#include "base/memory/weak_ptr.h"
+#include "components/translate/content/browser/browser_cld_data_provider.h"
+
+namespace content {

[droger, 2014/06/19 16:13:19]

namespace translate

[Andrew Hayden (chromium.org), 2014/06/19 19:49:30]

Done.

+
+class DataFileBrowserCldDataProvider : public BrowserCldDataProvider {
+ public:
+  explicit DataFileBrowserCldDataProvider(content::RenderViewHost*);
+  ~DataFileBrowserCldDataProvider();
+  // BrowserCldDataProvider implementations:
+  bool OnMessageReceived(const IPC::Message&);

[droger, 2014/06/19 16:13:19]

virtual and OVERRIDE?

[Andrew Hayden (chromium.org), 2014/06/19 19:49:30]

I don't currently plan to have any more subclasses, so I've left out virtual
here. OVERRIDE, though, is added.

+  void OnCldDataRequest();
+  void SendCldDataResponse();
+
+ private:
+  void SendCldDataResponseInternal(const base::File*,
+                                   const uint64,
+                                   const uint64);
+  void OnCldDataRequestInternal();
+
+  content::RenderViewHost* render_view_host;
+  base::WeakPtrFactory<DataFileBrowserCldDataProvider> weak_pointer_factory_;
+
+  DISALLOW_COPY_AND_ASSIGN(DataFileBrowserCldDataProvider);
+};
+
+// Sets the data file that this data provider will use to fulfill requests.
+// This method does nothing if the specified path is equal to the path that
+// is already configured. Otherwise, the specified path is cached and
+// subsequent requests for data will attempt to load CLD data from the file
+// at the specified path.
+// This method is threadsafe.
+void SetCldDataFilePath(const base::FilePath& filePath);
+
+// Returns the path most recently set by SetDataFilePath. The initial value
+// prior to any such call is the empty path.
+// This method is threadsafe.
+base::FilePath GetCldDataFilePath();
+
+}  // namespace content
+
+#endif  // COMPONENTS_TRANSLATE_CONTENT_BROWSER_DATA_FILE_BROWSER_CLD_DATA_PROVIDER_H_