Use ui::PAGE_TRANSITION_RELOAD transition for web page reload.

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 c51d5bf313e9dfbcb30dd75a5f69d7bfe8893670..a169d6d51fa480e7964f27bab156fc0bb6e14aed 100644
--- a/ios/web/web_state/ui/crw_web_controller.mm
+++ b/ios/web/web_state/ui/crw_web_controller.mm
@@ -2052,10 +2052,8 @@ registerLoadRequestForURL:(const GURL&)requestURL
           [transientItem->GetHttpRequestHeaders() copy]);
       [self loadWithParams:reloadParams];
     } else {
-      // As with back and forward navigation, load the URL manually instead of
-      // using the web view's reload. This ensures state processing and delegate

[Eugene But (OOO till 7-30), 2017/05/25 17:10:08]

This is not true for WKWebView. |loadCurrentURL| will eventually call
-[WKWebView reload]

-      // calls are consistent.
-      // TODO(eugenebut): revisit this for WKWebView.
+      self.currentNavItem->SetTransitionType(
+          ui::PageTransition::PAGE_TRANSITION_RELOAD);

[kkhorimoto, 2017/05/26 17:32:42]

Is this what's done in content// as well?  It seems weird that we love the
previous page transition type upon reloading.  I checked, and it looks like
RELOAD is not a bitwise enum, so we can't combine with the previous value, but
do they actually reset the NavigationEntry's transition type, or do they just
pass the RELOAD type around to interested functions?

[Eugene But (OOO till 7-30), 2017/05/26 17:48:32]

I'm pretty sure they reset navigation to reload which makes total sense to me:
https://cs.chromium.org/chromium/src/content/browser/frame_host/navigation_co...

       [self loadCurrentURL];
     }
   }