Bug 1382332: Remove duplicate URLs from AS top sites. r=grisha
MozReview-Commit-ID: E9dP9IitXQQ
--- a/mobile/android/base/java/org/mozilla/gecko/db/BrowserProvider.java
+++ b/mobile/android/base/java/org/mozilla/gecko/db/BrowserProvider.java
@@ -1213,16 +1213,18 @@ public class BrowserProvider extends Sha
PageMetadata.JSON + " AS " + TopSites.PAGE_METADATA_JSON +
" FROM (" + selectTopSites + ") AS " + TABLE_TOPSITES +
" LEFT OUTER JOIN " + TABLE_PAGE_METADATA + " ON " +
DBUtils.qualifyColumn(TABLE_TOPSITES, Combined.HISTORY_GUID) + " = " +
DBUtils.qualifyColumn(TABLE_PAGE_METADATA, PageMetadata.HISTORY_GUID) +
+ " GROUP BY " + DBUtils.qualifyColumn(TABLE_TOPSITES, Bookmarks.URL) + // remove duplicates.
+
// In case position is non-unique (as in Activity Stream pins, whose position
// is always zero), we need to ensure we get stable ordering.
" ORDER BY " + Bookmarks.POSITION + ", " + Bookmarks.URL,
null);
c.setNotificationUri(getContext().getContentResolver(),
BrowserContract.AUTHORITY_URI);