Need to show detail information of cookies.
Created attachment 130286 [details] Patch
Comment on attachment 130286 [details] Patch There's no reason to declare cookieMap outside the for loop, since it's not used again. You also shouldn't call getAllCookies as well as getAllChildCookies, since getAllChildCookies calls getAllCookies already, so this goes through the top-level map twice.
Created attachment 130499 [details] Patch
Comment on attachment 130499 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=130499&action=review > Source/WebCore/platform/blackberry/CookieManager.cpp:231 > + result.append(String("<table style=\"word-wrap:break-word\" cellSpacing=\"0\" cellPadding=\"0\" border=\"1\"><tr><th width=\"54\">Domain</th><th width=\"32\">Path</th><th width=\"57\">Protocol</th><th width=\"40\">Name</th><th width=\"40\">Value</th><th width=\"46\">Secure</th><th width=\"64\">HttpOnly</th><th width=\"49\">Session</th></tr>")); I don't link all the hardcoded numbers. Any other way to make it more intuitive ?
Created attachment 130774 [details] Patch
LGTM
Comment on attachment 130774 [details] Patch Clearing flags on attachment: 130774 Committed r110176: <http://trac.webkit.org/changeset/110176>
All reviewed patches have been landed. Closing bug.