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

Issue 123403 link

Starred by 2 users

Issue metadata

Status: Verified
Owner:
Email to this user bounced
Closed: Apr 2012
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug-Regression

Blocking:
issue 100885

Restricted
  • Only users with EditIssue permission may comment.



Sign in to add a comment

Regression: Can't delete individual cookies

Reported by vclarke@chromium.org, Apr 13 2012

Issue description

Version: 19.0.1084.24
OS: all

What steps will reproduce the problem?
1. Go to chrome://chrome/settings/cookies
2. Hover over an entry to bring up the 'x' 
3. Click the 'x' to remove the entry

What is the expected output? What do you see instead?
- Expect entry to be removed
- Instead nothing happens. Can still click the "Remove all" button to remove all cookies, but can't delete individual cookies.

Regression from 19.0.1084.15

You are probably looking for build 130891.
CHANGELOG URL:
http://build.chromium.org/f/chromium/perf/dashboard/ui/changelog.html?url=/trunk/src&range=130888:130891
Built at revision:
http://src.chromium.org/viewvc/chrome?view=rev&revision=130891

Most likely candidate: http://src.chromium.org/viewvc/chrome?view=rev&revision=130891

Assigning to author of 130891

 

Comment 1 by laforge@google.com, Apr 13 2012

Labels: -ReleaseBlock-Beta ReleaseBlock-Stable

Comment 2 by nepper@chromium.org, Apr 13 2012

Cc: battre@chromium.org jeffreyc@chromium.org
Adding battre and jeffreyc due to timezone fit.

Dominic, can you have a look?

Comment 3 by jeffreyc@google.com, Apr 13 2012

Cc: jhawkins@chromium.org
Labels: Feature-Preferences
hey James, is a regression in the WebUI UberPage code?

Comment 4 by battre@chromium.org, Apr 14 2012

The problem is in deleteItemAtIndex in chrome/browser/resources/options2/cookies_list.js: this.data_ is always empty. I haven't figured out a proper fix, yet.

Comment 5 by battre@chromium.org, Apr 14 2012

Blocking: 100885
Cc: khorimoto@chromium.org
CCing Kyle since he was working on the cookies list recently.
Project Member

Comment 7 by bugdroid1@chromium.org, Apr 16 2012

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

------------------------------------------------------------------------
r132375 | serya@google.com | Sun Apr 15 23:27:20 PDT 2012

Changed paths:
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/resources/options2/cookies_list.js?r1=132375&r2=132374&pathrev=132375

Fix cookies list.

BUG= 123403 
TEST=Manual test.

Review URL: https://chromiumcodereview.appspot.com/10096002
------------------------------------------------------------------------

Comment 8 by serya@chromium.org, Apr 16 2012

Labels: Merge-Requested

Comment 9 by laforge@google.com, Apr 16 2012

Labels: -Merge-Requested Merge-Approved
Project Member

Comment 10 by bugdroid1@chromium.org, Apr 16 2012

Labels: merge-merged-1084
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=132402

------------------------------------------------------------------------
r132402 | laforge@chromium.org | Mon Apr 16 07:41:56 PDT 2012

Changed paths:
 M http://src.chromium.org/viewvc/chrome/branches/1084/src/chrome/browser/resources/options2/cookies_list.js?r1=132402&r2=132401&pathrev=132402

Merge 132375 - Fix cookies list.

BUG= 123403 
TEST=Manual test.

Review URL: https://chromiumcodereview.appspot.com/10096002

TBR=serya@google.com
Review URL: https://chromiumcodereview.appspot.com/10096013
------------------------------------------------------------------------

Comment 11 by serya@chromium.org, Apr 17 2012

Status: Fixed
Status: Verified
Verify fixed in 20.0.1105.0 and 19.0.1084.28

Comment 13 by cdn@chromium.org, Jun 27 2012

Blocking: -100885 chromium:100885 chromium:100885

Comment 14 by k...@google.com, Aug 8 2012

Labels: -Merge-Approved
Removing Merge-Approved from past milestones.
Project Member

Comment 15 by bugdroid1@chromium.org, Oct 13 2012

Blocking: -chromium:100885
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 16 by laforge@google.com, Oct 17 2012

Labels: -Feature-Preferences Feature-Settings MovedToSettings
Project Member

Comment 17 by bugdroid1@chromium.org, Mar 9 2013

Labels: -Type-Regression -Area-Internals -Mstone-19 -Feature-Privacy -Internals-Network -Feature-Settings Cr-Privacy Cr-UI-Settings Type-Bug-Regression M-19 Cr-Internals Cr-Internals-Network
Project Member

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

Labels: -Restrict-AddIssueComment-Commit Restrict-AddIssueComment-EditIssue

Comment 19 by laforge@google.com, Jul 24 2013

Cc: -jeffreyc@chromium.org

Sign in to add a comment