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

Unified Diff: chrome/browser/resources/print_preview/print_preview_focus_manager.js

Issue 2863183004: Print Preview: Fix top level directory compile errors (Closed)
Patch Set: Created 3 years, 7 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/print_preview/print_preview_focus_manager.js
diff --git a/chrome/browser/resources/print_preview/print_preview_focus_manager.js b/chrome/browser/resources/print_preview/print_preview_focus_manager.js
index b5d74efdc21becda696c0e1517ffdbe2b0efe5ad..4aed146e385305e864bc1403f91e90f8096eae31 100644
--- a/chrome/browser/resources/print_preview/print_preview_focus_manager.js
+++ b/chrome/browser/resources/print_preview/print_preview_focus_manager.js
@@ -12,7 +12,7 @@ cr.define('print_preview', function() {
* @extends {cr.ui.FocusManager}
*/
function PrintPreviewFocusManager() {
- };
+ }
cr.addSingletonGetter(PrintPreviewFocusManager);
@@ -23,7 +23,7 @@ cr.define('print_preview', function() {
getFocusParent: function() {
var el = document.body;
var newEl = null;
- while (newEl = el.querySelector('.overlay:not([hidden])'))
+ while ((newEl = el.querySelector('.overlay:not([hidden])')))
dpapad 2017/05/05 23:30:29 That was probably an auto-fix for linter for http:
rbpotter 2017/05/06 00:02:19 Done.
el = newEl;
return el;
}

Powered by Google App Engine
This is Rietveld 408576698