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

Unified Diff: chrome/browser/resources/inline_login/inline_login.js

Issue 239143009: Disallow top-level navigation in gaia iframe (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: disable flaky test Created 6 years, 8 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/inline_login/inline_login.js
diff --git a/chrome/browser/resources/inline_login/inline_login.js b/chrome/browser/resources/inline_login/inline_login.js
index 8aa1c2a3bc2c490af5b8021fd46a58c5c1b9521a..388b5dd607ebba0672a4d9b5f121844307f0508c 100644
--- a/chrome/browser/resources/inline_login/inline_login.js
+++ b/chrome/browser/resources/inline_login/inline_login.js
@@ -18,10 +18,16 @@ cr.define('inline.login', function() {
var authExtHost;
/**
+ * Whether the auth ready event has been fired, for testing purpose.
+ */
+ var authReadyFired;
+
+ /**
* Handler of auth host 'ready' event.
*/
function onAuthReady() {
$('contents').classList.toggle('loading', false);
+ authReadyFired = true;
}
/**
@@ -69,7 +75,23 @@ cr.define('inline.login', function() {
$('contents').classList.toggle('loading', true);
}
+ /**
+ * Returns the auth host instance, for testing purpose.
+ */
+ function getAuthExtHost() {
+ return authExtHost;
+ }
+
+ /**
+ * Returns whether the auth UI is ready, for testing purpose.
+ */
+ function isAuthReady() {
+ return authReadyFired;
+ }
+
return {
+ getAuthExtHost: getAuthExtHost,
+ isAuthReady: isAuthReady,
initialize: initialize,
loadAuthExtension: loadAuthExtension,
closeDialog: closeDialog,
« no previous file with comments | « chrome/browser/resources/gaia_auth/main.html ('k') | chrome/browser/ui/webui/signin/inline_login_ui_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698