Fix ESC not closing the print preview dialog on Mac

chrome/browser/resources/print_preview/print_preview.js

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // TODO(rltoscano): Move data/* into print_preview.data namespace
 
 <include src="component.js">
 <include src="print_preview_focus_manager.js">
 
 cr.define('print_preview', function() {
@@ skipping 854 matching lines @@
      * @param {KeyboardEvent} e The keyboard event.
      * @private
      * @suppress {uselessCode}
      * Current compiler preprocessor leaves all the code inside all the <if>s,
      * so the compiler claims that code after first return is unreachable.
      */
     onKeyDown_: function(e) {
       // Escape key closes the dialog.
       if (e.keyCode == 27 && !e.shiftKey && !e.ctrlKey && !e.altKey &&
           !e.metaKey) {
-<if expr="toolkit_views">
-        // On the toolkit_views environment, ESC key is handled by C++-side
-        // instead of JS-side.
-        return;
-</if>
-<if expr="not toolkit_views">
-        this.close_();
-</if>
-        e.preventDefault();
+        // On non-mac with toolkit-views, ESC key is handled by C++-side instead
+        // of JS-side.
+        if (cr.isMac) {
+          this.close_();
+          e.preventDefault();
+        }
         return;
       }
 
       // On Mac, Cmd- should close the print dialog.
       if (cr.isMac && e.keyCode == 189 && e.metaKey) {
         this.close_();
         e.preventDefault();
         return;
       }
 
@@ skipping 421 matching lines @@
 <include src="search/cloud_destination_list.js">
 <include src="search/recent_destination_list.js">
 <include src="search/destination_list_item.js">
 <include src="search/destination_search.js">
 <include src="search/fedex_tos.js">
 
 window.addEventListener('DOMContentLoaded', function() {
   printPreview = new print_preview.PrintPreview();
   printPreview.initialize();
 });