Bug 57787

Summary: Get rid of WebContext::process() in more places
Product: WebKit Reporter: Anders Carlsson <andersca>
Component: New BugsAssignee: Anders Carlsson <andersca>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, aroben, eric, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Other   
OS: OS X 10.5   
Attachments:
Description Flags
Patch mitz: review+

Description Anders Carlsson 2011-04-04 14:04:13 PDT
Get rid of WebContext::process() in more places
Comment 1 Anders Carlsson 2011-04-04 14:07:30 PDT
Created attachment 88127 [details]
Patch
Comment 2 Adam Roben (:aroben) 2011-04-04 14:15:46 PDT
View in context: https://bugs.webkit.org/attachment.cgi?id=88127&action=review

> Source/WebKit2/UIProcess/WebApplicationCacheManagerProxy.cpp:72
> +    // FIXME (Multi-process): The application cache shouldn't be stored in the web process.

Maybe this should say where it *should* be stored (if you know).

> Source/WebKit2/UIProcess/WebApplicationCacheManagerProxy.cpp:89
> +    // FIXME (Multi-process): The application cache shouldn't be stored in the web process.

Ditto.

> Source/WebKit2/UIProcess/WebApplicationCacheManagerProxy.cpp:95
> +    // FIXME (Multi-process): The application cache shouldn't be stored in the web process.

Tritto.

> Source/WebKit2/UIProcess/WebCookieManagerProxy.cpp:79
> +    // FIXME (Multi-process): Cookies shouldn't be stored in the web process.

You know.

> Source/WebKit2/UIProcess/WebCookieManagerProxy.cpp:148
> +    // FIXME (Multi-process): Cookies shouldn't be stored in the web process.

I'm done now.
Comment 3 Anders Carlsson 2011-04-04 14:17:45 PDT
Committed r82873: <http://trac.webkit.org/changeset/82873>
Comment 4 WebKit Review Bot 2011-04-04 15:29:31 PDT
http://trac.webkit.org/changeset/82873 might have broken GTK Linux 32-bit Release