RESOLVED FIXED Bug 33880
[Chromium] Need a way of getting a WebApplicationCacheHost ptr out of the WebKitAPI. 
https://bugs.webkit.org/show_bug.cgi?id=33880
Summary [Chromium] Need a way of getting a WebApplicationCacheHost ptr out of the Web...
Michael Nordman
Reported 2010-01-19 21:39:16 PST
Some resource loads are performed without going thru WebCore::ResourceHandle in Chrome. To associate those resource loads with the appropriate appcache, we need to get a ptr to the WebApplicationCacheHost given a WebFrame/WebDataSource.
Attachments
WebDataSource::applicationCacheHost() getter (4.73 KB, patch)
2010-01-19 21:45 PST, Michael Nordman 		no flags
DetailsFormatted DiffDiff
fix style errors (4.73 KB, patch)
2010-01-19 22:00 PST, Michael Nordman 		no flags
DetailsFormatted DiffDiff
new .h file (23.00 KB, patch)
2010-01-20 13:04 PST, Michael Nordman 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Michael Nordman
Comment 1 2010-01-19 21:45:29 PST
Created attachment 46975 [details] WebDataSource::applicationCacheHost() getter
WebKit Review Bot
Comment 2 2010-01-19 21:50:47 PST
Attachment 46975 [details] did not pass style-queue: Failed to run "WebKitTools/Scripts/check-webkit-style" exit_code: 1 WebKit/chromium/src/WebDataSourceImpl.cpp:112: Place brace on its own line for function definitions. [whitespace/braces] [4] WebKit/chromium/src/ApplicationCacheHost.cpp:70: Place brace on its own line for function definitions. [whitespace/braces] [4] WebKit/chromium/src/ApplicationCacheHost.cpp:265: Place brace on its own line for function definitions. [whitespace/braces] [4] Total errors found: 3 If any of these errors are false positives, please file a bug against check-webkit-style.
Michael Nordman
Comment 3 2010-01-19 22:00:11 PST
Created attachment 46976 [details] fix style errors
Darin Fisher (:fishd, Google)
Comment 4 2010-01-19 23:35:07 PST
Comment on attachment 46976 [details] fix style errors > Index: WebKit/chromium/public/WebApplicationCacheHost.h ... > +#if WEBKIT_IMPLEMENTATION > + static WebApplicationCacheHost* fromApplicationCacheHost(WebCore::ApplicationCacheHost*); > +#endif hmm... how about creating src/ApplicationCacheHostInternal.h, and then just expose toWebApplicationCacheHost from there? That way we won't need any implementation details added to the public header (albeit in a WEBKIT_IMPLEMENTATION section). otherwise, this lgtm.
Michael Nordman
Comment 5 2010-01-20 00:19:40 PST
> hmm... how about creating src/ApplicationCacheHostInternal.h, and then > just expose toWebApplicationCacheHost from there? That way we won't > need any implementation details added to the public header (albeit > in a WEBKIT_IMPLEMENTATION section). that sounds good > otherwise, this lgtm.
Michael Nordman
Comment 6 2010-01-20 13:04:52 PST
Created attachment 47054 [details] new .h file
Michael Nordman
Comment 7 2010-01-21 11:53:45 PST
ping
WebKit Commit Bot
Comment 8 2010-01-21 20:21:27 PST
Comment on attachment 47054 [details] new .h file Clearing flags on attachment: 47054 Committed r53671: <http://trac.webkit.org/changeset/53671>
WebKit Commit Bot
Comment 9 2010-01-21 20:21:33 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.