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

Unified Diff: components/translate/content/browser/data_file_browser_cld_data_provider.h

Issue 333603002: Modularize Compact Language Detector 2 (CLD2) data sources (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Takashi's comments and virtual destructors Created 6 years, 6 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 side-by-side diff with in-line comments
Download patch
Index: components/translate/content/browser/data_file_browser_cld_data_provider.h
diff --git a/components/translate/content/browser/data_file_browser_cld_data_provider.h b/components/translate/content/browser/data_file_browser_cld_data_provider.h
new file mode 100644
index 0000000000000000000000000000000000000000..7dbe044b4cefd666284cea2b1d3150566c5e6eb1
--- /dev/null
+++ b/components/translate/content/browser/data_file_browser_cld_data_provider.h
@@ -0,0 +1,52 @@
+// 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
+ 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_

Powered by Google App Engine
This is Rietveld 408576698