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

Unified Diff: ios/web/web_state/js/resources/core.js

Issue 2814773002: Move password-related methods from core.js to password_controller.js. (Closed)
Patch Set: Address reviewer comment (fix verb tense in test comment) Created 3 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
« no previous file with comments | « ios/web/web_state/js/core_js_unittest.mm ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ios/web/web_state/js/resources/core.js
diff --git a/ios/web/web_state/js/resources/core.js b/ios/web/web_state/js/resources/core.js
index 878481a83e29cc7cfc9d3e1940370966fdbade8b..6ec37f02d7edc1bdbdd4da12b66484df983b1e42 100644
--- a/ios/web/web_state/js/resources/core.js
+++ b/ios/web/web_state/js/resources/core.js
@@ -40,40 +40,6 @@ goog.require('__crWeb.message');
{'command': 'window.error', 'message': event.message.toString()});
});
-
- // Returns true if the top window or any frames inside contain an input
- // field of type 'password'.
- __gCrWeb['hasPasswordField'] = function() {
- return hasPasswordField_(window);
- };
-
-
- // Returns true if the supplied window or any frames inside contain an input
- // field of type 'password'.
- // @private
- var hasPasswordField_ = function(win) {
- var doc = win.document;
-
- // We may will not be allowed to read the 'document' property from a frame
- // that is in a different domain.
- if (!doc) {
- return false;
- }
-
- if (doc.querySelector('input[type=password]')) {
- return true;
- }
-
- var frames = win.frames;
- for (var i = 0; i < frames.length; i++) {
- if (hasPasswordField_(frames[i])) {
- return true;
- }
- }
-
- return false;
- };
-
__gCrWeb['sendFaviconsToHost'] = function() {
__gCrWeb.message.invokeOnHost({'command': 'document.favicons',
'favicons': __gCrWeb.common.getFavicons()});
« no previous file with comments | « ios/web/web_state/js/core_js_unittest.mm ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698