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 298 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.
+  for (var i = 0; i < checkboxes.length; i++)
+    expectFalse(checkboxes[i].checked);

[Patrick Dubroy, 2013/02/05 20:43:14]

You could also do this with:

  expectEqual(document.querySelectorAll('input[type=checkbox]:checked'), 0)

[Sergiu, 2013/02/06 10:52:34]

Done, expectFalse doesn't actually work as it seems to do some type checking as
well.

+
+  var getNewEvent = function() {

[Patrick Dubroy, 2013/02/05 20:43:14]

Nit: maybe name this getClickEvent?

[Sergiu, 2013/02/06 10:52:34]

Made it getShiftClickEvent to make it clearer.

+    return new MouseEvent('click', { shiftKey: true });
+  };
+
+  // Check the start.
+  $('checkbox-4').dispatchEvent(getNewEvent());
+  // And the end.
+  $('checkbox-9').dispatchEvent(getNewEvent());
+
+  // See if they are checked.
+  for (var i = 0; i < checkboxes.length; i++) {
+    if (i >= 4 && i <= 9)

[Patrick Dubroy, 2013/02/05 20:43:14]

Again, you could use querySelectorAll('input[type=checkbox]:checked') and just
check that the first element is #checkbox-4 and the last one is #checkbox-9. I
think it would be cleaner than looping, but I'll leave it up to you.

[Sergiu, 2013/02/06 10:52:34]

Just for thoroughness I would like to check the id of every checked item, maybe
it will catch some subtle stuff later on. But the idea of looking only at the
checked boxes is good, so I've done that.

+      expectTrue(checkboxes[i].checked);
+    else
+      expectFalse(checkboxes[i].checked);
+  }
+
+  // Extend the selection.
+  $('checkbox-14').dispatchEvent(getNewEvent());
+
+  for (var i = 0; i < checkboxes.length; i++) {

[Patrick Dubroy, 2013/02/05 20:43:14]

Same here.

[Sergiu, 2013/02/06 10:52:34]

Done.

+    if (i >= 4 && i <= 14)
+      expectTrue(checkboxes[i].checked);
+    else
+      expectFalse(checkboxes[i].checked);
+  }
+
+  // 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(getNewEvent());
+
+  for (var i = 0; i < checkboxes.length; i++) {
+    if ((i >= 4 && i <= 14) || (i >= 19 && i <=24))
+      expectTrue(checkboxes[i].checked);
+    else
+      expectFalse(checkboxes[i].checked);
+  }
+

[Patrick Dubroy, 2013/02/05 20:43:14]

One more test I would add:

  $('checkbox-26').click();
  $('checkbox-20').dispatchEvent(getNewEvent());

and test that 19 is still checked, and 20-26 are unchecked.

[Sergiu, 2013/02/06 10:52:34]

Done.

+  testDone();
+});