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

Unified Diff: chrome/browser/resources/shared/js/cr/ui/dialogs.js

Issue 9594024: Fixed bug making it impossible to opt-out of auto-enrollment. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Added comment Created 8 years, 10 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/resources/shared/js/cr/ui/dialogs.js
diff --git a/chrome/browser/resources/shared/js/cr/ui/dialogs.js b/chrome/browser/resources/shared/js/cr/ui/dialogs.js
index f2ca51dc5c9628a41dfd2d903d19197f7d7bd50d..da683d66d7608a32330bfc79987b52dd6f1ed306 100644
--- a/chrome/browser/resources/shared/js/cr/ui/dialogs.js
+++ b/chrome/browser/resources/shared/js/cr/ui/dialogs.js
@@ -124,8 +124,11 @@ cr.define('cr.ui.dialogs', function() {
var iframes = doc.querySelectorAll('iframe');
for (var i = 0; i < iframes.length; i++) {
- elements = elements.concat(this.findFocusableElements_(
- iframes[i].contentDocument));
+ // Some iframes have an undefined contentDocument for security reasons,
+ // such as chrome://terms (which is used in the chromeos OOBE screens).
+ var contentDoc = iframes[i].contentDocument;
+ if (contentDoc)
+ elements = elements.concat(this.findFocusableElements_(contentDoc));
}
return elements;
};
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698