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

Unified Diff: ios/web_view/internal/cwv_web_view.mm

Issue 2741413007: Refactoring Reload in NavigationManager and CRWWebController. (Closed)
Patch Set: fixed more check_for_repost errors Created 3 years, 9 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: ios/web_view/internal/cwv_web_view.mm
diff --git a/ios/web_view/internal/cwv_web_view.mm b/ios/web_view/internal/cwv_web_view.mm
index ff7c2d1e01182d1fec1f846744d6a9de1582f80c..ae66ac0dee4b9dbb13af3b64ec1b8260fbf97b65 100644
--- a/ios/web_view/internal/cwv_web_view.mm
+++ b/ios/web_view/internal/cwv_web_view.mm
@@ -132,7 +132,10 @@ - (void)goForward {
}
- (void)reload {
- _webState->GetNavigationManager()->Reload(web::ReloadType::NORMAL, true);
+ // |check_for_repost| is false because CWVWebView does not support form
Eugene But (OOO till 7-30) 2017/03/16 18:39:04 s/form resubmission/repost form Sorry we have spe
liaoyuke 2017/03/17 17:59:21 Done.
+ // resubmission dialogs.
+ _webState->GetNavigationManager()->Reload(web::ReloadType::NORMAL,
+ false /* check_for_repost */);
}
- (void)stopLoading {
« ios/web/web_state/ui/crw_web_controller.mm ('K') | « ios/web/web_state/web_state_impl.mm ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698