From f1b4c26e8e68d5eeb9171e12ed068d4a8cd0ab21 Mon Sep 17 00:00:00 2001 From: Neil Lalonde Date: Tue, 4 Feb 2014 16:04:44 -0500 Subject: [PATCH] Add Mediapartners and AdsBot to list of user agents for Googlebot in browser-update.js.erb --- vendor/assets/javascripts/browser-update.js.erb | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/vendor/assets/javascripts/browser-update.js.erb b/vendor/assets/javascripts/browser-update.js.erb index 76dd581cd76..6a27df4c949 100644 --- a/vendor/assets/javascripts/browser-update.js.erb +++ b/vendor/assets/javascripts/browser-update.js.erb @@ -19,12 +19,12 @@ var $buo = function() { // sam: my main concern here is mobile, but its an outlier, for now we support ie9, set conditionally and stuff with pushState if (window.ie === "new" || (window.history && window.history.pushState && !badAndroid)) { - return; + return; } // we don't ask Googlebot to update their browser - if (ua.indexOf('Googlebot') >= 0) { - return; + if (ua.indexOf('Googlebot') >= 0 || ua.indexOf('Mediapartners') >= 0 || ua.indexOf('AdsBot') >= 0) { + return; } // retrieve localized browser upgrade text