Fix for crbug.com/675863 - don't clobber PAUSE state.

components/offline_pages/core/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/core/background/save_page_request.h"
 
 namespace offline_pages {
 
 SavePageRequest::SavePageRequest(int64_t request_id,
                                  const GURL& url,
@@ skipping 64 matching lines @@
 void SavePageRequest::MarkAttemptCompleted() {
   ++completed_attempt_count_;
   state_ = RequestState::AVAILABLE;
 }
 
 void SavePageRequest::MarkAttemptAborted() {
   DCHECK_GT(started_attempt_count_, 0);
   // We intentinally do not increment the completed_attempt_count_, since this
   // was killed before it completed, so we could use the phone or browser for
   // other things.
-  state_ = RequestState::AVAILABLE;
+  if (state_ == RequestState::OFFLINING)

[dougarnett, 2016/12/21 19:25:21]

Please update the method comment in .h to be accurate too (to mention state
change if OFFLINING and not lie about other stuff).

[Pete Williamson, 2016/12/21 19:30:55]

Done.

+    state_ = RequestState::AVAILABLE;
 }
 
 void SavePageRequest::MarkAttemptPaused() {
   state_ = RequestState::PAUSED;
 }
 
 }  // namespace offline_pages