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

Side by Side Diff: go-back-with-backspace/pages/common.js

Issue 2400303003: Update UI and catch executeScript errors now shown in Canary (Closed)
Patch Set: Fix global-replace bug Created 4 years, 1 month 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 Google Inc. All rights reserved. 1 // Copyright 2016 Google Inc. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 // Load strings from messages.json into the HTML page. Each element that needs 5 // Load strings from messages.json into the HTML page. Each element that needs
6 // an internationalized string should have an 'i18n' property holding the 6 // an internationalized string should have an 'i18n' property holding the
7 // name of the message to be used. 7 // name of the message to be used. If the element instead has an 'i18n-alt'
8 function LoadInternationalizedStrings() { 8 // property, that message will be used for the alt and title attributes of the
9 // element (generally an image).
10 function loadInternationalizedStrings() {
9 var all = document.querySelectorAll('[i18n]'); 11 var all = document.querySelectorAll('[i18n]');
10 for (var i = 0; i < all.length; ++i) { 12 for (var i = 0; i < all.length; ++i) {
11 var i18n = all[i].getAttribute('i18n'); 13 var i18n = all[i].getAttribute('i18n');
12 if (i18n) 14 if (i18n)
13 all[i].textContent = chrome.i18n.getMessage(i18n); 15 all[i].textContent = chrome.i18n.getMessage(i18n);
14 } 16 }
17
18 all = document.querySelectorAll('[i18n-alt]');
19 for (var i = 0; i < all.length; ++i) {
20 var i18n = all[i].getAttribute('i18n-alt');
21 if (i18n) {
Devlin 2016/11/01 15:12:14 When is this falsey? (Same for line 14)
Pam (message me for reviews) 2016/11/01 21:29:03 Leftover from a past version, when it iterated ove
22 var message = chrome.i18n.getMessage(i18n);
23 all[i].alt = message;
24 all[i].title = message;
25 }
26 }
15 } 27 }
16 28
17 // Open a pre-filled email to send feedback to the extension developers. The 29 // Open a pre-filled email to send feedback to the extension developers. The
18 // initial content of the email depends on whether the URL of the current page 30 // initial content of the email depends on whether the URL of the current page
19 // is provided. 31 // is provided.
20 function reportPage(url) { 32 function sendFeedback(url) {
21 var subject = chrome.i18n.getMessage('reportSubject'); 33 var subject = chrome.i18n.getMessage('reportSubject');
22 var body = ''; 34 var body = '';
23 if (url) 35 if (url)
24 body = chrome.i18n.getMessage('reportBodyWithURL', url); 36 body = chrome.i18n.getMessage('reportBodyWithURL', url);
25 else 37 else
26 body = chrome.i18n.getMessage('reportBody'); 38 body = chrome.i18n.getMessage('reportBody');
27 var msg = 'mailto:gobackwithbackspace@google.com' + 39 var msg = 'mailto:gobackwithbackspace@google.com' +
28 '?subject=' + encodeURIComponent(subject) + 40 '?subject=' + encodeURIComponent(subject) +
29 '&body=' + encodeURIComponent(body); 41 '&body=' + encodeURIComponent(body);
30 chrome.tabs.create({ 42 chrome.tabs.create({
31 url: msg, 43 url: msg,
32 active: true}); 44 active: true});
33 } 45 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698