From ba5c41212f31b80ecfd87d1e6ae3f7c4d378746d Mon Sep 17 00:00:00 2001
From: John Reck <jreck@google.com>
Date: Wed, 22 Dec 2010 15:42:34 -0800
Subject: [PATCH] Fix testGetAllVisitedUrls to match BP1 behavior

 Bug: 3306953
 Previously with BrowserProvider1 all bookmarks were considered
 visited URLs. Updated Browser.getAllVisitedUrls to match that behavior
 when using BrowserProvider2

Change-Id: I15e6e4e91af4e7d1b21860aabf4ce5a5c6a9d81e
---
 core/java/android/provider/Browser.java | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/core/java/android/provider/Browser.java b/core/java/android/provider/Browser.java
index 14d648558de0b..3bfd005d2058b 100644
--- a/core/java/android/provider/Browser.java
+++ b/core/java/android/provider/Browser.java
@@ -26,6 +26,7 @@ import android.database.DatabaseUtils;
 import android.graphics.BitmapFactory;
 import android.net.Uri;
 import android.provider.BrowserContract.Bookmarks;
+import android.provider.BrowserContract.Combined;
 import android.provider.BrowserContract.History;
 import android.provider.BrowserContract.Searches;
 import android.util.Log;
@@ -244,8 +245,9 @@ public class Browser {
      */
     public static final Cursor getAllVisitedUrls(ContentResolver cr) throws
             IllegalStateException {
-        return cr.query(History.CONTENT_URI,
-                new String[] { History.URL }, null, null, null);
+        return cr.query(Combined.CONTENT_URI,
+                new String[] { Combined.URL }, null, null,
+                Combined.DATE_CREATED + " ASC");
     }
 
     private static final void addOrUrlEquals(StringBuilder sb) {