Refactoring Reload in NavigationManager and CRWWebController.

ios/web/web_state/ui/crw_web_controller.mm

Index: ios/web/web_state/ui/crw_web_controller.mm
diff --git a/ios/web/web_state/ui/crw_web_controller.mm b/ios/web/web_state/ui/crw_web_controller.mm
index 81dd6e629220c078b2e7d23b102f4c48005a254e..d03ed0cea5bfd6c25680210d12ba630b53e82511 100644
--- a/ios/web/web_state/ui/crw_web_controller.mm
+++ b/ios/web/web_state/ui/crw_web_controller.mm
@@ -1844,6 +1844,12 @@ - (void)loadCurrentURLInNativeView {
 }
 
 - (void)loadWithParams:(const NavigationManager::WebLoadParams&)originalParams {
+  if (PageTransitionCoreTypeIs(originalParams.transition_type,
+                               ui::PAGE_TRANSITION_RELOAD)) {
+    [self reload];
+    return;
+  }

[kkhorimoto, 2017/03/16 18:13:04]

This is no longer needed right?  How about a DCHECK to ensure that reloads don't
go through this API.

[Eugene But (OOO till 7-30), 2017/03/16 18:39:04]

+1

[liaoyuke, 2017/03/17 17:59:21]

Done.

+
   // Make a copy of |params|, as some of the delegate methods may modify it.
   NavigationManager::WebLoadParams params(originalParams);