Request Picker task

chrome/browser/ui/webui/offline/offline_internals_ui_message_handler.h

Index: chrome/browser/ui/webui/offline/offline_internals_ui_message_handler.h
diff --git a/chrome/browser/ui/webui/offline/offline_internals_ui_message_handler.h b/chrome/browser/ui/webui/offline/offline_internals_ui_message_handler.h
index 168e28f5a739005057fab6a7fbfe9e288cfc2e31..107419d79cc774664f501d378288fda91327852c 100644
--- a/chrome/browser/ui/webui/offline/offline_internals_ui_message_handler.h
+++ b/chrome/browser/ui/webui/offline/offline_internals_ui_message_handler.h
@@ -8,6 +8,7 @@
 #include "base/memory/weak_ptr.h"
 #include "base/values.h"
 #include "components/offline_pages/background/request_coordinator.h"
+#include "components/offline_pages/background/request_queue_results.h"
 #include "components/offline_pages/background/save_page_request.h"
 #include "components/offline_pages/offline_page_model.h"
 #include "components/offline_pages/offline_store_types.h"
@@ -63,7 +64,7 @@ class OfflineInternalsUIMessageHandler : public content::WebUIMessageHandler {
   // Callback for async GetRequests calls.
   void HandleRequestQueueCallback(
       std::string callback_id,
-      offline_pages::RequestQueue::GetRequestsResult result,

[Evan Stade, 2016/11/09 00:43:03]

wait a second, enum class lets you forward declare this.

namespace offline_pages {
enum class GetRequestsResult;
}

at the top of this file instead of the header include.

[Pete Williamson, 2016/11/09 00:55:29]

Done.

+      offline_pages::GetRequestsResult result,
       std::vector<std::unique_ptr<offline_pages::SavePageRequest>> requests);
 
   // Callback for DeletePage/DeleteAllPages calls.