[NTP:Downloads] Request offline pages even if model is not loaded.

chrome/browser/ntp_snippets/download_suggestions_provider_unittest.cc

Index: chrome/browser/ntp_snippets/download_suggestions_provider_unittest.cc
diff --git a/chrome/browser/ntp_snippets/download_suggestions_provider_unittest.cc b/chrome/browser/ntp_snippets/download_suggestions_provider_unittest.cc
index ad9a326b0069f99546209240932c25956701c732..932098de3b5447c11c99ec04fc6a34310a90d7bb 100644
--- a/chrome/browser/ntp_snippets/download_suggestions_provider_unittest.cc
+++ b/chrome/browser/ntp_snippets/download_suggestions_provider_unittest.cc
@@ -856,7 +856,6 @@ TEST_F(DownloadSuggestionsProviderTest,
   IgnoreOnCategoryStatusChangedToAvailable();
 
   *(offline_pages_model()->mutable_items()) = CreateDummyOfflinePages({1, 2});
-  offline_pages_model()->set_is_loaded(true);

[Marc Treib, 2017/02/10 09:08:11]

I guess this just isn't required anymore; it wouldn't hurt to keep it there?

[vitaliii, 2017/02/10 09:39:56]

No, it wouldn't.
Anyway is_loaded = true by default, so that line had no effect except making
this explicit to the reader. Since we do not use it in our code, I though that
there is no point to notify the reader about it. 
Also dewittj@ was going to make is_loaded private in
https://codereview.chromium.org/2564163002/.

[Marc Treib, 2017/02/10 10:05:02]

Acknowledged.

   EXPECT_CALL(*observer(), OnNewSuggestions(_, downloads_category(),
                                             UnorderedElementsAre(
                                                 HasUrl("http://dummy.com/1"),
@@ -964,7 +963,6 @@ TEST_F(DownloadSuggestionsProviderTest,
   IgnoreOnSuggestionInvalidated();
 
   *(offline_pages_model()->mutable_items()) = CreateDummyOfflinePages({1, 2});
-  offline_pages_model()->set_is_loaded(true);
   EXPECT_CALL(*observer(), OnNewSuggestions(_, downloads_category(),
                                             UnorderedElementsAre(
                                                 HasUrl("http://dummy.com/1"),