Eliminate QueryOptions::most_recent_visit_only. All non-unittest consumers o...

chrome/browser/history/text_database.cc

 // 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.
 
 #include <limits>
 #include <set>
 
 #include "chrome/browser/history/text_database.h"
 
 #include "app/sql/connection.h"
@@ skipping 308 matching lines @@
   statement.BindInt64(1, effective_begin_time);
   statement.BindInt64(2, effective_end_time);
   statement.BindInt(3, effective_max_count);
 
   while (statement.Step()) {
     // TODO(brettw) allow canceling the query in the middle.
     // if (canceled_or_something)
     //   break;
 
     GURL url(statement.ColumnString(0));
-    if (options.most_recent_visit_only) {
-      URLSet::const_iterator found_url = found_urls->find(url);
-      if (found_url != found_urls->end())
-        continue;  // Don't add this duplicate when unique URLs are requested.
-    }
+    URLSet::const_iterator found_url = found_urls->find(url);
+    if (found_url != found_urls->end())
+      continue;  // Don't add this duplicate.
 
     // Fill the results into the vector (avoid copying the URL with Swap()).
     results->resize(results->size() + 1);
     Match& match = results->at(results->size() - 1);
     match.url.Swap(&url);
 
     match.title = UTF8ToWide(statement.ColumnString(1));
     match.time = base::Time::FromInternalValue(statement.ColumnInt64(2));
 
     // Extract any matches in the title.
@@ skipping 23 matching lines @@
   } else {
     // Since we got the results in order, we know the last item is the last
     // time we considered.
     *first_time_searched = results->back().time;
   }
 
   statement.Reset();
 }
 
 }  // namespace history