Added that deleting a cookie should not just set it to an empty string but...

Added that deleting a cookie should not just set it to an empty string but also instantly expire it #1118 [todd@robotcoop.com]

git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@1180 5ecf4fe2-1ee6-0310-87b1-e25e094e27de

actionpack/CHANGELOG

 *SVN*
 
+* Added that deleting a cookie should not just set it to an empty string but also instantly expire it #1118 [todd@robotcoop.com]
+
 * Added AssetTagHelper#image_path, AssetTagHelper#javascript_path, and AssetTagHelper#stylesheet_path #1110 [Larry Halff]
 
 * Fixed url_for(nil) in functional tests #1116 [Alisdair McDiarmid]

actionpack/lib/action_controller/cookies.rb

@@ -54,9 +54,10 @@ def []=(name, options)
       set_cookie(options)
     end
     
-    # Removes the cookie on the client machine by setting the value to an empty string.
+    # Removes the cookie on the client machine by setting the value to an empty string
+    # and setting its expiration date into the past
     def delete(name)
-      set_cookie("name" => name.to_s, "value" => "")
+      set_cookie("name" => name.to_s, "value" => "", "expires" => Time.at(0))
     end
 
     private