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

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

Issue 1399843003: Move prefix_set and parts of s_b_util into a new component safe_browsing_db. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 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: chrome/browser/safe_browsing/safe_browsing_util.cc
diff --git a/chrome/browser/safe_browsing/safe_browsing_util.cc b/chrome/browser/safe_browsing/safe_browsing_util.cc
index 728bd96c2487d1cb43859b9cf6dd4603a82141b5..177f2d8e335d8de48e46c98bfd6a27a5a7e7c8e0 100644
--- a/chrome/browser/safe_browsing/safe_browsing_util.cc
+++ b/chrome/browser/safe_browsing/safe_browsing_util.cc
@@ -7,7 +7,6 @@
#include "base/logging.h"
#include "base/strings/string_util.h"
#include "base/strings/stringprintf.h"
-#include "chrome/browser/browser_process.h"
#include "chrome/browser/safe_browsing/chunk.pb.h"
#include "components/google/core/browser/google_util.h"
#include "crypto/sha2.h"
@@ -15,18 +14,6 @@
#include "url/gurl.h"
#include "url/url_util.h"
-#if defined(OS_WIN)
-#include "chrome/installer/util/browser_distribution.h"
-#endif
-
-static const char kReportParams[] = "?tpl=%s&url=%s";
-
-SBFullHash SBFullHashForString(const base::StringPiece& str) {
- SBFullHash h;
- crypto::SHA256HashString(str, &h.full_hash, sizeof(h.full_hash));
- return h;
-}
-
// SBCachedFullHashResult ------------------------------------------------------
SBCachedFullHashResult::SBCachedFullHashResult() {}
@@ -491,28 +478,6 @@ void GeneratePatternsToCheck(const GURL& url, std::vector<std::string>* urls) {
}
}
-GURL GeneratePhishingReportUrl(const std::string& report_page,
Nathan Parker 2015/10/14 17:37:10 meta note: This is being removed because it is dea
vakh (old account. dont use) 2015/10/15 21:07:08 Acknowledged.
- const std::string& url_to_report,
- bool is_client_side_detection) {
- const std::string current_esc = net::EscapeQueryParamValue(url_to_report,
- true);
-
-#if defined(OS_WIN)
- BrowserDistribution* dist = BrowserDistribution::GetDistribution();
- std::string client_name(dist->GetSafeBrowsingName());
-#else
- std::string client_name("googlechrome");
-#endif
- if (is_client_side_detection)
- client_name.append("_csd");
-
- GURL report_url(report_page + base::StringPrintf(kReportParams,
- client_name.c_str(),
- current_esc.c_str()));
- return google_util::AppendGoogleLocaleParam(
- report_url, g_browser_process->GetApplicationLocale());
-}
-
SBFullHash StringToSBFullHash(const std::string& hash_in) {
DCHECK_EQ(crypto::kSHA256Length, hash_in.size());
SBFullHash hash_out;

Powered by Google App Engine
This is Rietveld 408576698