Please report any other irregularities here.
If I use PB, all of my bookmarks with live titles get temporarily hosed.
Could you please provide a bit more information on what happens here, and what is expected to happen? I've never used bookmarks with live titles, so this might be obvious... :/
If I'm not mistaken, Firefox stores microsummaries as Places annotations. If that is the case, storing them fails inside the private browsing mode because of bug 509106. Is this what this bug is about?
Depends on: 509106
Bugzilla supports this for saved queries, if you need an example (bookmark, change the dropdown). Since the microsummary relies on the cookie for the saved query to work, it returns useless data if you're in private browsing mode (and your cookie is no longer present). http://mxr.mozilla.org/mozilla-central/source/browser/components/microsummaries/src/nsMicrosummaryService.js#262 calls setItemTitle when it refreshes. We don't protect against that (for good reason) but we should simply cancel the timer (this._timer) on enter and restore it on exit.
Created attachment 398869 [details] [diff] [review] Patch (v1) Does this make sense?
Assignee: nobody → ehsan.akhgari
Status: NEW → ASSIGNED
Attachment #398869 - Flags: review?(mconnor)
Comment on attachment 398869 [details] [diff] [review] Patch (v1) Yep, that works, except I don't know whether we should disable the manual update as well. Thoughts?
(In reply to comment #5) > (From update of attachment 398869 [details] [diff] [review]) > Yep, that works, except I don't know whether we should disable the manual > update as well. Thoughts? Well, based on comment 3 (the fact that we don't send out cookies in PB mode), what's the point of letting the users update them manually?
Because not all sites with microsummaries require cookies (i.e. woot.com). And because I'm not sure how this interacts with allowing users to bookmark in PB.
Comment on attachment 398869 [details] [diff] [review] Patch (v1) Yeah, thought about this but failed to follow up. We should just stop the automatic refreshes, and allow manual refreshing/updating.
Attachment #398869 - Flags: review?(mconnor) → review-
Microsummaries are dead, long live microsummaries.
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.