Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(225)

Unified Diff: chrome/browser/ui/webui/offline/offline_internals_ui_message_handler.cc

Issue 2400393002: Set up a backup schedule request if we have deferred requests. (Closed)
Patch Set: Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/webui/offline/offline_internals_ui_message_handler.cc
diff --git a/chrome/browser/ui/webui/offline/offline_internals_ui_message_handler.cc b/chrome/browser/ui/webui/offline/offline_internals_ui_message_handler.cc
index 4e05ea2ec2549b46d013f1122c56753959902d8c..dd7bd0faf52f3f44ea4d2b38e749acae39cde643 100644
--- a/chrome/browser/ui/webui/offline/offline_internals_ui_message_handler.cc
+++ b/chrome/browser/ui/webui/offline/offline_internals_ui_message_handler.cc
@@ -288,7 +288,7 @@ void OfflineInternalsUIMessageHandler::HandleAddToRequestQueue(
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
Pete Williamson 2016/10/10 19:45:17 SavePageLater now returns a request_id. The reque
Pete Williamson 2016/10/10 22:53:12 Done. Ah, I didn't realize the random number gene
} else {
ResolveJavascriptCallback(*callback_id, base::FundamentalValue(false));
}

Powered by Google App Engine
This is Rietveld 408576698