Comment #1 on issue 26606 by [email protected]: void  
TreeView::TreeNodesRemoved doesn't work with hidden root
http://code.google.com/p/chromium/issues/detail?id=26606

Scott -

To reproduce, apply patch from http://codereview.chromium.org/365005  
(apologies,
still cleaning it up)

Go to show cookies (options -> under the hood -> show cookies), and try  
deleting any
of the top level entries (e.g. select abc.com, and hit the delete key or  
the remove
button).

Entry point in the code is in cookies_view.cc:

void CookiesTreeView::RemoveSelectedItems() {
   TreeModelNode* selected_node = GetSelectedNode();
   if (selected_node) {
     CookieTreeNode* ct_node =  
static_cast<CookieTreeNode*>(GetSelectedNode());
     ct_node->DeleteStoredObjects();
     // find the parent and index
     CookieTreeNode* parent_node = ct_node->GetParent();
     int ct_node_index = parent_node->IndexOfChild(ct_node);
     delete cookies_model_->Remove(parent_node, ct_node_index);
   }
}



--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---

Reply via email to