[Reading List iOS] Collapse whitespace on item title.

ios/chrome/browser/reading_list/reading_list_download_service.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 "ios/chrome/browser/reading_list/reading_list_download_service.h"
 
 #include <utility>
 
 #include "base/bind.h"
 #include "base/files/file_enumerator.h"
 #include "base/files/file_path.h"
 #include "base/files/file_util.h"
 #include "base/memory/ptr_util.h"
 #include "base/metrics/histogram_macros.h"
+#include "base/strings/string_util.h"
 #include "components/reading_list/ios/offline_url_utils.h"
 #include "components/reading_list/ios/reading_list_entry.h"
 #include "components/reading_list/ios/reading_list_model.h"
 #include "ios/chrome/browser/reading_list/reading_list_distiller_page_factory.h"
 #include "ios/web/public/web_thread.h"
 
 namespace {
 // Status of the download when it ends, for UMA report.
 // These match tools/metrics/histograms/histograms.xml.
 enum UMADownloadStatus {
@@ skipping 207 matching lines @@
     const GURL& distilled_url,
     URLDownloader::SuccessState success,
     const base::FilePath& distilled_path,
     const std::string& title) {
   DCHECK(reading_list_model_->loaded());
   if ((success == URLDownloader::DOWNLOAD_SUCCESS ||
        success == URLDownloader::DOWNLOAD_EXISTS) &&
       !distilled_path.empty()) {
     reading_list_model_->SetEntryDistilledInfo(url, distilled_path,
                                                distilled_url);
-    if (!title.empty())
-      reading_list_model_->SetEntryTitle(url, title);
+
+    std::string trimmed_title = base::CollapseWhitespaceASCII(title, false);
+    if (!trimmed_title.empty())
+      reading_list_model_->SetEntryTitle(url, trimmed_title);
 
     const ReadingListEntry* entry = reading_list_model_->GetEntryByURL(url);
     if (entry)
       UMA_HISTOGRAM_COUNTS_100("ReadingList.Download.Failures",
                                entry->FailedDownloadCounter());
     UMA_HISTOGRAM_ENUMERATION("ReadingList.Download.Status", SUCCESS,
                               STATUS_MAX);
 
   } else if (success == URLDownloader::ERROR_RETRY) {
     reading_list_model_->SetEntryDistilledState(url,
@@ skipping 34 matching lines @@
     for (auto& url : url_to_download_cellular_) {
       ScheduleDownloadEntry(url);
     }
   }
   if (type == net::NetworkChangeNotifier::CONNECTION_WIFI) {
     for (auto& url : url_to_download_wifi_) {
       ScheduleDownloadEntry(url);
     }
   }
 }