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

Unified Diff: components/neterror/resources/neterror.js

Issue 2617823002: GRIT: put <if> and <include> behind comments in .js files to make syntactically valid JS (Closed)
Patch Set: add dep Created 3 years, 11 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: components/neterror/resources/neterror.js
diff --git a/components/neterror/resources/neterror.js b/components/neterror/resources/neterror.js
index 1926b5c06f6e4264a78e203523c194283eec2553..16175091a1d1fd2fb65451f4b387ab93b194ecf6 100644
--- a/components/neterror/resources/neterror.js
+++ b/components/neterror/resources/neterror.js
@@ -22,17 +22,17 @@ function toggleHelpBox() {
}
function diagnoseErrors() {
-<if expr="not chromeos">
+// <if expr="not chromeos">
if (window.errorPageController)
errorPageController.diagnoseErrorsButtonClick();
-</if>
-<if expr="chromeos">
+// </if>
+// <if expr="chromeos">
var extensionId = 'idddmepepmjcgiedknnmlbadcokidhoa';
var diagnoseFrame = document.getElementById('diagnose-frame');
diagnoseFrame.innerHTML =
'<iframe src="chrome-extension://' + extensionId +
'/index.html"></iframe>';
-</if>
+// </if>
}
// Subframes use a different layout but the same html file. This is to make it
@@ -150,9 +150,9 @@ function setUpCachedButton(buttonStrings) {
}
var primaryControlOnLeft = true;
-<if expr="is_macosx or is_ios or is_linux or is_android">
+// <if expr="is_macosx or is_ios or is_linux or is_android">
primaryControlOnLeft = false;
-</if>
+// </if>
function onDocumentLoad() {
var controlButtonDiv = document.getElementById('control-buttons');

Powered by Google App Engine
This is Rietveld 408576698