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

Unified Diff: ios/web/public/test/web_test_with_web_state.mm

Issue 2306053002: Revert of [ios] Implemented LoadHtml w/o using data:// URLs. (Closed)
Patch Set: Created 4 years, 3 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/public/test/web_test_with_web_state.h ('k') | ios/web/web_state/js/core_js_unittest.mm » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ios/web/public/test/web_test_with_web_state.mm
diff --git a/ios/web/public/test/web_test_with_web_state.mm b/ios/web/public/test/web_test_with_web_state.mm
index c5e9aaef0c4e4b2854fe3514aa8124573f758a69..d16f409cc90343f0bbfd331f8f3adf33f18f4cbe 100644
--- a/ios/web/public/test/web_test_with_web_state.mm
+++ b/ios/web/public/test/web_test_with_web_state.mm
@@ -73,43 +73,48 @@
processed_a_task_ = true;
}
-void WebTestWithWebState::LoadHtml(NSString* html, const GURL& url) {
- ASSERT_FALSE(web_state()->IsLoading());
-
- CRWWebController* web_controller = GetWebController(web_state());
- [web_controller loadHTML:html forURL:url];
-
- // Wait until the navigation is committed to update MIME type.
- ASSERT_EQ(LOAD_REQUESTED, web_controller.loadPhase);
- base::TimeDelta spin_delay = base::TimeDelta::FromMilliseconds(10);
- while (web_controller.loadPhase != PAGE_LOADING)
- base::test::ios::SpinRunLoopWithMaxDelay(spin_delay);
-
- // loadHTML:forURL: does not notify web view delegate about received response,
- // so web controller does not get a chance to properly update MIME type and it
- // should be set manually after navigation is committed but before WebState
- // signal load completion and clients will start checking if MIME type is in
- // fact HTML.
- [web_controller webStateImpl]->SetContentsMimeType("text/html");
-
- // Wait until the page is loaded.
- ASSERT_EQ(PAGE_LOADING, web_controller.loadPhase);
- while (web_controller.loadPhase != PAGE_LOADED)
- base::test::ios::SpinRunLoopWithMaxDelay(spin_delay);
-
- // Wait until scripts execution becomes possible.
- base::test::ios::WaitUntilCondition(^bool {
- return [ExecuteJavaScript(@"0;") isEqual:@0];
- });
-}
-
void WebTestWithWebState::LoadHtml(NSString* html) {
- GURL url("https://chromium.test/");
- LoadHtml(html, url);
+ LoadHtml([html UTF8String]);
}
void WebTestWithWebState::LoadHtml(const std::string& html) {
- LoadHtml(base::SysUTF8ToNSString(html));
+ NSString* load_check = CreateLoadCheck();
+ std::string marked_html = html + [load_check UTF8String];
+ std::string encoded_html;
+ base::Base64Encode(marked_html, &encoded_html);
+ GURL url("data:text/html;charset=utf8;base64," + encoded_html);
+ LoadURL(url);
+
+ if (ResetPageIfNavigationStalled(load_check)) {
+ LoadHtml(html);
+ }
+}
+
+void WebTestWithWebState::LoadURL(const GURL& url) {
+ // First step is to ensure that the web controller has finished any previous
+ // page loads so the new load is not confused.
+ while ([GetWebController(web_state()) loadPhase] != PAGE_LOADED)
+ WaitForBackgroundTasks();
+ id originalMockDelegate =
+ [OCMockObject niceMockForProtocol:@protocol(CRWWebDelegate)];
+ id mockDelegate =
+ [[WebDelegateMock alloc] initWithRepresentedObject:originalMockDelegate];
+ id existingDelegate = GetWebController(web_state()).delegate;
+ GetWebController(web_state()).delegate = mockDelegate;
+
+ web::NavigationManagerImpl& navManager =
+ [GetWebController(web_state()) webStateImpl]->GetNavigationManagerImpl();
+ navManager.InitializeSession(@"name", nil, NO, 0);
+ [navManager.GetSessionController() addPendingEntry:url
+ referrer:web::Referrer()
+ transition:ui::PAGE_TRANSITION_TYPED
+ rendererInitiated:NO];
+
+ [GetWebController(web_state()) loadCurrentURL];
+ while ([GetWebController(web_state()) loadPhase] != PAGE_LOADED)
+ WaitForBackgroundTasks();
+ GetWebController(web_state()).delegate = existingDelegate;
+ [web_state()->GetView() layoutIfNeeded];
}
void WebTestWithWebState::WaitForBackgroundTasks() {
@@ -172,6 +177,18 @@
return web_state_.get();
}
+bool WebTestWithWebState::ResetPageIfNavigationStalled(NSString* load_check) {
+ id inner_html = ExecuteJavaScript(
+ @"(document && document.body && document.body.innerHTML) || 'undefined'");
+ if (![inner_html rangeOfString:load_check].length) {
+ web_state_->SetWebUsageEnabled(false);
+ web_state_->SetWebUsageEnabled(true);
+ [GetWebController(web_state()) triggerPendingLoad];
+ return true;
+ }
+ return false;
+}
+
NSString* WebTestWithWebState::CreateLoadCheck() {
return [NSString stringWithFormat:@"<p style=\"display: none;\">%d</p>",
s_html_load_count++];
« no previous file with comments | « ios/web/public/test/web_test_with_web_state.h ('k') | ios/web/web_state/js/core_js_unittest.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698