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

Unified Diff: components/translate/content/renderer/renderer_cld_data_provider.cc

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: Fix merge error in chrome/browser/BUILD.gn Created 6 years, 2 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/renderer/renderer_cld_data_provider.cc
diff --git a/components/translate/content/renderer/renderer_cld_data_provider.cc b/components/translate/content/renderer/renderer_cld_data_provider.cc
new file mode 100644
index 0000000000000000000000000000000000000000..43b2b072b6faad2ad45d393be99496b856373d7a
--- /dev/null
+++ b/components/translate/content/renderer/renderer_cld_data_provider.cc
@@ -0,0 +1,67 @@
+// 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.
+
+#include "base/lazy_instance.h"
+#include "components/translate/content/renderer/renderer_cld_data_provider.h"
+
+namespace {
+translate::RendererCldDataProvider* g_instance = NULL;
Takashi Toyoshima 2014/10/15 08:45:07 empty line between line 8 and 9.
Andrew Hayden (chromium.org) 2014/10/30 16:56:32 Done.
+
+// The default is a no-op implementation used to avoid a hard-crash if code
+// fails to set a more functional version.
+struct WrappedDefault {
+ WrappedDefault() {
+ value = new translate::RendererCldDataProvider();
+ }
+ translate::RendererCldDataProvider* value;
+};
+base::LazyInstance<WrappedDefault>::Leaky g_wrapped_default =
+ LAZY_INSTANCE_INITIALIZER;
+
+} // namespace
+
+
+namespace translate {
+
+RendererCldDataProvider::RendererCldDataProvider() {
+}
+
+bool RendererCldDataProvider::OnMessageReceived(const IPC::Message&) {
+ return false; // Message not handled
+}
+
+void RendererCldDataProvider::SendCldDataRequest() {
+}
+
+bool RendererCldDataProvider::IsCldDataAvailable() {
+ return true; // Vacuously true
+}
+
+void RendererCldDataProvider::SetCldAvailableCallback(
+ base::Callback<void(void)> callback) {
+ callback.Run();
+}
+
+void RendererCldDataProvider::Set(
+ RendererCldDataProvider* instance, bool overwrite) {
+ if (overwrite || g_instance == NULL) {
+ g_instance = instance;
+ }
+}
+
+bool RendererCldDataProvider::IsInitialized() {
+ return g_instance != NULL;
+}
+
+RendererCldDataProvider* RendererCldDataProvider::Get() {
+ if (g_instance == NULL) {
+ // No provider set. OK for test code and narrow use cases, but bad if it is
+ // a real renderer. Since there's no good way to differentiate these use
+ // cases at runtime, don't log a warning here.
+ return g_wrapped_default.Get().value;
+ }
+ return g_instance;
+}
+
+} // namespace translate

Powered by Google App Engine
This is Rietveld 408576698