Refactor language detection logic to allow non-static CLD data sources.

components/translate/content/browser/browser_cld_data_provider.h

Index: components/translate/content/browser/browser_cld_data_provider.h
diff --git a/components/translate/content/browser/browser_cld_data_provider.h b/components/translate/content/browser/browser_cld_data_provider.h
index 4e8776ae07b110d93dc02030d597d7df468a1781..00d286ea886345d885e02fc5898125c28d2249de 100644
--- a/components/translate/content/browser/browser_cld_data_provider.h
+++ b/components/translate/content/browser/browser_cld_data_provider.h
@@ -7,17 +7,13 @@
 
 #include <string>
 
-#include "base/files/file_path.h"
+#include "base/compiler_specific.h"
 #include "ipc/ipc_listener.h"
 
 namespace IPC {
 class Message;
 }
 
-namespace content {
-class WebContents;
-}
-
 namespace translate {
 
 // Browser-side interface responsible for providing CLD data.
@@ -31,6 +27,7 @@ namespace translate {
 // constant from ipc_message_start.h
 class BrowserCldDataProvider : public IPC::Listener {
  public:
+  BrowserCldDataProvider();
   virtual ~BrowserCldDataProvider() {}
 
   // IPC::Listener implementation:
@@ -38,7 +35,7 @@ class BrowserCldDataProvider : public IPC::Listener {
   // OnCldDataRequest() and returns true. In all other cases, this method does
   // nothing. This method is defined as virtual in order to force the
   // implementation to define the specific IPC message(s) that it handles.
-  virtual bool OnMessageReceived(const IPC::Message&) = 0;
+  virtual bool OnMessageReceived(const IPC::Message&) OVERRIDE;
 
   // Called when the browser process receives an appropriate message in
   // OnMessageReceived, above. The implementation should attempt to locate
@@ -48,35 +45,15 @@ class BrowserCldDataProvider : public IPC::Listener {
   // The renderer process may poll for data, in which case this method may be
   // repeatedly invoked. The implementation must be safe to call any number
   // of times.
-  virtual void OnCldDataRequest() = 0;
+  virtual void OnCldDataRequest();

[Andrew Hayden (chromium.org), 2014/09/26 10:22:49]

Now that this isn't a pure virtual (same with SendCldDataResponse, below),
document the default behavior as being empty.

[Andrew Hayden (chromium.org), 2014/10/28 15:18:39]

Done.

 
   // Invoked when OnCldDataRequest, above, results in a successful lookup or
   // the data is already cached and ready to respond to. The implementation
   // should take whatever action is appropriate for responding to the paired
   // RendererCldDataProvider, typically by sending an IPC response.
-  virtual void SendCldDataResponse() = 0;
+  virtual void SendCldDataResponse();
 };
 
-// Static factory function defined by the implementation that produces a new
-// provider for the specified WebContents.
-BrowserCldDataProvider* CreateBrowserCldDataProviderFor(
-    content::WebContents*);
-
-// For data sources that support a separate CLD data file, configures the path
-// of that data file.
-//
-// The 'component' and 'standalone' data sources need this method to be called
-// in order to locate the CLD data on disk.
-// If the data source doesn't need or doesn't support such configuration, this
-// function should do nothing. This is the case for, e.g., the static data
-// source.
-void SetCldDataFilePath(const base::FilePath& path);
-
-// Returns the path most recently set by SetCldDataFilePath. The initial value
-// prior to any such call is the empty path. If the data source doesn't support
-// a data file, returns the empty path.
-base::FilePath GetCldDataFilePath();
-
 }  // namespace translate
 
 #endif  // COMPONENTS_TRANSLATE_CONTENT_BROWSER_BROWSER_CLD_DATA_PROVIDER_H_