[webkit-reviews] review granted: [Bug 130133] [EFL][WK2] Add an API to set process model : [Attachment 227643] updated documentation

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Mar 24 16:34:55 PDT 2014


Gyuyoung Kim <gyuyoung.kim at samsung.com> has granted Michal Pakula vel Rutka
<m.pakula at samsung.com>'s request for review:
Bug 130133: [EFL][WK2] Add an API to set process model
https://bugs.webkit.org/show_bug.cgi?id=130133

Attachment 227643: updated documentation
https://bugs.webkit.org/attachment.cgi?id=227643&action=review

------- Additional Comments from Gyuyoung Kim <gyuyoung.kim at samsung.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=227643&action=review


I'm happy now we can select web process model. LGTM except for my trivial
comments. Please fix them before landing.

> Source/WebKit2/UIProcess/API/efl/ewk_context.h:384
> + * web process. When EWK_PROCESS_MODEL_MULTIPLY_SECONDARY is set a

Typo: EWK_PROCESS_MODEL_MULTIPLY_SECONDARY ->
EWK_PROCESS_MODEL_MULTIPLE_SECONDARY.

I wonder whether current our process model only supports network process. Won't
we support database process ?


More information about the webkit-reviews mailing list