Set up a backup schedule request if we have deferred requests.

chrome/browser/ui/webui/offline/offline_internals_ui_message_handler.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 "chrome/browser/ui/webui/offline/offline_internals_ui_message_handler.h"
 
 #include <stdint.h>
 #include <stdlib.h>
 #include <vector>
 
@@ skipping 270 matching lines @@
     // and AsyncNamespace in their ClientId.
     std::ostringstream id_stream;
     id_stream << base::GenerateGUID();
 
     ResolveJavascriptCallback(
         *callback_id,
         base::FundamentalValue(request_coordinator_->SavePageLater(
                 GURL(url), offline_pages::ClientId(
                                offline_pages::kAsyncNamespace, id_stream.str()),
                 true, offline_pages::RequestCoordinator::RequestAvailability::
-                          ENABLED_FOR_OFFLINER) > 0));
+                          ENABLED_FOR_OFFLINER) != 0));

[Dmitry Titov, 2016/10/10 17:29:45]

I think ">" is even "stronger" condition here, why is it important to check for
"!="?

[Pete Williamson, 2016/10/10 19:45:17]

SavePageLater now returns a request_id.  The request id is an integer instead of
a uint, a randomly generated request id could be negative.

If desired, in another changelist, we could change it to uint everywhere, but
that will be a big change in both offline_page_model and request_coordinator, so
we should do it separately.

[Pete Williamson, 2016/10/10 22:53:12]

Done.  Ah, I didn't realize the random number generator promised to give us
positive numbers.

   } else {
     ResolveJavascriptCallback(*callback_id, base::FundamentalValue(false));
   }
 }
 
 void OfflineInternalsUIMessageHandler::RegisterMessages() {
   web_ui()->RegisterMessageCallback(
       "deleteSelectedPages",
       base::Bind(&OfflineInternalsUIMessageHandler::HandleDeleteSelectedPages,
                  weak_ptr_factory_.GetWeakPtr()));
@@ skipping 37 matching lines @@
 
   // Get the offline page model associated with this web ui.
   Profile* profile = Profile::FromWebUI(web_ui());
   offline_page_model_ =
       offline_pages::OfflinePageModelFactory::GetForBrowserContext(profile);
   request_coordinator_ =
       offline_pages::RequestCoordinatorFactory::GetForBrowserContext(profile);
 }
 
 }  // namespace offline_internals