MD Settings: make devices open in a new tab rather than clobber current one

chrome/browser/resources/settings/printing_page/printing_page.js

Index: chrome/browser/resources/settings/printing_page/printing_page.js
diff --git a/chrome/browser/resources/settings/printing_page/printing_page.js b/chrome/browser/resources/settings/printing_page/printing_page.js
index 72213f0c6ed5105788bae6d40f0c7bdfc9c85cfc..43df17710df47a5676f54a1713daaad6fc8d5c78 100644
--- a/chrome/browser/resources/settings/printing_page/printing_page.js
+++ b/chrome/browser/resources/settings/printing_page/printing_page.js
@@ -14,6 +14,6 @@ Polymer({
 
   /** @private */
   onManageTap_: function() {
-    location.href = loadTimeData.getString('devicesUrl');
+    window.open(loadTimeData.getString('devicesUrl'));

[groby-ooo-7-16, 2016/06/14 23:47:00]

nit: We'll move these strings to $i18n{} at some point, right?

[Dan Beam, 2016/06/14 23:59:14]

we only search/replace $i18n{} in html files right now

we could do data-url="$i18n{devicesUrl}" and window.open(this.data.url) but that
doesn't seem all too much better.

also, $i18n{} itself can break complex URLs with ?multi=part&query=params

   },
 });