Refactoring Reload in NavigationManager and CRWWebController.

ios/web/navigation/navigation_manager_impl.mm

Index: ios/web/navigation/navigation_manager_impl.mm
diff --git a/ios/web/navigation/navigation_manager_impl.mm b/ios/web/navigation/navigation_manager_impl.mm
index 9acc14128e0cf1c54c5558b26371a22b9fae8f39..56ff351911d84128a929b9c05c3637189421c607 100644
--- a/ios/web/navigation/navigation_manager_impl.mm
+++ b/ios/web/navigation/navigation_manager_impl.mm
@@ -348,16 +348,7 @@ bool AreURLsInPageNavigation(const GURL& existing_url, const GURL& new_url) {
 
 void NavigationManagerImpl::Reload(ReloadType reload_type,
                                    bool check_for_reposts) {
-  // Navigation manager may be empty if the only pending item failed to load
-  // with SSL error and the user has decided not to proceed.
-  NavigationItem* item = GetVisibleItem();
-  GURL url = item ? item->GetURL() : GURL(url::kAboutBlankURL);
-  web::Referrer referrer = item ? item->GetReferrer() : web::Referrer();
-
-  WebState::OpenURLParams params(url, referrer,
-                                 WindowOpenDisposition::CURRENT_TAB,
-                                 ui::PAGE_TRANSITION_RELOAD, NO);
-  delegate_->GetWebState()->OpenURL(params);
+  delegate_->Reload();

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

We can handle |reload_type| here by updating the visible NavigaitonItem's URL to
its original URL.  However, I think we're going to need to handle the
|check_for_repost| parameter in WebStateImpl or CRWWebController, so we should
pass |check_for_repost| in this delegate callback.

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

Since it's still unclear where we want to handle |check_for_repost|, I think we
should keep the APIs simple for now to avoid potential confusions, and add
necessary parameters later when actually implement it.

 }
 
 std::unique_ptr<std::vector<BrowserURLRewriter::URLRewriter>>