Modularize Compact Language Detector 2 (CLD2) data sources

components/translate/content/renderer/data_file_renderer_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 CHROME_RENDERER_TRANSLATE_DATA_FILE_RENDERER_CLD_DATA_PROVIDER_H_

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

macro name

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

Done.

+#define CHROME_RENDERER_TRANSLATE_DATA_FILE_RENDERER_CLD_DATA_PROVIDER_H_
+
+#include "base/files/file.h"
+#include "base/files/memory_mapped_file.h"
+#include "base/lazy_instance.h"
+#include "base/memory/weak_ptr.h"
+#include "components/translate/content/renderer/renderer_cld_data_provider.h"
+#include "content/public/renderer/render_view_observer.h"
+#include "ipc/ipc_message.h"
+#include "ipc/ipc_platform_file.h"
+
+namespace content {
+
+class DataFileRendererCldDataProvider : public RendererCldDataProvider {
+ public:
+  DataFileRendererCldDataProvider(content::RenderViewObserver*);

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

explicit

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

Done.

+  bool OnMessageReceived(const IPC::Message&);
+  void SendCldDataRequest();
+  void SetCldAvailableCallback(base::Callback<void(void)>);
+  bool IsCldDataAvailable();
+private:
+  void OnCldDataAvailable(const IPC::PlatformFileForTransit ipc_file_handle,
+                          const uint64 data_offset,
+                          const uint64 data_length);
+  void LoadCldData(base::File file,
+                    const uint64 data_offset,
+                    const uint64 data_length);
+  content::RenderViewObserver* render_view_observer;
+  base::Callback<void(void)> cld_available_callback;
+  // A struct that contains the pointer to the CLD mmap. Used so that we can
+  // leverage LazyInstance:Leaky to properly scope the lifetime of the mmap.
+  struct CLDMmapWrapper {
+    CLDMmapWrapper() {
+      value = NULL;
+    }
+    base::MemoryMappedFile* value;
+  };
+  static base::LazyInstance<CLDMmapWrapper>::Leaky s_cld_mmap_;

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

DISALLOW_COPY_AND_ASSIGN

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

Done.

+};
+
+// Implementation of the static factory method from BrowserCldDataProvider,
+// hooking up this specific implementation for all of Chromium.
+RendererCldDataProvider* RendererCldDataProviderFor(

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

ditto, any reason to implement this in a header?

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

Same response, moved to .cc file.

+    content::RenderViewObserver* render_view_observer) {
+  return new DataFileRendererCldDataProvider(render_view_observer);
+}
+
+}  // namespace content
+
+#endif  // CHROME_RENDERER_TRANSLATE_DATA_FILE_RENDERER_CLD_DATA_PROVIDER_H_