WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
57787
Get rid of WebContext::process() in more places
https://bugs.webkit.org/show_bug.cgi?id=57787
Summary
Get rid of WebContext::process() in more places
Anders Carlsson
Reported
Monday, April 4, 2011 10:04:13 PM UTC
Get rid of WebContext::process() in more places
Attachments
Patch
(24.26 KB, patch)
2011-04-04 14:07 PDT
,
Anders Carlsson
mitz: review+
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Anders Carlsson
Comment 1
Monday, April 4, 2011 10:07:30 PM UTC
Created
attachment 88127
[details]
Patch
Adam Roben (:aroben)
Comment 2
Monday, April 4, 2011 10:15:46 PM UTC
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.
Anders Carlsson
Comment 3
Monday, April 4, 2011 10:17:45 PM UTC
Committed
r82873
: <
http://trac.webkit.org/changeset/82873
>
WebKit Review Bot
Comment 4
Monday, April 4, 2011 11:29:31 PM UTC
http://trac.webkit.org/changeset/82873
might have broken GTK Linux 32-bit Release
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug