1190648 - Add ability to clear cookies domain-wide in dev tools

Status: RESOLVED DUPLICATE of bug 1231434

(DevTools :: Storage Inspector, defect)

Description

[Alex Bell]

Attachment: clear-cookies-by-domain-in-console.png

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.10; rv:41.0) Gecko/20100101 Firefox/41.0
Build ID: 20150803004002

Steps to reproduce:

Chrome offers the ability to clear all the cookies for a domain through a contextual menu (right-click) on that domain in the "Resources" tab. See attached screenshot. This is really handy and useful. Targeting individual cookies for deletion during testing is currently tedious in FF.

Comment 1

[Hubert B Manilla]

Attachment: 1190648wip.patch

I made a push at working on this. There are couple of things to do.
Just need to make sure i'm on the right path
I need some help understanding with actor would be used to clear the items.

Things done
 - Added support for context menus to the TreeWidget
 - Created a clear menu item on the context menu for the Storage Tree

Where i need help
 - Confirm that i'm on the right path

 - How to send a message to server actor to clear the objects for the host?

Cheers

Comment 2

[Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe]]

Just letting you know that we have not forgotten about this. I have been planning how we should approach adding context menus to the tree and table widgets in general.

Our approach should be the same for both widgets and should be completely generic.

We should allow each of the widgets to accept an object such as the following:

    [
      {
        item: "Remove cookie"
        message: "storage:storage-cookie-request-parent",
        methodName: "clearCookie"
      },
      {
        item: "Remove all cookies"
        message: "storage:storage-cookie-request-parent",
        methodName: "clearAllCookies"
      },
    ]

From here we can build the menu and send the appropriate message back to the content process and have it act appropriately.

Anyhow, I will hold off on reviewing this until the plan is more concrete.

Comment 3

[Hubert B Manilla]

Cool thanks!

Comment 4

[Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe]]

This was recently fixed as part of bug 1231434