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

Unified Diff: chrome/browser/resources/certificate_viewer.js

Issue 10010019: JS style nits (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: couple more Created 8 years, 8 months 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/resources/about_stats.js ('k') | chrome/browser/resources/crashes.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/resources/certificate_viewer.js
diff --git a/chrome/browser/resources/certificate_viewer.js b/chrome/browser/resources/certificate_viewer.js
index 315552198177acda9f374ce2245b4180d5a2db55..95a5d821f66e43daa6e81354d81cdfdc55b6ddaa 100644
--- a/chrome/browser/resources/certificate_viewer.js
+++ b/chrome/browser/resources/certificate_viewer.js
@@ -21,9 +21,8 @@ cr.define('cert_viewer', function() {
chrome.send('requestCertificateInfo');
// TODO(kochi): ESC key should be handled in the views window side.
document.addEventListener('keydown', function(e) {
- if (e.keyCode == 27) { // ESC
+ if (e.keyCode == 27) // ESC
chrome.send('DialogClose');
- }
});
}
@@ -42,7 +41,7 @@ cr.define('cert_viewer', function() {
/**
* The tab name strings in the languages file have accessor keys indicated
* by a preceding & sign. Strip these out for now.
- * @TODO(flackr) These accessor keys could be implemented with Javascript or
+ * TODO(flackr) These accessor keys could be implemented with Javascript or
* translated strings could be added / modified to remove the & sign.
*/
function stripGtkAccessorKeys() {
@@ -50,7 +49,7 @@ cr.define('cert_viewer', function() {
var nodes = Array.prototype.slice.call($('tabs').childNodes, 0);
nodes.push($('export'));
for (var i = 0; i < nodes.length; i++)
- nodes[i].textContent = nodes[i].textContent.replace('&','');
+ nodes[i].textContent = nodes[i].textContent.replace('&', '');
}
/**
« no previous file with comments | « chrome/browser/resources/about_stats.js ('k') | chrome/browser/resources/crashes.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698