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

Unified Diff: chrome/browser/resources/md_extensions/code_section.js

Issue 2948443003: Run clang-format on .js files in c/b/r/md_extensions (Closed)
Patch Set: Created 3 years, 6 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
Index: chrome/browser/resources/md_extensions/code_section.js
diff --git a/chrome/browser/resources/md_extensions/code_section.js b/chrome/browser/resources/md_extensions/code_section.js
index 502c86dc1fffea9e53fba639fb71bef9c0751e92..d2f4e8f32eb42cb8d4f7c8c61bfdb34771503bf7 100644
--- a/chrome/browser/resources/md_extensions/code_section.js
+++ b/chrome/browser/resources/md_extensions/code_section.js
@@ -35,8 +35,8 @@ cr.define('extensions', function() {
*/
isEmpty: function() {
return !this.code ||
- (!this.code.beforeHighlight && !this.code.highlight &&
- !this.code.afterHighlight);
+ (!this.code.beforeHighlight && !this.code.highlight &&
Devlin 2017/06/17 02:25:39 this now violates the rectangle rule, which I *tho
Dan Beam 2017/06/19 23:01:24 there's an internal reference to this rectangle ru
Devlin 2017/06/20 16:02:06 The rule says you should be able to "draw" a recta
+ !this.code.afterHighlight);
},
/**
@@ -49,9 +49,9 @@ cr.define('extensions', function() {
if (!this.code)
return '';
- var lines = [this.code.beforeHighlight,
- this.code.highlight,
- this.code.afterHighlight].join('').match(/\n/g);
+ var lines = [
+ this.code.beforeHighlight, this.code.highlight, this.code.afterHighlight
+ ].join('').match(/\n/g);
var lineCount = lines ? lines.length : 0;
var textContent = '';
for (var i = 1; i <= lineCount; ++i)

Powered by Google App Engine
This is Rietveld 408576698