[Offline pages] Clean up unnecessary DeleteAll calls in offline internals

chrome/browser/resources/offline_pages/offline_internals.js

 // 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.
 
 cr.define('offlineInternals', function() {
   'use strict';
 
   /** @type {!Array<OfflinePage>} */
   var offlinePages = [];
 
@@ skipping 110 matching lines @@
     browserProxy_.getNetworkStatus().then(function(networkStatus) {
       $('current-status').textContent = networkStatus;
     });
     refreshLog();
   }
 
   /**
    * Delete all pages in the offline store.
    */
   function deleteAllPages() {
-    browserProxy_.deleteAllPages().then(pagesDeleted);
+    var checkboxes = document.getElementsByName('stored');
+    var selectedIds = [];
+
+    for (var i = 0; i < checkboxes.length; i++) {
+        selectedIds.push(checkboxes[i].value);

[Bernhard Bauer, 2016/09/15 08:36:28]

Indent two spaces.

[chili, 2016/09/15 18:18:26]

Done.

+    }
+
+    browserProxy_.deleteSelectedPages(selectedIds).then(pagesDeleted);
   }
 
   /**
    * Delete all pending SavePageRequest items in the request queue.
    */
   function deleteAllRequests() {
-    browserProxy_.deleteAllRequests().then(requestsDeleted);
+    var checkboxes = document.getElementsByName('requests');
+    var selectedIds = [];
+
+    for (var i = 0; i < checkboxes.length; i++) {
+        selectedIds.push(checkboxes[i].value);
+    }
+
+    browserProxy_.deleteSelectedRequests(selectedIds).then(requestsDeleted);
   }
 
   /**
    * Callback when pages are deleted.
    * @param {string} status The status of the request.
    */
   function pagesDeleted(status) {
     $('page-actions-info').textContent = status;
     browserProxy_.getStoredPages().then(fillStoredPages);
   }
@@ skipping 89 matching lines @@
     $('clear-all').disabled = incognito;
     $('clear-selected').disabled = incognito;
     $('delete-all-requests').disabled = incognito;
     $('delete-selected-requests').disabled = incognito;
     $('log-model-on').disabled = incognito;
     $('log-model-off').disabled = incognito;
     $('log-request-on').disabled = incognito;
     $('log-request-off').disabled = incognito;
     $('refresh').disabled = incognito;
 
-    $('clear-all').onclick = deleteAllPages;
-    $('clear-selected').onclick = deleteSelectedPages;
+    $('delete-all-pages').onclick = deleteAllPages;
+    $('delete-selected-pages').onclick = deleteSelectedPages;
     $('delete-all-requests').onclick = deleteAllRequests;
     $('delete-selected-requests').onclick = deleteSelectedRequests;
     $('refresh').onclick = refreshAll;
     $('download').onclick = download;
     $('log-model-on').onclick = togglePageModelLog.bind(this, true);
     $('log-model-off').onclick = togglePageModelLog.bind(this, false);
     $('log-request-on').onclick = toggleRequestQueueLog.bind(this, true);
     $('log-request-off').onclick = toggleRequestQueueLog.bind(this, false);
     $('refresh-logs').onclick = refreshLog;
     $('add-to-queue').onclick = function() {
@@ skipping 22 matching lines @@
       refreshAll();
   }
 
   // Return an object with all of the exports.
   return {
     initialize: initialize,
   };
 });
 
 document.addEventListener('DOMContentLoaded', offlineInternals.initialize);