Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1006)

Unified Diff: chrome/browser/dom_ui/history_ui.cc

Issue 341087: Eliminate QueryOptions::most_recent_visit_only. All non-unittest consumers o... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 11 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/dom_ui/history_ui.cc
===================================================================
--- chrome/browser/dom_ui/history_ui.cc (revision 30836)
+++ chrome/browser/dom_ui/history_ui.cc (working copy)
@@ -1,4 +1,4 @@
-// Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
+// Copyright (c) 2009 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.
@@ -148,9 +148,6 @@
options.end_time = base::Time::Now().LocalMidnight();
options.end_time -= base::TimeDelta::FromDays(day - 1);
- // As we're querying per-day, we can turn entry repeats off.
- options.most_recent_visit_only = true;
-
// Need to remember the query string for our results.
search_text_ = std::wstring();
@@ -174,10 +171,8 @@
// Set the query ranges for the given month.
history::QueryOptions options = CreateMonthQueryOptions(month);
- // When searching, limit the number of results returned and only show the
- // most recent matches.
+ // When searching, limit the number of results returned.
options.max_count = kMaxSearchResults;
- options.most_recent_visit_only = true;
// Need to remember the query string for our results.
search_text_ = query;
« no previous file with comments | « chrome/browser/autocomplete/history_contents_provider.cc ('k') | chrome/browser/extensions/extension_history_api.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698