Closed Bug 934030 Opened 11 years ago Closed 4 years ago

[meta] Top Sites query improvements

Categories

(Firefox for Android Graveyard :: Awesomescreen, defect)

All
Android
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED INCOMPLETE

People

(Reporter: rnewman, Unassigned)

References

Details

(Keywords: meta)

* Clean profile. * Add http://www.amazon.com as a bookmark. * Tap it in Top Sites. * Return to about:home. * Observe how there's a favicon-only bookmark for Amazon, and a top site named Amazon with a thumbnail. We should be aware of redirects and URL changes, and not regard these things as different Top Sites. Right now we have a pretty crummy experience when your desktop sites get synced to your phone -- those top sites end up with no thumbnail, and perhaps not even a favicon, because they're not the site that actually ends up being tracked. This will probably involve improvements to our data representation.
Depends on: 934702
Blocks: 934644
Even more exciting: http://i.imgur.com/kVXH1jE.png
The solution Places uses is to mark some pages in history as hidden: 1111 /** 1112 * Some pages in history are marked "hidden" and thus don't appear by default 1113 * in queries. These include automatic framed visits and redirects. Setting 1114 * this attribute will return all pages, even hidden ones. Does nothing for 1115 * bookmark queries. Defaults to false. 1116 */ 1117 attribute boolean includeHidden; We definitely need that capability to handle JS redirection (though we'd need to be judicious about its application). We can perhaps hackily handle network redirection in the same way, but really a redirect is a property of a visit, not a page. That means a schema change. mfinkle, care to prioritize?
tracking-fennec: --- → ?
We could use the "hidden" feature as a way to exclude sensitive websites from the thumbnails on top-sites too. This seems like a useful project for reducing the "cruft" from the awesomescreen results too. What approaches are we considering? What is the scope of the project? Milestones? I would like to see this move forward.
(In reply to Mark Finkle (:mfinkle) from comment #3) > We could use the "hidden" feature as a way to exclude sensitive websites > from the thumbnails on top-sites too. > > This seems like a useful project for reducing the "cruft" from the > awesomescreen results too. > > What approaches are we considering? What is the scope of the project? > Milestones? There are two distinct bits to this. One is eliminating straight-up redirects: Comment 1. That we can do by observing automatic page redirects, hiding those interstitial pages. This requires a small schema change -- either hiding history entries altogether, or merging in Sync's notion of visits and moving us closer to Places' schema. The other is Comment 0: allowing Fennec to recognize that m.foo.com and www.foo.com, with a redirect (or "Request desktop version") between them, represent a "site", and that we should: 1. Merge the mobile and desktop sites together for the purposes of computing frecency, if we've visited the mobile version at all. 2. Use the mobile thumbnail in the Top Sites view. 3. Share favicons 4. etc. The latter depends on Bug 934702. I'm going to turn this one into a meta bug and re-jig the dependency tree accordingly.
Keywords: meta
Summary: Top Sites should be smarter about redirects → [meta] Top Sites should be smarter about redirects
Depends on: 949104
No longer blocks: 943408
Depends on: 949541
we don't track meta bugs, please nom leaf bugs
tracking-fennec: ? → ---
No longer depends on: 949541
Summary: [meta] Top Sites should be smarter about redirects → [meta] Top Sites should be smarter
Depends on: 1046709
Blocks: 788146
Blocks: top-sites
Depends on: 1265516
No longer depends on: 1046709
No longer blocks: top-sites
Summary: [meta] Top Sites should be smarter → [meta] Top Sites query improvements
No longer blocks: 788146
Depends on: 1254660
Depends on: 1213085
Depends on: 1237038
We have completed our launch of our new Firefox on Android. The development of the new versions use GitHub for issue tracking. If the bug report still reproduces in a current version of [Firefox on Android nightly](https://play.google.com/store/apps/details?id=org.mozilla.fenix) an issue can be reported at the [Fenix GitHub project](https://github.com/mozilla-mobile/fenix/). If you want to discuss your report please use [Mozilla's chat](https://wiki.mozilla.org/Matrix#Connect_to_Matrix) server https://chat.mozilla.org and join the [#fenix](https://chat.mozilla.org/#/room/#fenix:mozilla.org) channel.
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → INCOMPLETE
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.