Don't start a second pre-render if one is in progress.

components/offline_pages/background/request_coordinator.h

Index: components/offline_pages/background/request_coordinator.h
diff --git a/components/offline_pages/background/request_coordinator.h b/components/offline_pages/background/request_coordinator.h
index ffbb8570499257b13d5d6f8b66ab650d85d5e887..9f24ccc4f0c1629850207308aacd58eb511cbf91 100644
--- a/components/offline_pages/background/request_coordinator.h
+++ b/components/offline_pages/background/request_coordinator.h
@@ -73,6 +73,10 @@ class RequestCoordinator : public KeyedService {
     return last_offlining_status_;
   }
 
+  bool is_busy() {
+    return is_busy_;
+  }
+
  private:
   void AddRequestResultCallback(RequestQueue::AddRequestResult result,
                                 const SavePageRequest& request);
@@ -96,6 +100,7 @@ class RequestCoordinator : public KeyedService {
 
   friend class RequestCoordinatorTest;
 
+  bool is_busy_;

[fgorski, 2016/06/21 18:11:18]

please add documentation here. What does is_busy means?

I think is_request_in_progress_ had better chances of being self documenting.
With is_busy you need to put in more work.

[Pete Williamson, 2016/06/21 18:33:56]

Done.

   // RequestCoordinator takes over ownership of the policy
   std::unique_ptr<OfflinerPolicy> policy_;
   // OfflinerFactory.  Used to create offline pages. Owned.