Reading List iOS: Adapt loading offline for new reload process

ios/chrome/browser/reading_list/reading_list_web_state_observer.mm

Index: ios/chrome/browser/reading_list/reading_list_web_state_observer.mm
diff --git a/ios/chrome/browser/reading_list/reading_list_web_state_observer.mm b/ios/chrome/browser/reading_list/reading_list_web_state_observer.mm
index 58ac2e84151042de615158c96e3823be937c52a7..69f8c7528f093931bcd384583ad3afb2b48865aa 100644
--- a/ios/chrome/browser/reading_list/reading_list_web_state_observer.mm
+++ b/ios/chrome/browser/reading_list/reading_list_web_state_observer.mm
@@ -295,8 +295,8 @@ void ReadingListWebStateObserver::LoadOfflineReadingListEntry() {
     item = navigationManager->GetLastCommittedItem();
     item->SetURL(url);
     item->SetVirtualURL(pending_url_);
-    navigationManager->Reload(web::ReloadType::NORMAL,

[Eugene But (OOO till 7-30), 2017/03/21 15:48:19]

The fact that Reload does not work anymore looks like a bug. Yuke's CL was a
refactoring and behavior change was not intentional and should be treated as a
bug.

[Olivier, 2017/03/21 16:41:31]

The problem here is that 
navigationManager->Reload calls CRWWebController reload.
At that point, there is an error page, so CRWWebController calls
ErrorPageContent reload.
The item defined here is lost.

[Eugene But (OOO till 7-30), 2017/03/21 17:10:24]

Thank you for explanation. I asked Kurt if this is WAI on the bug. 

-                              false /*check_for_repost*/);
+    navigationManager->GoToIndex(
+        navigationManager->GetLastCommittedItemIndex());
   } else if (navigationManager->GetPendingItemIndex() != -1 &&
              navigationManager->GetItemAtIndex(
                  navigationManager->GetPendingItemIndex()) == item) {