[NTP] Remove unused resource string and TODOs.

chrome/browser/ntp_snippets/download_suggestions_provider.cc

 // 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.
 
 #include "chrome/browser/ntp_snippets/download_suggestions_provider.h"
 
 #include <algorithm>
 #include <utility>
 
 #include "base/bind.h"
@@ skipping 380 matching lines @@
   // ContentSuggestion move constructor noexcept.
   while (suggestions.size() > kMaxSuggestionsCount)
     suggestions.pop_back();
 
   observer()->OnNewSuggestions(this, provided_category_,
                                std::move(suggestions));
 }
 
 ContentSuggestion DownloadSuggestionsProvider::ConvertOfflinePage(
     const OfflinePageItem& offline_page) const {
-  // TODO(vitaliii): Make sure the URL is actually opened as an offline URL even
-  // when the user is online. See crbug.com/641568.
   ContentSuggestion suggestion(
       ContentSuggestion::ID(provided_category_, GetOfflinePagePerCategoryID(
                                                     offline_page.offline_id)),
       offline_page.url);
 
   if (offline_page.title.empty()) {
     // TODO(vitaliii): Remove this fallback once the OfflinePageModel provides
     // titles for all (relevant) OfflinePageItems.
     suggestion.set_title(base::UTF8ToUTF16(offline_page.url.spec()));
   } else {
@@ skipping 227 matching lines @@
 
 void DownloadSuggestionsProvider::UnregisterDownloadItemObservers() {
   DCHECK_NE(download_manager_, nullptr);
 
   std::vector<DownloadItem*> all_downloads;
   download_manager_->GetAllDownloads(&all_downloads);
 
   for (DownloadItem* item : all_downloads)
     item->RemoveObserver(this);
 }