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

Unified Diff: components/translate/content/browser/static_browser_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: 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/static_browser_cld_data_provider.cc
diff --git a/components/translate/content/browser/static_browser_cld_data_provider.cc b/components/translate/content/browser/static_browser_cld_data_provider.cc
deleted file mode 100644
index 675db42d08e3c26af8792622c17ea6129df2be67..0000000000000000000000000000000000000000
--- a/components/translate/content/browser/static_browser_cld_data_provider.cc
+++ /dev/null
@@ -1,56 +0,0 @@
-// 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 "static_browser_cld_data_provider.h"
-
-#include "base/logging.h"
-#include "content/public/browser/web_contents.h"
-#include "ipc/ipc_message.h"
-
-namespace translate {
-
-// Implementation of the static factory method from BrowserCldDataProvider,
-// hooking up this specific implementation for all of Chromium.
-BrowserCldDataProvider* CreateBrowserCldDataProviderFor(
- content::WebContents* web_contents) {
- // This log line is to help with determining which kind of provider has been
- // configured. See also: chrome://translate-internals
- VLOG(1) << "Creating StaticBrowserCldDataProvider";
- return new StaticBrowserCldDataProvider();
-}
-
-void SetCldDataFilePath(const base::FilePath& path) {
- LOG(WARNING) << "Not supported: SetCldDataFilePath";
- return;
-}
-
-base::FilePath GetCldDataFilePath() {
- return base::FilePath(); // empty path
-}
-
-void ConfigureBrowserCldDataProvider(const void* config) {
- // No-op: data is statically linked
-}
-
-StaticBrowserCldDataProvider::StaticBrowserCldDataProvider() {
-}
-
-StaticBrowserCldDataProvider::~StaticBrowserCldDataProvider() {
-}
-
-bool StaticBrowserCldDataProvider::OnMessageReceived(
- const IPC::Message& message) {
- // No-op: data is statically linked
- return false;
-}
-
-void StaticBrowserCldDataProvider::OnCldDataRequest() {
- // No-op: data is statically linked
-}
-
-void StaticBrowserCldDataProvider::SendCldDataResponse() {
- // No-op: data is statically linked
-}
-
-} // namespace translate
« no previous file with comments | « components/translate/content/browser/static_browser_cld_data_provider.h ('k') | components/translate/content/common/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698