Implement NavigationManager::Reload() without requiring a web delegate. |
|||
Issue descriptionThe current implementation of NavigationManager::Reload() goes through WebStateImpl::OpenURL(), which in turn calls the web delegate's |-openURLWithParams:|. The web layer should handle reloads internally, as the web::WebState::OpenURLParams::disposition logic in [Tab openURLWithParams:| is not necessary for reloads since it will always occur in the same Tab.
,
Dec 29 2016
I would add that NavigationManager::Reload should call [WKWebView reload] to avoid messing up with WKBackForwardNavigationList. This can be piped as follows: NavigationManagerDelegate -> WebStateImpl -> CRWWebController.
,
Mar 15 2017
,
Apr 6 2017
This is already done |
|||
►
Sign in to add a comment |
|||
Comment 1 by kkhorimoto@chromium.org
, Dec 29 2016