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

Unified Diff: webkit/glue/user_agent.cc

Issue 7922023: Remove webkit_glue::BuildUserAgent(), remove windows spoofing (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: merge to r102225 Created 9 years, 3 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: webkit/glue/user_agent.cc
diff --git a/webkit/glue/user_agent.cc b/webkit/glue/user_agent.cc
index 7f9b43882e9f64f54a90e5be21f3776d7a9f66d7..c9b3828429bfb4576ffc1a2a94ecaebbaf8b7a67 100644
--- a/webkit/glue/user_agent.cc
+++ b/webkit/glue/user_agent.cc
@@ -113,8 +113,7 @@ int GetWebKitMinorVersion() {
return WEBKIT_VERSION_MINOR;
}
-std::string BuildUserAgentHelper(bool mimic_windows,
- const std::string& product) {
+std::string BuildUserAgentFromProduct(const std::string& product) {
const char kUserAgentPlatform[] =
#if defined(OS_WIN)
"";
@@ -128,7 +127,6 @@ std::string BuildUserAgentHelper(bool mimic_windows,
std::string user_agent;
- // Replace Safari's Version/X string with the product name/version passed in.
// This is done to expose our product name in a manner that is maximally
// compatible with Safari, we hope!!
@@ -137,7 +135,7 @@ std::string BuildUserAgentHelper(bool mimic_windows,
&user_agent,
"Mozilla/5.0 (%s%s) AppleWebKit/%d.%d"
" (KHTML, like Gecko) %s Safari/%d.%d",
- mimic_windows ? "Windows " : kUserAgentPlatform,
+ kUserAgentPlatform,
webkit_glue::BuildOSCpuInfo().c_str(),
WEBKIT_VERSION_MAJOR,
WEBKIT_VERSION_MINOR,

Powered by Google App Engine
This is Rietveld 408576698