202324 - Cookie Manager: "remove cookie" and "remove site" should disable with un-selection

Status: VERIFIED FIXED

(Core :: Networking: Cookies, defect)

Description

[benc]

Comment 1

[benc]

This is stranger than I originally described.



If you open the cookie manager, and it somehow has selected no cookies, then the
button is grey.

The problem I described happens when you select some cookies and then unselect.

STEPS:
select a cookie, then unselect.
click on "remove cookie" button.
the bottom cookie becomes selected in light grey (not the normal highlight color).
click on "remove cookie" button.
the grey-selected cookie is removed.

Comment 2

[benc]

happens in "Cookie Sites" with "Remove site" as well.

(Sorry, I'm testing and writing bugs as I go).

Comment 3

[[e-mail]]

yeah, more wallet bugs.

recommend wallet bugs for future, unless someone actually wants to work on it
right now ;)

Comment 4

[Darin Fisher]

-> suresh

Comment 5

[Mike Connor [:mconnor]]

Duping another bug about this, happens on Linux and Windows too

Comment 6

[Mike Connor [:mconnor]]

*** Bug 211771 has been marked as a duplicate of this bug. ***

Comment 7

[Mike Connor [:mconnor]]

YACMB, -> me

Comment 8

[Mike Connor [:mconnor]]

Attachment: patch

Comment 9

[Alec Flett]

Comment on attachment 140242 [details] [diff] [review]
patch

great! sr=alecf

Comment 10

[Michiel van Leeuwen (email: mvl+moz@)]

Comment on attachment 140242 [details] [diff] [review]
patch

>+    document.getElementById("removeCookie").setAttribute("disabled","true");

ubernit: add a space after the comma. Same in PermissionSelected.

r=mvl

Comment 11

[Michiel van Leeuwen (email: mvl+moz@)]

patch checked in

Comment 12

[Aakash Desai [:aakashd]]

verified FIXED on build:

Build identifier: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.1.2) Gecko/20090729 Firefox/3.5.2