Fix nwjs#5975: Add %chromium_ver to user-agent field in manifest

rogerwang · rogerwang · commit 047636c4bd59 · 2017-12-20T09:29:49.000+08:00
diff --git a/docs/References/Manifest Format.md b/docs/References/Manifest Format.md
@@ -86,7 +86,8 @@ The following placeholders are available to composite the user agent dynamically
 * `%name`: replaced by the `name` field in the manifest.
 * `%ver`: replaced by the `version` field in the manifest, if available.
 * `%nwver`: replaced by the version of NW.js.
-* `%webkit_ver`: replaced by the version of WebKit engine.
+* `%webkit_ver`: replaced by the version of Blink engine.
+* `%chromium_ver`: replaced by the Chromium version
 * `%osinfo`: replaced by the OS and CPU information you would see in browser's user agent string.
 
 ### node-remote
diff --git a/src/common/nw_content_common_hooks.cc b/src/common/nw_content_common_hooks.cc
@@ -13,6 +13,7 @@
 #include "content/nw/src/nw_package.h"
 #include "content/nw/src/nw_version.h"
 
+#include "chrome/common/chrome_version.h"
 namespace nw {
 
 namespace {
@@ -48,6 +49,7 @@ void SetUserAgentOverride(const std::string& agent,
   base::ReplaceSubstringsAfterOffset(&g_user_agent, 0, "%nwver", NW_VERSION_STRING);
   base::ReplaceSubstringsAfterOffset(&g_user_agent, 0, "%webkit_ver", content::GetWebKitVersion());
   base::ReplaceSubstringsAfterOffset(&g_user_agent, 0, "%osinfo", content::BuildOSInfo());
+  base::ReplaceSubstringsAfterOffset(&g_user_agent, 0, "%chromium_ver", CHROME_VERSION_STRING);
 }
 
-} // nw
+} // nw
