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

Unified Diff: ios/chrome/browser/autofill/form_suggestion_controller_unittest.mm

Issue 2888773003: Do not create CRWWebUIManager for non-WebUI URLs. (Closed)
Patch Set: Updated comments 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
« no previous file with comments | « no previous file | ios/web/web_state/ui/crw_web_controller.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ios/chrome/browser/autofill/form_suggestion_controller_unittest.mm
diff --git a/ios/chrome/browser/autofill/form_suggestion_controller_unittest.mm b/ios/chrome/browser/autofill/form_suggestion_controller_unittest.mm
index e65903e05c6785b5cf395b74fd922f2b55531b69..2bbd34cebb1a77af2fe33f3aefd2a59ef0dea6ae 100644
--- a/ios/chrome/browser/autofill/form_suggestion_controller_unittest.mm
+++ b/ios/chrome/browser/autofill/form_suggestion_controller_unittest.mm
@@ -157,7 +157,9 @@ class FormSuggestionControllerTest : public ChromeWebTest {
}
// Sets |url| to be current for WebState.
- void SetCurrentUrl(const std::string& url) { LoadHtml(@"", GURL(url)); }
+ void SetCurrentUrl(const std::string& url) {
+ LoadHtml(@"<html></html>", GURL(url));
+ }
// Swizzles the current web controller to set whether the content is HTML.
void SetContentIsHtml(BOOL content_is_html) {
« no previous file with comments | « no previous file | ios/web/web_state/ui/crw_web_controller.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698