Closed
Bug 465082
Opened 16 years ago
Closed 16 years ago
Add "Forget About This Site" to History sidebar's site container context menu
Categories
(Firefox :: Bookmarks & History, enhancement, P3)
Firefox
Bookmarks & History
Tracking
()
VERIFIED
FIXED
Firefox 3.6a1
People
(Reporter: u60234, Assigned: mkohler)
References
Details
(Keywords: verified1.9.1)
Attachments
(1 file, 2 obsolete files)
(deleted),
patch
|
mkohler
:
review+
beltzner
:
approval1.9.1+
|
Details | Diff | Splinter Review |
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1b2pre) Gecko/20081114 Minefield/3.1b2pre When the History sidebar is set to group by "By Site" or "By Date and Site", the context menu for a site container should have the "Forget About This Site" item. It is a bit unintuitive to have to expand the site container and bring up the context menu for an individual page to delete the complete history for the site. Steps to Reproduce: 1.Open the History sidebar. 2.Click the sidebar's "View" button and select "By Site". 3.Open the context menu for a site container. Actual Results: There is no way to delete all history (cookies, cache, downloads, permissions, etc.) for the site. There is only a "Delete" command that AFAIK only removes the site from the visited site's list. Expected Results: There should be a "Forget About This Site" command that delete all history related to the site.
Comment 1•16 years ago
|
||
you can delete the host container that should make exactly what you're asking for...
Comment 3•16 years ago
|
||
Would be nice to get early in 3.2.
Priority: -- → P3
Target Milestone: --- → Firefox 3.2a1
Updated•16 years ago
|
Flags: in-litmus?
Assignee | ||
Comment 4•16 years ago
|
||
Comment 5•16 years ago
|
||
Comment on attachment 375589 [details] [diff] [review] Patch v1 >diff --git a/browser/components/places/content/controller.js b/browser/components/places/content/controller.js >--- a/browser/components/places/content/controller.js >+++ b/browser/components/places/content/controller.js >@@ -232,19 +232,27 @@ PlacesController.prototype = { > this.copy(); > break; > case "cmd_paste": > this.paste(); > break; > case "cmd_delete": > this.remove("Remove Selection"); > break; >- case "placesCmd_deleteDataHost": >- let uri = PlacesUtils._uri(this._view.selectedNode.uri); >- PlacesUIUtils.privateBrowsing.removeDataFromDomain(uri.host); >+ case "placesCmd_deleteDataHost": nit: trailing spaces here >+ if(this._view.selectedNode.uri.match(/place/)) { you should use PlacesUtils.nodeIsHost >+ var uriSubstring = "" + this._view.selectedNode.uri.match(/domain=.+&/); >+ var uriComplete = "http://" + uriSubstring.slice(uriSubstring.indexOf("=")+1, uriSubstring.indexOf("&")); >+ let uri = PlacesUtils._uri(uriComplete); >+ PlacesUIUtils.privateBrowsing.removeDataFromDomain(uri.host); Every query node has a getQueries method, so you should get the first query in the queries array, and use query.domain >+ } >+ else { >+ let uri = PlacesUtils._uri(this._view.selectedNode.uri); >+ PlacesUIUtils.privateBrowsing.removeDataFromDomain(uri.host); >+ } get an host var, and then call removeDataFromDomain(host) out of the if/else
Attachment #375589 -
Flags: review?(mak77) → review-
Assignee | ||
Comment 6•16 years ago
|
||
includes changes from comment 5.
Attachment #375589 -
Attachment is obsolete: true
Attachment #376048 -
Flags: review?(mak77)
Comment 7•16 years ago
|
||
Comment on attachment 376048 [details] [diff] [review] includes the changes from comment 5 >diff --git a/browser/components/places/content/controller.js b/browser/components/places/content/controller.js >--- a/browser/components/places/content/controller.js >+++ b/browser/components/places/content/controller.js >@@ -233,18 +233,24 @@ PlacesController.prototype = { > break; > case "cmd_paste": > this.paste(); > break; > case "cmd_delete": > this.remove("Remove Selection"); > break; > case "placesCmd_deleteDataHost": >- let uri = PlacesUtils._uri(this._view.selectedNode.uri); >- PlacesUIUtils.privateBrowsing.removeDataFromDomain(uri.host); >+ var host; >+ if(PlacesUtils.nodeIsHost(this._view.selectedNode)) { space between if and ( r=me with those fixed, thanks!
Attachment #376048 -
Flags: review?(mak77) → review+
Assignee | ||
Comment 8•16 years ago
|
||
Added the space after the if.
Attachment #376048 -
Attachment is obsolete: true
Attachment #376258 -
Flags: review+
Assignee | ||
Updated•16 years ago
|
Keywords: checkin-needed
Comment 9•16 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/18243f70a33c can't make 1.9.1 due to l10n changes.
Updated•16 years ago
|
Attachment #376258 -
Flags: approval1.9.1?
Comment 11•16 years ago
|
||
Comment on attachment 376258 [details] [diff] [review] includes the space after if (checkin-ready) this fix is really small and low risk, enables the Forget This site context menu on Site containers.
Comment 12•16 years ago
|
||
Comment on attachment 376258 [details] [diff] [review] includes the space after if (checkin-ready) a191=beltzner and please add a test or flip the in-litmus? flag
Attachment #376258 -
Flags: approval1.9.1? → approval1.9.1+
Updated•16 years ago
|
Keywords: checkin-needed
Comment 13•16 years ago
|
||
http://hg.mozilla.org/releases/mozilla-1.9.1/rev/129dd637477d
Keywords: checkin-needed → fixed1.9.1
Comment 14•16 years ago
|
||
Verified fixed on the trunk using Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.2a1pre) Gecko/20090527 Minefield/3.6a1pre. Verified fixed on the 1.9.1 branch using Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1pre) Gecko/20090527 Shiretoko/3.5pre and Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.1pre) Gecko/20090527 Shiretoko/3.5pre.
Status: RESOLVED → VERIFIED
Keywords: fixed1.9.1 → verified1.9.1
Comment 15•16 years ago
|
||
https://litmus.mozilla.org/show_test.cgi?id=7538 is already in Litmus to cover this case so setting the flag.
Flags: in-litmus? → in-litmus+
Comment 16•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
•