More detailed implementation of the RequestPicker

components/offline_pages/background/save_page_request.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/save_page_request.h"
 
 namespace offline_pages {
 
 SavePageRequest::SavePageRequest(int64_t request_id,
                                  const GURL& url,
                                  const ClientId& client_id,
-                                 const base::Time& creation_time)
+                                 const base::Time& creation_time,
+                                 const bool was_user_requested)
     : request_id_(request_id),
       url_(url),
       client_id_(client_id),
       creation_time_(creation_time),
       activation_time_(creation_time),
-      attempt_count_(0) {}
+      attempt_count_(0),
+      user_requested_(was_user_requested) {}
 
 SavePageRequest::SavePageRequest(int64_t request_id,
                                  const GURL& url,
                                  const ClientId& client_id,
                                  const base::Time& creation_time,
-                                 const base::Time& activation_time)
+                                 const base::Time& activation_time,
+                                 const bool was_user_requested)
     : request_id_(request_id),
       url_(url),
       client_id_(client_id),
       creation_time_(creation_time),
       activation_time_(activation_time),
-      attempt_count_(0) {}
+      attempt_count_(0),
+  user_requested_(was_user_requested) {}

[fgorski, 2016/07/20 16:23:31]

nit: alignment

[Pete Williamson, 2016/07/20 19:50:44]

Done.

 
 SavePageRequest::SavePageRequest(const SavePageRequest& other)
     : request_id_(other.request_id_),
       url_(other.url_),
       client_id_(other.client_id_),
       creation_time_(other.creation_time_),
       activation_time_(other.activation_time_),
       attempt_count_(other.attempt_count_),
-      last_attempt_time_(other.last_attempt_time_) {}
+      last_attempt_time_(other.last_attempt_time_),
+      user_requested_(other.user_requested_) {}
 
 SavePageRequest::~SavePageRequest() {}
 
 // TODO(fgorski): Introduce policy parameter, once policy is available.
 SavePageRequest::Status SavePageRequest::GetStatus(
     const base::Time& now) const {
   if (now < activation_time_)
     return Status::NOT_READY;
 
   // TODO(fgorski): enable check once policy available.
@@ skipping 17 matching lines @@
   // other cases.
   last_attempt_time_ = start_time;
   ++attempt_count_;
 }
 
 void SavePageRequest::MarkAttemptCompleted() {
   last_attempt_time_ = base::Time();
 }
 
 }  // namespace offline_pages