NavigationController: Reload methods migration

extensions/browser/guest_view/web_view/web_view_guest.cc

Index: extensions/browser/guest_view/web_view/web_view_guest.cc
diff --git a/extensions/browser/guest_view/web_view/web_view_guest.cc b/extensions/browser/guest_view/web_view/web_view_guest.cc
index 97e34148d84d118fb116692c5be450faeb05e3fe..9627ced42d11befd624c4d859e18ee4078794177 100644
--- a/extensions/browser/guest_view/web_view/web_view_guest.cc
+++ b/extensions/browser/guest_view/web_view/web_view_guest.cc
@@ -727,7 +727,7 @@ void WebViewGuest::Reload() {
   // TODO(fsamuel): Don't check for repost because we don't want to show
   // Chromium's repost warning. We might want to implement a separate API
   // for registering a callback if a repost is about to happen.
-  web_contents()->GetController().Reload(false);
+  web_contents()->GetController().Reload(content::ReloadType::NORMAL, false);

[Takashi Toyoshima, 2016/12/15 06:21:52]

IIUC, this is for Chrome Apps <webview> tag. If so, it does not have a prompting
UI. For the same reason, it provides a manifest description for API permissions.
So using false is correct.

 }
 
 void WebViewGuest::SetUserAgentOverride(
@@ -888,7 +888,7 @@ void WebViewGuest::UserAgentOverrideSet(const std::string& user_agent) {
   if (!entry)
     return;
   entry->SetIsOverridingUserAgent(!user_agent.empty());
-  web_contents()->GetController().Reload(false);
+  web_contents()->GetController().Reload(content::ReloadType::NORMAL, false);

[Takashi Toyoshima, 2016/12/15 06:21:52]

ditto

 }
 
 void WebViewGuest::FrameNameChanged(RenderFrameHost* render_frame_host,