New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.
Starred by 2 users

Issue metadata

Status: Verified
Owner:
Closed: Mar 2010
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 3
Type: Bug
M-5

Restricted
  • Only users with EditIssue permission may comment.



Sign in to add a comment
link

Issue 33320: Cookie Manager: On removal of last cookie auto select the previous one.

Reported by ismail@chromium.org, Jan 27 2010 Project Member

Issue description

Platform:
  Hostname: testings-macbook-pro-15.local
  Mac OS X Version 10.6.2 (Build 10C540)
  Processor: 2 Intel 2.33 GHz
  RAM: 2048 MB

Chrome:
  Chrome version: 4.0.305.0 r36956  <<<Release>>>
  QuickTime Player: <unknown>
  QuickTime PlayerX: 90.3.1
  Flash Player: 10.0.42.34

Precondition:  Have few cookies stored on your computer.

1. Go Chrome-->Preferences-->Under the hood->"Show cookies and website permissions"
2. Select the last cookie element from the list of entries, and perform "Remove"

Actual: Last cookie gets deleted and "Remove" button goes disabled. User has to auto select manually the above cookie element in the list.

Expected: Auto select the previous cookie when the last cookie in the list got removed. ( Windows behavior )


Also applies to :
==========

1. Expand a domain in the list, select the "Cookies" folder and perform "Remove"
Expected: It should auto select the domain name. ( Windows behavior )
Actual: "Remove" button goes disabled.

1. Expand a domain in the list, From the "Cookies" folder, select the last child and perform "Remove"
Expected: It should auto select the "Cookies" folder ( Windows behavior )
Actual: "Remove" button goes disabled.
 

Comment 1 by krisr@chromium.org, Jan 28 2010

Comment 2 by mal@google.com, Jan 29 2010

Labels: Mstone-5 ReleaseBlock-Stable
Status: Assigned

Comment 3 by lafo...@chromium.org, Feb 17 2010

Labels: -Area-Feature Area-UI

Comment 4 by mikesmith@chromium.org, Mar 16 2010

Labels: -ReleaseBlock-Stable ReleaseBlock-NextBeta HelpWanted

Comment 5 by mikesmith@chromium.org, Mar 23 2010

Labels: -Pri-2 -ReleaseBlock-NextBeta Pri-3 ReleaseBlock-Beta

Comment 6 by rsesek@chromium.org, Mar 30 2010

Status: Started

Comment 7 by bugdro...@gmail.com, Mar 31 2010

The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=43157 

------------------------------------------------------------------------
r43157 | rsesek@chromium.org | 2010-03-30 18:02:17 -0700 (Tue, 30 Mar 2010) | 9 lines
Changed paths:
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/cocoa/cookies_window_controller.h?r1=43157&r2=43156
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/cocoa/cookies_window_controller.mm?r1=43157&r2=43156
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/cocoa/cookies_window_controller_unittest.mm?r1=43157&r2=43156

[Mac] Select the previous node when the last cookie in the list is deleted.

Also recursively bubble up the delete if removing a child node will leave the
parent empty. Empty parents look awkward.

BUG= 33320 
TEST=See bug report. Partially covered by unit tests.

Review URL: http://codereview.chromium.org/1598001
------------------------------------------------------------------------

Comment 8 by rsesek@chromium.org, Mar 31 2010

Status: Fixed

Comment 9 by srikanthk@chromium.org, Apr 7 2010

Status: Verified
Google Chrome	5.0.369.1 (Official Build 43598) dev
WebKit	533.4
V8	2.2.0.3
User Agent	Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10_6_3; en-US) AppleWebKit/533.4 (KHTML, like Gecko) 
Chrome/5.0.369.1 Safari/533.4
Command Line	 /Applications/Google Chrome.app/Contents/MacOS/Google Chrome --enable-internal-flash

Comment 10 by bugdroid1@chromium.org, Oct 12 2012

Project Member
Labels: Restrict-AddIssueComment-Commit
This issue has been closed for some time. No one will pay attention to new comments.
If you are seeing this bug or have new data, please click New Issue to start a new bug.

Comment 11 by bugdroid1@chromium.org, Mar 10 2013

Project Member
Labels: -Mstone-5 -Area-UI M-5 Cr-UI

Comment 12 by bugdroid1@chromium.org, Mar 13 2013

Project Member
Labels: -Restrict-AddIssueComment-Commit Restrict-AddIssueComment-EditIssue

Sign in to add a comment