From b8a6658ba9acb71613af3fc64158827690f4bddf Mon Sep 17 00:00:00 2001 From: Cheng Zhao Date: Fri, 12 Sep 2014 23:08:13 +0800 Subject: [PATCH] Make our user agent string follow standard. --- atom/browser/native_window.cc | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/atom/browser/native_window.cc b/atom/browser/native_window.cc index be41bf25e17..ba2444fc088 100644 --- a/atom/browser/native_window.cc +++ b/atom/browser/native_window.cc @@ -67,6 +67,14 @@ const char* kWebRuntimeFeatures[] = { switches::kSharedWorker, }; +std::string RemoveWhitespace(const std::string& str) { + std::string trimmed; + if (base::RemoveChars(str, " ", &trimmed)) + return trimmed; + else + return str; +} + } // namespace NativeWindow::NativeWindow(content::WebContents* web_contents, @@ -106,8 +114,8 @@ NativeWindow::NativeWindow(content::WebContents* web_contents, // Override the user agent to contain application and atom-shell's version. Browser* browser = Browser::Get(); std::string product_name = base::StringPrintf( - "%s/%s Chrome/%s Atom-Shell/" ATOM_VERSION_STRING, - browser->GetName().c_str(), + "%s/%s Chrome/%s AtomShell/" ATOM_VERSION_STRING, + RemoveWhitespace(browser->GetName()).c_str(), browser->GetVersion().c_str(), CHROME_VERSION_STRING); web_contents->GetMutableRendererPrefs()->user_agent_override =