[ios] Reduce access to private //ios/web API.

ios/chrome/browser/tabs/tab_model_selected_tab_observer.mm

Index: ios/chrome/browser/tabs/tab_model_selected_tab_observer.mm
diff --git a/ios/chrome/browser/tabs/tab_model_selected_tab_observer.mm b/ios/chrome/browser/tabs/tab_model_selected_tab_observer.mm
index 5f19d6183c3039641cb996c39cea4291bc3c4fb5..92b51ebdb74bee98470a2d86fee0f7e08c595188 100644
--- a/ios/chrome/browser/tabs/tab_model_selected_tab_observer.mm
+++ b/ios/chrome/browser/tabs/tab_model_selected_tab_observer.mm
@@ -32,13 +32,10 @@
                 oldWebState:(web::WebState*)oldWebState
                     atIndex:(int)atIndex
                  userAction:(BOOL)userAction {
-  Tab* oldTab = nil;
-  Tab* newTab = nil;
   if (oldWebState) {
     // Save state, such as scroll position, ... of the old selected Tab.
-    oldTab = LegacyTabHelper::GetTabForWebState(oldWebState);
-    if (userAction)
-      [oldTab recordStateInHistory];

[kkhorimoto, 2017/04/11 18:20:00]

From what I can tell, it looks like we're no longer recording state when we
switch Tabs.  It looks like WebStates record their display state when
loads/navigations occur and when the session storage is built.  If we were to
enter the Tab switcher and a WKWebView is evicted, do we have a mechanism to
restore the scroll offset when the page is reloaded?  Are we building a session
storage every time we enter the Tab Switcher?

[Eugene But (OOO till 7-30), 2017/04/11 18:39:39]

Would it be useful to commit "safe" part in one CL and then work on case by case
basis to remove |recordStateInHistory| usage from places which can potentially
break the app?

+    Tab* oldTab = LegacyTabHelper::GetTabForWebState(oldWebState);
+    DCHECK(oldTab);

[Eugene But (OOO till 7-30), 2017/04/11 14:50:48]

nit: I don't think you need to DCHECK here. |@{kTabModelTabKey : oldTab}| will
crash anyways.

 
     // Avoid artificially extending the lifetime of oldTab until the global
     // autoreleasepool is purged.
@@ -51,7 +48,7 @@
   }
 
   if (newWebState) {
-    newTab = LegacyTabHelper::GetTabForWebState(newWebState);
+    Tab* newTab = LegacyTabHelper::GetTabForWebState(newWebState);
     [newTab updateLastVisitedTimestamp];
 
     // Persist the session state.