Closed
Bug 439129
Opened 17 years ago
Closed 16 years ago
Clearing browsing history does not hide livemarks children from the location bar
Categories
(Firefox :: Bookmarks & History, defect)
Tracking
()
VERIFIED
FIXED
Firefox 3.6a1
People
(Reporter: 7cascavel, Assigned: mak)
References
Details
(Keywords: verified1.9.1)
Attachments
(2 files, 1 obsolete file)
(deleted),
image/jpeg
|
Details | |
(deleted),
patch
|
dietrich
:
review+
beltzner
:
approval1.9.1+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9) Gecko/2008060906 Iceweasel/3.0 (Debian-3.0~rc2-1)
Build Identifier: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9) Gecko/2008060906 Iceweasel/3.0 (Debian-3.0~rc2-1)
Please see the steps to reproduce.
Reproducible: Always
Steps to Reproduce:
1. Subscribe to a RSS feed using the bookmark toolbar (eg http://www.mininova.org/rss.xml?cat=5)
2. Visit one or more of the pages in this RSS feed
3. Clear all private data in edit/preferences/privacy.
4. Restart firefox
5. Click the down arrow to open the dropdown menu from the URL box.
Actual Results:
The pages you visited from the RSS feed appear in this dropdown menu.
Expected Results:
These pages should not be listed there, since browsing history was deleted.
A grep in .mozilla files shows that information on pages visited throug RSS feeds remain in the file places.sqlite after clearing private data and exiting firefox.
Updated•17 years ago
|
Component: Security → Places
QA Contact: firefox → places
Comment 1•17 years ago
|
||
Usually when you view a new RSS feed from the Bookmarks toolbar, you'll see an icon (if available) in the feed after loading the RSS feed. After clearing all private data, history and browsing history in Firefox, everything is supposed to be deleted, including all the icons that are loaded for the previously-viewed RSS feeds. However in the screenshot, Firefox 3 still showing the icons of the previously-read RSS feeds after clearing all the history and private data. This didn't happen to Firefox 2.
Comment 2•17 years ago
|
||
In a typical Firefox 2, when you open a RSS feed in your bookmarks, an icon will be loaded for the specific RSS feed. But after clearing all the history, those history and cache files in Firefox will be deleted, including all the data and icon in the RSS feeds that you've viewed previously.
However in Firefox 3, after viewing some RSS feeds, clearing all private data (with all the available options selected in Tools > Clear Private Data), the icons for the previously-viewed feeds are still there.
I have also seen something similar. After you read a story from an RSS feed, then clear all your private data, the History (in sidebar or library) is completely clear (good), but the story you read can still be found in the autocomplete drop-down box of the location bar. The story has not been bookmarked, as evidenced by the absence of a "star" in the urlbar drop-down box, so that's not why it's there.
I have come up with a simple, repeatable test scenario below:
. Created clean new "test" profile in Profile Manager
. Click on first story on Latest Headlines RSS in toolbar
. Hit Ctrl-Shift-Del
. Clear the location bar and hit Alt-Down (or type "rss")
. You will see the story you read still there in the drop-down box
. Ctrl-H: History sidebar will be (correctly) empty
More RSS stories you read will continue to accumulate in this way in the location bar drop-down box, but not in the history.
You can stop more stories from being added by unchecking "Keep my history for at least X days" in Options, but this does not clear ones already there.
To get rid of them, you can either:
. Manually delete each story from the drop-down box with the Delete key
. Delete the (entire) RSS feed from the bookmarks toolbar
Assignee | ||
Comment 4•16 years ago
|
||
This is not a bug, RSS feed entries are bookmarks, so it appear in the location bar even if it's unvisited.
But you can hide unvisited bookmarks in the location bar setting places.frecency.unvisitedBookmarkBonus to 0 in about:config
Status: UNCONFIRMED → RESOLVED
Closed: 16 years ago
Resolution: --- → INVALID
Let me just make sure we're talking about the same thing. You seem to refer to *unvisited* RSS feed entries showing up in the location bar, as does your workaround with the pref "unvisitedBookmarkBonus". I think we are saying that only *visited* RSS entries are showing up, which is clearly history-related data and technically a privacy concern. Of course, if your response to this case is also the same, then ok, though I respectfully disagree. I think this is related to the more general issue that frequency data should be clearable with the "Clear Private Date" feature.
Assignee | ||
Comment 6•16 years ago
|
||
the general behaviour is that the RSS feed children are bookmarks and so should be showed in the locationbar in both cases or in no case...
If you are saying that if they have been visited at least once will appear, while if they have never been visited will not appear then you should reopen this report. Not a big privacy concern since the feed is there visible, but still something to take a look at.
Yes, it is just the visited sites appearing, not the unvisited ones.
Though I think it is low-priority, it should be re-opened, as you said, to be thorough. As I am not the bug filer, I do not know if I should change the status - is Jose still reading this? I guess I will if he doesn't, if there are no objections.
While I think it's fine to display previously-read RSS entries in the location bar, I think it is a reasonable expectation that "Clear Private Data" should clear them, instead of letting them accumulate over time (to some limit, I would guess the "Keep my history for at least X days" limit)
I agree it's not a major privacy concern, but I think it will still bother some people to see a portion of their browsing history show up in the urlbar just after they cleared their history, who will then probably file more dupes on it (unless they see this).
Comment 8•16 years ago
|
||
Never realised anyone replied to this bug report till now, lolz. Anyway, I've got to say that Rick's claim is totally true, as I've been suffering this bug from the day I installed Firefox 3.0. The "unvisited" RSS feeds aren't related to this bug.
I emailed Jose (original filer) a few days ago to reopen this bug, but he hasn't replied, so I assume he has stopped tracking this. I'm not really familiar with with bugzilla, but it seems I cannot reopen this bug myself as Marco suggested - can someone else? I don't think I should open a new one.
Assignee | ||
Comment 10•16 years ago
|
||
reopening, needs investigation
Status: RESOLVED → UNCONFIRMED
Resolution: INVALID → ---
Comment 12•16 years ago
|
||
Sorry for the duplication, just wanna' have this issue re-investigated. Just read the comments above and you'll know what the problem is. Perhaps just try it in your Firefox 3, this happens all the time.
Anyway, thanks for reopening the bug report.
Assignee | ||
Comment 13•16 years ago
|
||
actually looking at the code this was done by design:
nsNavHistory::FixInvalidFrecenciesForExcludedPlaces()
{
// for every moz_place that has an invalid frecency (< 0) and
// *is an unvisited child of a livemark feed*, or begins with "place:",
// set frecency to 0 so that it is excluded from url bar autocomplete.
So we need to state why this was done and if we want to change the behaviour.
Mardak, any thought on this?
Keywords: uiwanted
Comment 14•16 years ago
|
||
If you clear your history, there should be no page visit corresponding to that RSS feed bookmark entry, so it should act like an unvisited bookmark. There's ways in Firefox 3.1 to prevent unvisited bookmarks from showing up in the location bar.
Assignee | ||
Comment 15•16 years ago
|
||
imho the problem is that this should be consistent, so show all livemark childs or don't show them at all, what i get from here is that we show a livemark child if it was visited before a clear history, while we don't show it if it was never visited. Also we are showing previously visited items like an unvisited bookmark while the ui is not showing it like a bookmark (no star highlight) so this needs clarification on our side. Not a big problem though.
There is also bug 450502 that could change the behaviour not resetting frecency in clearhistory.
Assignee | ||
Comment 17•16 years ago
|
||
confirming for further thoughts on changing how we reset frecency on clear history.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Assignee | ||
Updated•16 years ago
|
OS: Linux → All
Hardware: PC → All
Updated•16 years ago
|
Summary: Clearing browsing history does not fully clear info for pages viewed through RSS feeds. → Clearing browsing history does not fully clear info for pages viewed through RSS feeds. (i.e. favicons, showing in awesomebar)
Comment 18•16 years ago
|
||
I suggest that you include a "clear ALL data and history including RSS feeds and livemarks" in the "clear recent history" function.
Assignee | ||
Updated•16 years ago
|
Assignee: nobody → mak77
Status: NEW → ASSIGNED
Target Milestone: --- → Firefox 3.1
Comment 19•16 years ago
|
||
I think there should be an additional option relating to this issue, perhaps an option for clearing the history of RSS feeds and live bookmarks, as suggested in the previous comment.
By the way, thanks for realizing and taking action on this bug.
Assignee | ||
Comment 20•16 years ago
|
||
we are going to hide unvisited livemarks children by default, so that should solve all users doubts.
Comment 21•16 years ago
|
||
Is it possible to delete visited livemarks children and RSS feeds data under the current "clear recent history" function after the intended results mention above?
Assignee | ||
Comment 22•16 years ago
|
||
(In reply to comment #21)
> Is it possible to delete visited livemarks children and RSS feeds data under
> the current "clear recent history" function after the intended results mention
> above?
history is always and already cleared, those results appear because they are unvisited bookmarks, so that would not change anything
Assignee | ||
Comment 23•16 years ago
|
||
The issue was quite different from what i expected, the problem is not we are doing the wrong thing, instead the query is completely wrong, we are setting frecencies on livemarks, not on their children :\
As usual, writing the test before changing code helps.
Attachment #353927 -
Flags: review?(dietrich)
Assignee | ||
Comment 24•16 years ago
|
||
fixed minor nit
Attachment #353927 -
Attachment is obsolete: true
Attachment #353929 -
Flags: review?(dietrich)
Attachment #353927 -
Flags: review?(dietrich)
Assignee | ||
Updated•16 years ago
|
Flags: in-testsuite?
Assignee | ||
Updated•16 years ago
|
Summary: Clearing browsing history does not fully clear info for pages viewed through RSS feeds. (i.e. favicons, showing in awesomebar) → Clearing browsing history does not hide livemarks children from the location bar
Comment 25•16 years ago
|
||
Comment on attachment 353929 [details] [diff] [review]
patch v1.1
r=me
Attachment #353929 -
Flags: review?(dietrich) → review+
Assignee | ||
Comment 26•16 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 16 years ago → 16 years ago
Resolution: --- → FIXED
Target Milestone: Firefox 3.1 → Firefox 3.2a1
Assignee | ||
Updated•16 years ago
|
Flags: in-testsuite? → in-testsuite+
Assignee | ||
Updated•16 years ago
|
Attachment #353929 -
Flags: approval1.9.1?
Assignee | ||
Comment 27•16 years ago
|
||
Comment on attachment 353929 [details] [diff] [review]
patch v1.1
asking approval, this is one of the things that users complain about location bar. Remaining items after a clear history should be bookmarks, but visited livemark children were still appearing (and they don't have a star).
Updated•16 years ago
|
Attachment #353929 -
Flags: approval1.9.1? → approval1.9.1+
Comment 28•16 years ago
|
||
Comment on attachment 353929 [details] [diff] [review]
patch v1.1
a191=beltzner
Comment 29•16 years ago
|
||
Keywords: fixed1.9.1
Comment 30•16 years ago
|
||
verified FIXED on Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1b4pre) Gecko/20090331 Shiretoko/3.5b4pre ID:20090331041754
and on trunk on build: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2a1pre) Gecko/20090331 Minefield/3.6a1pre ID:20090331044608
Status: RESOLVED → VERIFIED
Keywords: fixed1.9.1 → verified1.9.1
Comment 31•15 years ago
|
||
Bug 451915 - move Firefox/Places bugs to Firefox/Bookmarks and History. Remove all bugspam from this move by filtering for the string "places-to-b-and-h".
In Thunderbird 3.0b, you do that as follows:
Tools | Message Filters
Make sure the correct account is selected. Click "New"
Conditions: Body contains places-to-b-and-h
Change the action to "Delete Message".
Select "Manually Run" from the dropdown at the top.
Click OK.
Select the filter in the list, make sure "Inbox" is selected at the bottom, and click "Run Now". This should delete all the bugspam. You can then delete the filter.
Gerv
Component: Places → Bookmarks & History
QA Contact: places → bookmarks
You need to log in
before you can comment on or make changes to this bug.
Description
•