Adds a delete button to chrome:offline-internals

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 f85d641049a4fd99d235f1c07fea94b3f5e5366b..0b13f4a2e6ce0a4c709987e3e877bcaf585acac9 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
@@ -30,6 +30,12 @@ class OfflineInternalsUIMessageHandler : public content::WebUIMessageHandler {
   // Delete selected list of page ids from the store.
   void HandleDeleteSelectedPages(const base::ListValue* args);
 
+  // Deletes all the requests in the request queue.
+  void HandleDeleteAllRequests(const base::ListValue* args);
+
+  // Delete selected list of requests from the request queue.
+  void HandleDeleteSelectedRequests(const base::ListValue* args);
+
   // Load Request Queue info.
   void HandleGetRequestQueue(const base::ListValue* args);
 
@@ -54,6 +60,11 @@ class OfflineInternalsUIMessageHandler : public content::WebUIMessageHandler {
   // Load whether device is currently offline.
   void HandleGetNetworkStatus(const base::ListValue* args);
 
+  // Callback used by DeleteAllRequests to get the request_ids to delete.
+  void DeleteAllGetCallback(

[chili, 2016/09/12 20:26:02]

nit: this method name is really confusing...  what about...
"HandleGetAllRequestsForDelete"? or "OnGetAllRequestsForDelete"?

[Pete Williamson, 2016/09/12 22:46:36]

How about HandleGetAllRequestsForDeleteCallback.  (I left the word callback in
the function title to help us realized that it is part of an async processing
chain.)

+      std::string callback_id,
+      std::vector<std::unique_ptr<offline_pages::SavePageRequest>> requests);
+
   // Callback for async GetAllPages calls.
   void HandleStoredPagesCallback(
       std::string callback_id,
@@ -65,14 +76,23 @@ class OfflineInternalsUIMessageHandler : public content::WebUIMessageHandler {
       offline_pages::RequestQueue::GetRequestsResult result,
       std::vector<std::unique_ptr<offline_pages::SavePageRequest>> requests);
 
-  // Callback for DeletePage/ClearAll calls.
+  // Callback for DeletePage/DeleteAllPages calls.
   void HandleDeletedPagesCallback(std::string callback_id,
                                   const offline_pages::DeletePageResult result);
 
+  // Callback for DeleteRequest/DeleteAllRequests calls.
+  void HandleDeletedRequestsCallback(
+      std::string callback_id,
+      const offline_pages::RequestQueue::UpdateMultipleRequestResults& results);
+
   // Turns a DeletePageResult enum into logical string.
   std::string GetStringFromDeletePageResult(
       offline_pages::DeletePageResult value);
 
+  // Summarizes the UpdateMultipleRequestResults vector with a string.
+  std::string GetStringFromDeleteRequestResults(
+      const offline_pages::RequestQueue::UpdateMultipleRequestResults& result);
+
   // Turns a SavePageRequest::Status into logical string.
   std::string GetStringFromSavePageStatus();