Do not reload UI if the Reading List entry does not exist

ios/chrome/browser/ui/reading_list/reading_list_egtest.mm

 // Copyright 2016 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.
 
 #import <EarlGrey/EarlGrey.h>
 #import <UIKit/UIKit.h>
 #import <XCTest/XCTest.h>
 
 #include "base/strings/sys_string_conversions.h"
 #include "base/strings/utf_string_conversions.h"
 #include "components/reading_list/ios/reading_list_model.h"
 #include "ios/chrome/browser/reading_list/reading_list_model_factory.h"
 #import "ios/chrome/browser/ui/commands/generic_chrome_command.h"
 #include "ios/chrome/browser/ui/commands/ios_command_ids.h"
 #import "ios/chrome/browser/ui/reading_list/reading_list_collection_view_item.h"
+#import "ios/chrome/browser/ui/reading_list/reading_list_empty_collection_background.h"
 #include "ios/chrome/browser/ui/ui_util.h"
 #include "ios/chrome/grit/ios_strings.h"
 #include "ios/chrome/grit/ios_theme_resources.h"
 #import "ios/chrome/test/app/chrome_test_util.h"
 #import "ios/chrome/test/earl_grey/accessibility_util.h"
 #import "ios/chrome/test/earl_grey/chrome_earl_grey.h"
 #import "ios/chrome/test/earl_grey/chrome_earl_grey_ui.h"
 #import "ios/chrome/test/earl_grey/chrome_matchers.h"
 #import "ios/chrome/test/earl_grey/chrome_test_case.h"
 #import "ios/testing/earl_grey/disabled_test_macros.h"
@@ skipping 162 matching lines @@
 // Computes the number of read entries in the model.
 size_t ModelReadSize(ReadingListModel* model) {
   size_t size = 0;
   for (const auto& url : model->Keys()) {
     if (model->GetEntryByURL(url)->IsRead()) {
       size++;
     }
   }
   return size;
 }
+
+// Returns a match for the Reading List Empty Collection Background.
+id<GREYMatcher> EmptyBackground() {
+  return grey_accessibilityID(
+      [ReadingListEmptyCollectionBackground accessibilityIdentifier]);
+}
 }  // namespace
 
 // Test class for the Reading List menu.
 @interface ReadingListTestCase : ChromeTestCase
 
 @end
 
 @implementation ReadingListTestCase
 
 - (void)setUp {
@@ skipping 313 matching lines @@
 
   // Tap the action sheet.
   TapButtonWithID(IDS_IOS_READING_LIST_MARK_READ_BUTTON);
 
   AssertAllEntriesVisible();
   XCTAssertEqual(kNumberReadEntries + 1, ModelReadSize(GetReadingListModel()));
   XCTAssertEqual(kNumberUnreadEntries - 1,
                  GetReadingListModel()->unread_size());
 }
 
+// Tests that you can delete multiple read items in the Reading List without
+// creating a crash (crbug.com/701956).
+- (void)testDeleteMultipleItems {
+  // Add entries.
+  ReadingListModel* model = GetReadingListModel();
+  for (int i = 0; i < 11; i++) {
+    std::string increment = std::to_string(i);
+    model->AddEntry(GURL(kReadURL + increment),
+                    std::string(kReadTitle + increment),
+                    reading_list::ADDED_VIA_CURRENT_APP);
+    model->SetReadStatus(GURL(kReadURL + increment), true);
+  }
+
+  // Delete them from the Reading List view.
+  OpenReadingList();
+  [[EarlGrey selectElementWithMatcher:EmptyBackground()]
+      assertWithMatcher:grey_nil()];
+  TapButtonWithID(IDS_IOS_READING_LIST_EDIT_BUTTON);
+  TapButtonWithID(IDS_IOS_READING_LIST_DELETE_ALL_READ_BUTTON);
+
+  // Verify the background string is displayed.
+  [[EarlGrey selectElementWithMatcher:EmptyBackground()]
+      assertWithMatcher:grey_notNil()];
+}
+
 @end