Rename an Enum to a slightly better name.

components/offline_pages/background/request_picker_unittest.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 "components/offline_pages/background/request_picker.h"
 
 #include "base/bind.h"
 #include "base/test/test_simple_task_runner.h"
 #include "base/threading/thread_task_runner_handle.h"
 #include "base/time/time.h"
@@ skipping 38 matching lines @@
 
 class RequestNotifierStub : public RequestNotifier {
  public:
   RequestNotifierStub()
       : last_expired_request_(kEmptyRequest), total_expired_requests_(0) {}
 
   void NotifyAdded(const SavePageRequest& request) override {}
   void NotifyChanged(const SavePageRequest& request) override {}
 
   void NotifyCompleted(const SavePageRequest& request,
-                       SavePageStatus status) override {
+                       BackgroundSavePageResult status) override {
     last_expired_request_ = request;
     last_request_expiration_status_ = status;
     total_expired_requests_++;
   }
 
   const SavePageRequest& last_expired_request() {
     return last_expired_request_;
   }
 
-  RequestCoordinator::SavePageStatus last_request_expiration_status() {
+  RequestCoordinator::BackgroundSavePageResult
+  last_request_expiration_status() {
     return last_request_expiration_status_;
   }
 
   int32_t total_expired_requests() { return total_expired_requests_; }
 
  private:
-  SavePageStatus last_request_expiration_status_;
+  BackgroundSavePageResult last_request_expiration_status_;
   SavePageRequest last_expired_request_;
   int32_t total_expired_requests_;
 };
 
 class RequestPickerTest : public testing::Test {
  public:
   RequestPickerTest();
 
   ~RequestPickerTest() override;
 
@@ skipping 234 matching lines @@
   SavePageRequest request2(kRequestId2, kUrl2, kClientId2, expired_time,
                            kUserRequested);
 
   QueueRequestsAndChooseOne(request1, request2);
 
   PumpLoop();
 
   EXPECT_EQ(kRequestId1, last_picked_->request_id());
   EXPECT_FALSE(request_queue_not_picked_called_);
   EXPECT_EQ(kRequestId2, GetNotifier()->last_expired_request().request_id());
-  EXPECT_EQ(RequestNotifier::SavePageStatus::EXPIRED,
+  EXPECT_EQ(RequestNotifier::BackgroundSavePageResult::EXPIRED,
             GetNotifier()->last_request_expiration_status());
   EXPECT_EQ(1, GetNotifier()->total_expired_requests());
 }
 
 TEST_F(RequestPickerTest, ChooseRequestThatHasNotExceededStartLimit) {
   base::Time creation_time1 =
       base::Time::Now() - base::TimeDelta::FromSeconds(1);
   base::Time creation_time2 = base::Time::Now();
   SavePageRequest request1(kRequestId1, kUrl1, kClientId1, creation_time1,
                            kUserRequested);
@@ skipping 22 matching lines @@
   // With default policy settings, we should choose the earlier request.
   // However, we will make the earlier reqeust exceed the limit.
   request1.set_completed_attempt_count(policy_->GetMaxCompletedTries());
 
   QueueRequestsAndChooseOne(request1, request2);
 
   EXPECT_EQ(kRequestId2, last_picked_->request_id());
   EXPECT_FALSE(request_queue_not_picked_called_);
 }
 }  // namespace offline_pages