Duplicate bookmarks create duplicate shortcuts in the Top Sites Panel

RESOLVED DUPLICATE of bug 1382332

Status

()

Firefox for Android
Awesomescreen
P2
normal
RESOLVED DUPLICATE of bug 1382332
2 years ago
11 months ago

People

(Reporter: Rolcol, Unassigned, Mentored, NeedInfo)

Tracking

({good-first-bug})

Trunk
good-first-bug
Points:
---

Firefox Tracking Flags

(fennec+, firefox50 affected, firefox51 affected, firefox52 wontfix, firefox53 affected, firefox54 affected)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(4 attachments)

(Reporter)

Description

2 years ago
Created attachment 8814639 [details]
Screenshot of duplicate ars technica tiles

When there are multiple bookmarks to the same site, Firefox for Android will create duplicate shortcut tiles in the Top Sites panel of the home screen.  Attached is a screenshot of Aurora v52 running on a Nexus 7 (2012).  

The expected behavior is not to have duplicate tiles to the same site.

As you can see, there are 3 tiles pointing to Ars Technica.  I had 3 bookmarks pointing to it: 1 in Bookmarks Menu, 1 in Other Bookmarks, and 1 in Mobile Bookmarks.  They all pointed to the same URL of http://arstechnica.com/ 

When I deleted all but the Bookmarks Menu bookmark in the desktop bookmark manager, forced the 2 devices to Sync Now, and swiped away Aurora from the multitasking overview, the duplicate tiles went away.

Devices:
Firefox Beta v51.0b3 on macOS 10.12.1 
Firefox Aurora v52.0a2 updated daily on Nexus 7 (2012) running a custom ROM.  Android v6.0.1
Blocks: 1311099
Created attachment 8816965 [details]
Screenshot_20161205-222205.jpg

Thank you for report!
Tested with Asus ZenPad 8 (Android 6.0.1) and the issue is reproducible on all branches. I will change the status to New and set the flags accordingly.
Status: UNCONFIRMED → NEW
status-firefox50: --- → affected
status-firefox51: --- → affected
status-firefox52: --- → affected
status-firefox53: --- → affected
Ever confirmed: true
Version: 52 Branch → Trunk

Updated

a year ago
status-firefox54: --- → affected
Too late for firefox 52, mass-wontfix.
status-firefox52: affected → wontfix
Created attachment 8860302 [details]
Screenshot_2017_04_21_Top_Sites.png_Photos.png

This is the behavior of this bug in Firefox 54 Beta 1. And Switch to tab is shown for one URL (https://people-mozilla.org/~fqueze2/webrtc/).
tracking-fennec: --- → ?
[triage]
P2, and maybe even P3 if we will have clear timeline for activity stream landing in the near future.
tracking-fennec: ? → +
Keywords: good-first-bug
Priority: -- → P2
We might just need to deduplicate URLs when querying the frecency based topsites from the combined table, i.e. at:
https://dxr.mozilla.org/mozilla-central/source/mobile/android/base/java/org/mozilla/gecko/db/BrowserProvider.java#1071

Note: the same URL could conceivably be pinned multiple times - that means we can't deduplicate later in getTopSites(), we need to do it as early as possible, i.e. when fetching the pure topsites (before pinned/suggested sites are added).

IIRC we use the same query for AS, so this is probably independent of AS being enabled.
Mentor: ahunt

Comment 6

a year ago
Hi, may I please work on this bug?

Updated

a year ago
Flags: needinfo?(andrzej)
Comment hidden (mozreview-request)

Comment 8

a year ago
If I can work on it, what URLs are considered duplicates? Are they considered duplicates when they are exact copies or a subpages. For example, there can be facebook.com and facebook.com/home in the top sites. In this case, should they both be included? If not, which one should be displayed?
No longer blocks: 1311099
I will be fixing this as part of bug 1382332.

I'm sorry no one got back to you, Varun and that now the issue will be fixed. If you would like to have another bug to work on, please add a needinfo flag for me and respond to this message.
Status: NEW → RESOLVED
Last Resolved: 11 months ago
Resolution: --- → DUPLICATE
Duplicate of bug: 1382332
You need to log in before you can comment on or make changes to this bug.