History: Add search browser test.

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 156 matching lines @@
   }
 };
 
 function queryHistoryImpl(args, beginTime, history) {
   var searchText = args[0];
   var offset = args[1];
   var range = args[2];
   var endTime = args[3] || Number.MAX_VALUE;
   var maxCount = args[4];
 
+  // Search only by title in this stub.

[Patrick Dubroy, 2013/04/03 14:12:33]

nit: I'd move this comment down above the "if history[k].." statement.

[Sergiu, 2013/04/03 17:04:39]

Done.

+  var results = [];
+  if (searchText) {
+    for (var k = 0; k < history.length; k++) {
+      if (history[k].title.indexOf(searchText) != -1)
+        results.push(history[k]);
+    }
+  } else {
+    results = history;
+  }
+
   // Advance past all entries newer than the specified end time.
   var i = 0;
-  while (i < history.length && history[i].time >= endTime)
+  while (i < results.length && results[i].time >= endTime)
     ++i;
 
-  var results = new Array();
   if (beginTime) {
     var j = i;
-    while (j < history.length && history[j].time >= beginTime)
+    while (j < results.length && results[j].time >= beginTime)
       ++j;
 
-    results = history.slice(i, j);
+    results = results.slice(i, j);
   } else {
-    results = history.slice(i);
+    results = results.slice(i);
   }
 
   if (maxCount)
     results = results.slice(0, maxCount);
 
   var queryStartTime = '';
   var queryEndTime = '';
   if (results.length) {
     queryStartTime = results[results.length - 1].dateRelativeDay;
     queryEndTime = results[0].dateRelativeDay;
   } else if (beginTime) {
     queryStartTime = Date(beginTime);
     queryEndTime = Date(endTime);
   }
 
   callFrontendAsync(
       'historyResult',
       {
         term: searchText,
-        finished: (history.length <= i + results.length),
+        finished: (searchText || history.length <= i + results.length),

[Patrick Dubroy, 2013/04/03 14:12:33]

Might as well make this correct too.

[Sergiu, 2013/04/03 17:04:39]

Done.

         queryStartTime: queryStartTime,
         queryEndTime: queryEndTime
       },
       results);
 }
 
 /**
  * Fixture for History WebUI testing which returns some fake history results
  * to the frontend.
  * @extends {BaseHistoryWebUITest}
@@ skipping 273 matching lines @@
   // checkbox-20 to checkbox-24 should be deselected now.
   expectEquals(12, checked.length);
   // First set of checkboxes (11).
   checkInterval(checked, 4, 14);
   // Only checkbox-19 should still be selected.
   expectEquals('checkbox-19', checked[11].id);
 
   testDone();
 });
 
+TEST_F('HistoryWebUITest', 'searchHistory', function() {
+  var getResultCount = function() {
+    return document.querySelectorAll('.entry').length;
+  };
+  // See that all the elements are there.
+  expectEquals(RESULTS_PER_PAGE, getResultCount());
+
+  // See that the search works.
+  $('search-field').value = 'Thu Oct 02 2008';
+  $('search-button').click();
+
+  waitForCallback('historyResult', function() {
+    expectEquals(31, getResultCount());
+
+    // Clear the search.
+    $('search-field').value = '';
+    $('search-button').click();
+    waitForCallback('historyResult', function() {
+      expectEquals(RESULTS_PER_PAGE, getResultCount());
+      testDone();
+    });
+  });
+});
+
 function setPageState(searchText, page, groupByDomain, range, offset) {
   window.location = '#' + PageState.getHashString(
       searchText, page, groupByDomain, range, offset);
 }
 
 function RangeHistoryWebUITest() {}
 
 RangeHistoryWebUITest.prototype = {
   __proto__: HistoryWebUITest.prototype,
 
@@ skipping 149 matching lines @@
 
   waitForCallback('historyResult', function() {
     // See if the correct number of days is shown.
     var resultsDisplay = $('results-display');
     assertEquals(0, resultsDisplay.querySelectorAll('.months-results').length);
     assertEquals(1, resultsDisplay.querySelectorAll('div').length);
 
     testDone();
   });
 });