Print Preview; Replace cr.isWin with cr.isWindows.

cr.isWin() does not exist. Also fix formatting for some TODO entries.

Change-Id: Ib64b2eeac0166484e60bf93c5a60159def81afec
Reviewed-on: https://chromium-review.googlesource.com/1204590Reviewed-by: Rebekah Potter <rbpotter@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>
Cr-Commit-Position: refs/heads/master@{#588895}

chrome/browser/resources/print_preview/new/advanced_settings_dialog.js

@@ -44,7 +44,7 @@ Polymer({
                              .ADVANCED_SETTINGS_DIALOG_SHOWN);
     // This async() call is a workaround to prevent a DCHECK - see
     // https://crbug.com/804047.
-    // TODO (rbpotter): Remove after Polymer2 migration is complete.
+    // TODO(rbpotter): Remove after Polymer2 migration is complete.
     this.async(() => {
       this.$.dialog.showModal();
     }, 1);

chrome/browser/resources/print_preview/new/app.js

@@ -268,7 +268,7 @@ Polymer({
         // Don't try to print with system dialog on Windows if the document is
         // not ready, because we send the preview document to the printer on
         // Windows.
-        if (!cr.isWin || this.state == print_preview_new.State.READY)
+        if (!cr.isWindows || this.state == print_preview_new.State.READY)
           this.onPrintWithSystemDialog_();
         e.preventDefault();
         return;
@@ -305,7 +305,7 @@ Polymer({
   onCrDialogClose_: function(e) {
     // Note: due to event re-firing in cr_dialog.js, this event will always
     // appear to be coming from the outermost child dialog.
-    // TODO (rbpotter): Fix event re-firing so that the event comes from the
+    // TODO(rbpotter): Fix event re-firing so that the event comes from the
     // dialog that has been closed, and add an assertion that the removed
     // dialog matches e.composedPath()[0].
     if (e.composedPath()[0].nodeName == 'CR-DIALOG')

chrome/browser/resources/print_preview/new/destination_settings.js

@@ -89,7 +89,7 @@ Polymer({
     const dialog = this.$.destinationDialog.get();
     // This async() call is a workaround to prevent a DCHECK - see
     // https://crbug.com/804047.
-    // TODO (rbpotter): Remove after Polymer2 migration is complete.
+    // TODO(rbpotter): Remove after Polymer2 migration is complete.
     this.async(() => {
       dialog.show();
     }, 1);

chrome/browser/resources/print_preview/new/model.js

@@ -755,7 +755,7 @@ Polymer({
     }
 
     if (destination.isPrivet || destination.isExtension) {
-      // TODO (rbpotter): Get local and PDF printers to use the same ticket and
+      // TODO(rbpotter): Get local and PDF printers to use the same ticket and
       // send only this ticket instead of nesting it in a larger ticket.
       ticket.ticket = this.createCloudJobTicket(destination);
       ticket.capabilities = JSON.stringify(destination.capabilities);