Remove deleted offline page suggestions from opened NTPs

chrome/android/java/src/org/chromium/chrome/browser/ntp/cards/SuggestionsSection.java

Index: chrome/android/java/src/org/chromium/chrome/browser/ntp/cards/SuggestionsSection.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/ntp/cards/SuggestionsSection.java b/chrome/android/java/src/org/chromium/chrome/browser/ntp/cards/SuggestionsSection.java
index edd97047d4c9470b32f2f29600d2ee0daeb9b40c..dc493d830fca267b45dd73a938e08e3673acd554 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/ntp/cards/SuggestionsSection.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/ntp/cards/SuggestionsSection.java
@@ -61,6 +61,15 @@ public class SuggestionsSection implements ItemGroup {
         mSuggestions.remove(suggestion);
     }
 
+    public void removeSuggestionById(String suggestionId) {

[Michael van Ouwerkerk, 2016/08/15 11:06:09]

This method is called |removeSuggestion| and the one above |dismissSuggestion|.
Is there a semantic difference between |remove| and |dismiss|? If not, maybe we
can use the same term for both.

[Philipp Keck, 2016/08/16 10:54:40]

Right. They should both be "removeSuggestion", as they don't actually perform
the dismissing anymore, that's done in the C++ backend. Renamed it and
"removeSuggestionById" now calls "removeSuggestion".

+        for (SnippetArticleListItem suggestion : mSuggestions) {
+            if (suggestion.mId.equals(suggestionId)) {
+                mSuggestions.remove(suggestion);
+                return;
+            }
+        }
+    }
+
     public boolean hasSuggestions() {
         return !mSuggestions.isEmpty();
     }