Summary: | [Soup] CookieJarSoup::deleteCookie() should stop looking for the cookie after it is removed | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Chris Dumez <cdumez> | ||||
Component: | WebCore Misc. | Assignee: | Chris Dumez <cdumez> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | cgarcia, danw, gustavo, laszlo.gombos, mrobinson, rakuco, webkit.review.bot | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Chris Dumez
2013-02-18 04:07:50 PST
Actually, we need to keep iterating so that the cookies get freed. However, we can introduce a boolean to stop comparing cookie names after the cookie was removed. Created attachment 188847 [details]
Patch
Could someone from GTK port please take a look before landing this? LGTM Comment on attachment 188847 [details] Patch Clearing flags on attachment: 188847 Committed r143204: <http://trac.webkit.org/changeset/143204> All reviewed patches have been landed. Closing bug. |