Remove the GetStatus() call from SavePageRequest.

chrome/browser/ui/webui/offline_internals_ui.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/ui/webui/offline_internals_ui.h"
 
 #include <stdint.h>
 #include <stdlib.h>
 #include <vector>
 
@@ skipping 74 matching lines @@
 
   // Callback for DeletePage/ClearAll calls.
   void HandleDeletedPagesCallback(std::string callback_id,
                                   const offline_pages::DeletePageResult result);
 
   // Turns a DeletePageResult enum into logical string.
   std::string GetStringFromDeletePageResult(
       offline_pages::DeletePageResult value);
 
   // Turns a SavePageRequest::Status into logical string.
-  std::string GetStringFromSavePageStatus(
-      offline_pages::SavePageRequest::Status status);
+  std::string GetStringFromSavePageStatus();
 
   // Offline page model to call methods on.
   offline_pages::OfflinePageModel* offline_page_model_;
 
   // Request coordinator for background offline actions.
   offline_pages::RequestCoordinator* request_coordinator_;
 
   // Factory for creating references in callbacks.
   base::WeakPtrFactory<OfflineInternalsUIMessageHandler> weak_ptr_factory_;
 
@@ skipping 20 matching lines @@
       return "Device failure";
     case offline_pages::DeletePageResult::NOT_FOUND:
       return "Not found";
     case offline_pages::DeletePageResult::RESULT_COUNT:
       break;
   }
   NOTREACHED();
   return "Unknown";
 }
 
-std::string OfflineInternalsUIMessageHandler::GetStringFromSavePageStatus(
-    offline_pages::SavePageRequest::Status status) {
-  switch (status) {
-    case offline_pages::SavePageRequest::Status::NOT_READY:
-      return "Not ready";
-    case offline_pages::SavePageRequest::Status::PENDING:
-      return "Pending";
-    case offline_pages::SavePageRequest::Status::STARTED:
-      return "Started";
-    case offline_pages::SavePageRequest::Status::FAILED:
-      return "Failed";
-    case offline_pages::SavePageRequest::Status::EXPIRED:
-      return "Expired";
-  }
-  NOTREACHED();
-  return "Unknown";
+std::string OfflineInternalsUIMessageHandler::GetStringFromSavePageStatus() {
+  return "Available";
 }
 
 void OfflineInternalsUIMessageHandler::HandleDeleteAllPages(
     const base::ListValue* args) {
   std::string callback_id;
   CHECK(args->GetString(0, &callback_id));
 
   // Pass back success because ClearAll doesn't return a status.
   offline_page_model_->ClearAll(
       base::Bind(&OfflineInternalsUIMessageHandler::HandleDeletedPagesCallback,
@@ skipping 61 matching lines @@
   base::ListValue save_page_requests;
   if (result == offline_pages::RequestQueue::GetRequestsResult::SUCCESS) {
     for (const auto& request : requests) {
       base::DictionaryValue* save_page_request = new base::DictionaryValue();
       save_page_requests.Append(save_page_request);
       save_page_request->SetString("onlineUrl", request.url().spec());
       save_page_request->SetDouble("creationTime",
                                    request.creation_time().ToJsTime());
       save_page_request->SetString(
           "status",
-          GetStringFromSavePageStatus(request.GetStatus(base::Time::Now())));
+          GetStringFromSavePageStatus());
       save_page_request->SetString("namespace", request.client_id().name_space);
       save_page_request->SetDouble("lastAttempt",
                                    request.last_attempt_time().ToJsTime());
       save_page_request->SetString("id", std::to_string(request.request_id()));
     }
   }
   ResolveJavascriptCallback(base::StringValue(callback_id), save_page_requests);
 }
 
 void OfflineInternalsUIMessageHandler::HandleGetRequestQueue(
@@ skipping 174 matching lines @@
   html_source->AddResourcePath("offline_internals_browser_proxy.js",
                                IDR_OFFLINE_INTERNALS_BROWSER_PROXY_JS);
   html_source->SetDefaultResource(IDR_OFFLINE_INTERNALS_HTML);
 
   content::WebUIDataSource::Add(Profile::FromWebUI(web_ui), html_source);
 
   web_ui->AddMessageHandler(new OfflineInternalsUIMessageHandler());
 }
 
 OfflineInternalsUI::~OfflineInternalsUI() {}