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

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

Issue 461633002: Refactor language detection logic to allow non-static CLD data sources. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Make some of the harness factory methods private Created 6 years, 1 month 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/browser_cld_utils.h
diff --git a/components/translate/content/browser/browser_cld_utils.h b/components/translate/content/browser/browser_cld_utils.h
new file mode 100644
index 0000000000000000000000000000000000000000..f3fab95c332ea0a4c0d42859a6acdbf6f9cb98cb
--- /dev/null
+++ b/components/translate/content/browser/browser_cld_utils.h
@@ -0,0 +1,38 @@
+// 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_BROWSER_CLD_UTILS_H_
+#define COMPONENTS_TRANSLATE_CONTENT_BROWSER_BROWSER_CLD_UTILS_H_
+
+#include "base/macros.h"
+
+namespace translate {
+
+// Browser-side utilities for dealing with CLD data source configuration.
+// This class exists primarily to avoid duplicating code in high-level targets
+// such as the Chrome browser, Chrome shell, and so on.
+class BrowserCldUtils {
+ public:
+ // Perform conditional configuration of the CLD data provider.
+ // If no specific BrowserCldDataProviderFactory has yet been set, an instance
+ // is created and assigned as appropriate for each platform. If a specific
+ // BrowserCldDataProviderFactory *has* been set, this method does nothing
+ // (as a side effect of setting the default factory, subsequent invocations
+ // do nothing - making this method reentrant).
+ //
+ // This method will not overwrite a data provider configured by an embedder.
+ // This method must be invoked prior to registering components with the
+ // component updater if the component updater implementation is to be used;
+ // it is best to ensure that this method is called as early as possible.
+ static void ConfigureDefaultDataProvider();
+
+ private:
+ BrowserCldUtils() {}
+ ~BrowserCldUtils() {}
+ DISALLOW_COPY_AND_ASSIGN(BrowserCldUtils);
+};
+
+} // namespace translate
+
+#endif // COMPONENTS_TRANSLATE_CONTENT_BROWSER_BROWSER_CLD_UTILS_H_

Powered by Google App Engine
This is Rietveld 408576698