Remove CRWSessionEntry from NavigationManagerImpl.

ios/web/public/navigation_manager.h

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef IOS_WEB_PUBLIC_NAVIGATION_MANAGER_H_
 #define IOS_WEB_PUBLIC_NAVIGATION_MANAGER_H_
 
 #include <stddef.h>
 
 #import "base/mac/scoped_nsobject.h"
@@ skipping 94 matching lines @@
 
   // Adds |rewriter| to a transient list of URL rewriters.  Transient URL
   // rewriters will be executed before the rewriters already added to the
   // BrowserURLRewriter singleton, and the list will be cleared after the next
   // attempted page load.  |rewriter| must not be null.
   virtual void AddTransientURLRewriter(
       BrowserURLRewriter::URLRewriter rewriter) = 0;
 
   // Returns the number of items in the NavigationManager, excluding
   // pending and transient entries.
+  // TODO(crbug.com/533848): Update to return size_t.
   virtual int GetItemCount() const = 0;
 
   // Returns the committed NavigationItem at |index|.
   virtual NavigationItem* GetItemAtIndex(size_t index) const = 0;
 
   // Returns the index from which web would go back/forward or reload.
+  // TODO(crbug.com/533848): Update to return size_t.
   virtual int GetCurrentItemIndex() const = 0;
 
   // Returns the index of the last committed item or -1 if the last
   // committed item correspond to a new navigation.
+  // TODO(crbug.com/533848): Update to return size_t.
   virtual int GetLastCommittedItemIndex() const = 0;
 
   // Returns the index of the pending item or -1 if the pending item
   // corresponds to a new navigation.
+  // TODO(crbug.com/533848): Update to return size_t.
   virtual int GetPendingItemIndex() const = 0;
 
   // Removes the item at the specified |index|.  If the index is the last
   // committed index or the pending item, this does nothing and returns false.
   // Otherwise this call discards any transient or pending entries.
+  // TODO(crbug.com/533848): Update to use size_t.
   virtual bool RemoveItemAtIndex(int index) = 0;
 
   // Navigation relative to the current item.
   virtual bool CanGoBack() const = 0;
   virtual bool CanGoForward() const = 0;
   virtual bool CanGoToOffset(int offset) const = 0;
   virtual void GoBack() = 0;
   virtual void GoForward() = 0;
 
   // Navigates to the specified absolute index.
+  // TODO(crbug.com/533848): Update to use size_t.
   virtual void GoToIndex(int index) = 0;
 
   // Reloads the current item. If |check_for_repost| is true and the current
   // item has POST data the user is prompted to see if they really want to
   // reload the page. In nearly all cases pass in true.  If a transient item is
   // showing, initiates a new navigation to its URL.
   virtual void Reload(bool check_for_repost) = 0;
 
   // Forces the pending item to be loaded using desktop user agent. Note that
   // the pending item may or may not already exist.
   // TODO(crbug.com/692303): Remove this when overriding the user agent doesn't
   // create a new NavigationItem.
   virtual void OverrideDesktopUserAgentForNextPendingItem() = 0;
 };
 
 }  // namespace web
 
 #endif  // IOS_WEB_PUBLIC_NAVIGATION_MANAGER_H_