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

components/offline_pages/core/background/save_page_request.cc

Index: components/offline_pages/core/background/save_page_request.cc
diff --git a/components/offline_pages/core/background/save_page_request.cc b/components/offline_pages/core/background/save_page_request.cc
index 6e3b36ab52ebca1d4b04ffe3d4d282f8ce3ff28b..02019dec5238914ed3bc00383b5476d2a2ab109b 100644
--- a/components/offline_pages/core/background/save_page_request.cc
+++ b/components/offline_pages/core/background/save_page_request.cc
@@ -82,7 +82,8 @@ void SavePageRequest::MarkAttemptAborted() {
   // 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() {