RESOLVED FIXED 105111
[BlackBerry] Prevent CookieManager from blocking the WKT thread 
https://bugs.webkit.org/show_bug.cgi?id=105111
Summary [BlackBerry] Prevent CookieManager from blocking the WKT thread
otcheung
Reported 2012-12-15 14:43:13 PST
Make the databasebackingstore make the getBackingStoreCookies call to prevent blocking on the WKT thread.
Attachments
Patch (4.33 KB, patch)
2012-12-15 14:50 PST, otcheung 		no flags
DetailsFormatted DiffDiff
Patch (3.76 KB, patch)
2012-12-15 14:59 PST, otcheung 		no flags
DetailsFormatted DiffDiff
Patch (8.48 KB, patch)
2012-12-17 12:21 PST, otcheung 		no flags
DetailsFormatted DiffDiff
Patch (8.09 KB, patch)
2012-12-17 13:24 PST, otcheung 		no flags
DetailsFormatted DiffDiff
Show Obsolete (3) View All Add attachment proposed patch, testcase, etc.
otcheung
Comment 1 2012-12-15 14:50:31 PST
Created attachment 179620 [details] Patch
Rob Buis
Comment 2 2012-12-15 14:58:03 PST
Comment on attachment 179620 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=179620&action=review > Source/WebCore/platform/blackberry/CookieManager.h:107 > + void getBackingStoreCookies(); Could be just private AFAICS.
otcheung
Comment 3 2012-12-15 14:59:21 PST
Created attachment 179621 [details] Patch
Rob Buis
Comment 4 2012-12-15 15:04:27 PST
Comment on attachment 179621 [details] Patch LGTM.
WebKit Review Bot
Comment 5 2012-12-15 15:33:15 PST
Comment on attachment 179621 [details] Patch Clearing flags on attachment: 179621 Committed r137814: <http://trac.webkit.org/changeset/137814>
WebKit Review Bot
Comment 6 2012-12-15 15:33:18 PST
All reviewed patches have been landed. Closing bug.
otcheung
Comment 7 2012-12-17 12:21:40 PST
Reopening to attach new patch.
otcheung
Comment 8 2012-12-17 12:21:47 PST
Created attachment 179779 [details] Patch
otcheung
Comment 9 2012-12-17 13:24:39 PST
Created attachment 179790 [details] Patch
Rob Buis
Comment 10 2012-12-17 13:26:18 PST
Comment on attachment 179790 [details] Patch LGTM.
WebKit Review Bot
Comment 11 2012-12-17 13:49:54 PST
Comment on attachment 179790 [details] Patch Clearing flags on attachment: 179790 Committed r137940: <http://trac.webkit.org/changeset/137940>
WebKit Review Bot
Comment 12 2012-12-17 13:49:57 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.