History: Fix selecting multiple visits.

chrome/test/data/webui/history_browsertest.js

 // Copyright (c) 2012 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.
 
 /** @const */ var TOTAL_RESULT_COUNT = 160;
 /** @const */ var WAIT_TIMEOUT = 200;
 
 /**
  * Create a fake history result with the given timestamp.
  * @param {Number} timestamp Timestamp of the entry, in ms since the epoch.
@@ skipping 32 matching lines @@
  * should be used for all calls from backend stubs to the frontend.
  */
 function callFrontendAsync(functionName) {
   var args = Array.prototype.slice.call(arguments, 1);
   setTimeout(function() {
     window[functionName].apply(window, args);
   }, 1);
 }
 
 /**
+ * Checks that all the checkboxes in the [|start|, |end|] interval are checked
+ * and that their IDs are properly set. Does that against the checkboxes in
+ * |checked|, starting from the |startInChecked| position.
+ * @param {Array} checked An array of all the relevant checked checkboxes
+ *    on this page.
+ * @param {Number} start The starting checkbox id.
+ * @param {Number} end The ending checkbox id.
+ * @param {Number} startInChecked The first position in the |checked| array
+ *    to check.
+ */
+function checkInterval(checked, start, end, startInChecked) {
+  for (var i = start; i <= end; i++)
+    expectEquals('checkbox-' + i, checked[i - start + startInChecked].id);
+}
+
+/**
  * Base fixture for History WebUI testing.
  * @extends {testing.Test}
  * @constructor
  */
 function BaseHistoryWebUITest() {}
 
 BaseHistoryWebUITest.prototype = {
   __proto__: testing.Test.prototype,
 
   /**
@@ skipping 246 matching lines @@
     var nextEntry = document.querySelectorAll('.title a')[3];
     removeButton.click();
     waitForCallback('historyResult', function() {
       // The next entry after the deleted ones should now be the first.
       expectEquals(document.querySelector('.title a').textContent,
                    nextEntry.textContent);
       testDone();
     });
   });
 });
+
+/**
+ * Test selecting multiple entries using shift click.
+ */
+TEST_F('HistoryWebUITest', 'multipleSelect', function() {
+  var checkboxes = document.querySelectorAll(
+      '#results-display input[type=checkbox]');
+
+  // Make sure that nothing is checked.
+  expectEquals(0,
+      document.querySelectorAll(
+          '#results-display input[type=checkbox]:checked').length);

[Patrick Dubroy, 2013/02/06 18:53:07]

All of the duplicated querySelectorAll stuff clutters things up. I'd pull it out
into a separate function like getCheckedCheckboxes() or something. It would make
it a lot easier to see the actual test logic.

[Sergiu, 2013/02/07 10:18:18]

Done.

+
+  var getShiftClickEvent = function() {
+    return new MouseEvent('click', { shiftKey: true });
+  };
+
+  // Check the start.
+  $('checkbox-4').dispatchEvent(getShiftClickEvent());

[Patrick Dubroy, 2013/02/06 18:53:07]

Maybe factor the dispatchEvent out too? Something like:

  function shiftClick(el) {
    el.dispatchEvent(new MouseEvent('click', { shiftKey: true });
  }

[Sergiu, 2013/02/07 10:18:18]

Done.

+  // And the end.
+  $('checkbox-9').dispatchEvent(getShiftClickEvent());
+
+  // See if they are checked.
+  var checked = document.querySelectorAll(
+      '#results-display input[type=checkbox]:checked');
+  expectEquals(6, checked.length);
+  checkInterval(checked, 4, 9, 0);
+
+  // Extend the selection.
+  $('checkbox-14').dispatchEvent(getShiftClickEvent());
+
+  checked = document.querySelectorAll(
+      '#results-display input[type=checkbox]:checked');
+  expectEquals(11, checked.length);
+  checkInterval(checked, 4, 14, 0);
+
+  // Now do a normal click on a higher ID box and a shift click on a lower ID
+  // one (test the other way around).
+  $('checkbox-24').click();
+  $('checkbox-19').dispatchEvent(getShiftClickEvent());
+
+  checked = document.querySelectorAll(
+      '#results-display input[type=checkbox]:checked');
+  expectEquals(17, checked.length);
+  // First set of checkboxes (11).
+  checkInterval(checked, 4, 14, 0);
+  // Second set (6).
+  checkInterval(checked, 19, 24, 11);

[Patrick Dubroy, 2013/02/06 18:53:07]

It might be more clear if you eliminated the last parameter from all these
calls, and instead used Array.slice(), e.g.:

  checkInterval(checked.slice(11), 19, 24)

[Sergiu, 2013/02/07 10:18:18]

Heh, actually it seems like querySelectorAll returns a NodeList and not an Array
so slice doesn't work there.
But Array.prototype.slice.call(list) transforms it into one so I did that.

+
+  // Test deselection.
+  $('checkbox-26').click();
+  $('checkbox-20').dispatchEvent(getShiftClickEvent());
+
+  checked = document.querySelectorAll(
+      '#results-display input[type=checkbox]:checked');
+  // checkbox-20 to checkbox-24 should be deselected now.
+  expectEquals(12, checked.length);
+  // First set of checkboxes (11).
+  checkInterval(checked, 4, 14, 0);
+  // Only checkbox-19 should still be selected.
+  expectEquals('checkbox-19', checked[11].id);
+
+  testDone();
+});