(Set)IsOverridingUserAgent should be called on VisibleItem

ios/web/navigation/navigation_manager_impl.h

Index: ios/web/navigation/navigation_manager_impl.h
diff --git a/ios/web/navigation/navigation_manager_impl.h b/ios/web/navigation/navigation_manager_impl.h
index 11663bd8a2d402c64e304fc04f87860d430cc48d..f869c8c2ee53fbb51dea7abcdb5281e73610fbae 100644
--- a/ios/web/navigation/navigation_manager_impl.h
+++ b/ios/web/navigation/navigation_manager_impl.h
@@ -145,6 +145,10 @@ class NavigationManagerImpl : public NavigationManager {
   void Reload(bool check_for_reposts) override;
   void OverrideDesktopUserAgentForNextPendingItem() override;
 
+  // Whether or not desktop user agent is used for the currently loaded page.
+  // Returns true, if use a desktop user agent and false for a mobile one.
+  bool GetUseDesktopUserAgent() const;

[kkhorimoto, 2017/02/22 01:54:33]

This should also be moved to the public NavigationManager interface, for the
same reason as you moved the other function.

[liaoyuke, 2017/02/22 02:55:24]

Done.

+
   // Returns the current list of transient url rewriters, passing ownership to
   // the caller.
   // TODO(crbug.com/546197): remove once NavigationItem creation occurs in this