Fix all remaining print preview closure compiler errors

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">
 //
 
@@ skipping 1333 matching lines @@
 // <include src="../pdf/pdf_scripting_api.js">
 // <include src="previewarea/preview_area.js">
 // <include src="preview_generator.js">
 
 // <include src="search/destination_list.js">
 // <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/provisional_destination_resolver.js">
-
+var printPreview;

[dpapad, 2017/05/08 20:40:10]

Can this be moved inside the anonymous function below?

var printPreview = new print_preview.PrintPreview();
printPreview.initialize();

[rbpotter, 2017/05/08 23:28:20]

Done. FYI, This triggered a bunch of new errors from some enums,
which is why there are some additional changes in the new
patchset. Also caused problems for the browser tests that were
relying on the global variable, so I added the boolean above and
check in this function to fix them. Let me know if this was not
a correct approach.

 window.addEventListener('DOMContentLoaded', function() {
   printPreview = new print_preview.PrintPreview();
   printPreview.initialize();
 });