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

Unified Diff: chrome/browser/safe_browsing/protocol_manager_helper.cc

Issue 2650973005: Componentize ping_manager (Closed)
Patch Set: rebase Created 3 years, 10 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: chrome/browser/safe_browsing/protocol_manager_helper.cc
diff --git a/chrome/browser/safe_browsing/protocol_manager_helper.cc b/chrome/browser/safe_browsing/protocol_manager_helper.cc
deleted file mode 100644
index 93c87a4f06e0d4da4526c0fcb924120562b3dbfc..0000000000000000000000000000000000000000
--- a/chrome/browser/safe_browsing/protocol_manager_helper.cc
+++ /dev/null
@@ -1,76 +0,0 @@
-// Copyright (c) 2012 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 "chrome/browser/safe_browsing/protocol_manager_helper.h"
-
-#ifndef NDEBUG
-#include "base/base64.h"
-#endif
-#include "base/environment.h"
-#include "base/logging.h"
-#include "base/stl_util.h"
-#include "base/strings/string_util.h"
-#include "base/strings/stringprintf.h"
-#include "components/version_info/version_info.h"
-#include "google_apis/google_api_keys.h"
-#include "net/base/escape.h"
-
-namespace safe_browsing {
-
-SafeBrowsingProtocolConfig::SafeBrowsingProtocolConfig()
- : disable_auto_update(false) {}
-
-SafeBrowsingProtocolConfig::SafeBrowsingProtocolConfig(
- const SafeBrowsingProtocolConfig& other) = default;
-
-SafeBrowsingProtocolConfig::~SafeBrowsingProtocolConfig() {}
-
-// static
-std::string SafeBrowsingProtocolManagerHelper::Version() {
- if (version_info::GetVersionNumber().empty())
- return "0.1";
- else
- return version_info::GetVersionNumber();
-}
-
-// static
-std::string SafeBrowsingProtocolManagerHelper::ComposeUrl(
- const std::string& prefix,
- const std::string& method,
- const std::string& client_name,
- const std::string& version,
- const std::string& additional_query) {
- DCHECK(!prefix.empty() && !method.empty() &&
- !client_name.empty() && !version.empty());
- std::string url = base::StringPrintf("%s/%s?client=%s&appver=%s&pver=3.0",
- prefix.c_str(), method.c_str(),
- client_name.c_str(), version.c_str());
- std::string api_key = google_apis::GetAPIKey();
- if (!api_key.empty()) {
- base::StringAppendF(&url, "&key=%s",
- net::EscapeQueryParamValue(api_key, true).c_str());
- }
- if (!additional_query.empty()) {
- DCHECK(url.find("?") != std::string::npos);
- url.append("&");
- url.append(additional_query);
- }
- return url;
-}
-
-// static
-std::string SafeBrowsingProtocolManagerHelper::ComposeUrl(
- const std::string& prefix,
- const std::string& method,
- const std::string& client_name,
- const std::string& version,
- const std::string& additional_query,
- ExtendedReportingLevel reporting_level) {
- std::string url =
- ComposeUrl(prefix, method, client_name, version, additional_query);
- url.append(base::StringPrintf("&ext=%d", reporting_level));
- return url;
-}
-
-} // namespace safe_browsing
« no previous file with comments | « chrome/browser/safe_browsing/protocol_manager_helper.h ('k') | chrome/browser/safe_browsing/safe_browsing_service.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698