Summary: | [WK2][WKTR] TestRunner needs to implement clearApplicationCacheForOrigin | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Chris Dumez <cdumez> | ||||
Component: | WebKit2 | Assignee: | Chris Dumez <cdumez> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | gyuyoung.kim, kenneth, webkit.review.bot | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Chris Dumez
2012-09-11 04:05:53 PDT
Created attachment 163328 [details]
Patch
Comment on attachment 163328 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=163328&action=review > Source/WebKit2/WebProcess/InjectedBundle/API/c/WKBundlePrivate.h:98 > +WK_EXPORT void WKBundleClearApplicationCacheForOrigin(WKBundleRef bundle, WKStringRef origin); > WK_EXPORT void WKBundleSetAppCacheMaximumSize(WKBundleRef bundle, uint64_t size); > WK_EXPORT uint64_t WKBundleGetAppCacheUsageForOrigin(WKBundleRef bundle, WKStringRef origin); Why are these api's inconsistent? AppCache vs ApplicationCache. That should really be fixed (In reply to comment #2) > (From update of attachment 163328 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=163328&action=review > > > Source/WebKit2/WebProcess/InjectedBundle/API/c/WKBundlePrivate.h:98 > > +WK_EXPORT void WKBundleClearApplicationCacheForOrigin(WKBundleRef bundle, WKStringRef origin); > > WK_EXPORT void WKBundleSetAppCacheMaximumSize(WKBundleRef bundle, uint64_t size); > > WK_EXPORT uint64_t WKBundleGetAppCacheUsageForOrigin(WKBundleRef bundle, WKStringRef origin); > > Why are these api's inconsistent? AppCache vs ApplicationCache. That should really be fixed I agree with you but do you want me to do this in the same patch? Nope, another patch and please make sure that users of these API's are informed... ie add Anders Carlson or Sam Weinig (In reply to comment #4) > Nope, another patch and please make sure that users of these API's are informed... ie add Anders Carlson or Sam Weinig Sure. Can I get cq+ for this patch then? Comment on attachment 163328 [details] Patch Clearing flags on attachment: 163328 Committed r128173: <http://trac.webkit.org/changeset/128173> All reviewed patches have been landed. Closing bug. |