Single URL Expires Were Not Being Deleted.

chrome/browser/history/expire_history_backend.cc

Index: chrome/browser/history/expire_history_backend.cc
===================================================================
--- chrome/browser/history/expire_history_backend.cc	(revision 131457)
+++ chrome/browser/history/expire_history_backend.cc	(working copy)
@@ -359,6 +359,8 @@
     // determine if they care whether anything was deleted).
     URLsDeletedDetails* deleted_details = new URLsDeletedDetails;
     deleted_details->all_history = false;
+    deleted_details->rows = dependencies->deleted_urls;

[sky, 2012/04/12 20:54:10]

Why do we need both? If we're providing rows can't we nuked dleeted_urls since
rows includes the url?

+    // Build a set of the URLs for those of our clients who prefer only URLs.
     for (size_t i = 0; i < dependencies->deleted_urls.size(); i++)
       deleted_details->urls.insert(dependencies->deleted_urls[i].url());
     delegate_->BroadcastNotifications(