Bug 100176

Summary: Disable auto termination of WebKit2 processes on mac
Product: WebKit Reporter: Kiran Muppala <cmuppala>
Component: WebKit2Assignee: Kiran Muppala <cmuppala>
Status: RESOLVED FIXED    
Severity: Normal CC: ap, mrowe, sam, webkit.review.bot
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch none

Kiran Muppala
Reported 2012-10-23 18:03:08 PDT
Disable suppression of WebKit2 processes on mac until UI process adds logic to determine when it can be safely enabled.
Attachments
Patch (4.40 KB, patch)
2012-10-23 18:49 PDT, Kiran Muppala 		no flags
DetailsFormatted DiffDiff
Patch (4.42 KB, patch)
2012-10-24 13:24 PDT, Kiran Muppala 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Kiran Muppala
Comment 1 2012-10-23 18:03:24 PDT
<rdar://problem/12561597>
Kiran Muppala
Comment 2 2012-10-23 18:49:58 PDT
Created attachment 170288 [details] Patch
Alexey Proskuryakov
Comment 3 2012-10-23 19:52:52 PDT
This won't make launchd restart the service after termination, will it?
Kiran Muppala
Comment 4 2012-10-23 19:54:45 PDT
(In reply to comment #3) > This won't make launchd restart the service after termination, will it? No, it doesn't interact with launched at all.
Mark Rowe (bdash)
Comment 5 2012-10-23 20:09:11 PDT
As I understand it, sudden termination interacts with launchd's automatic relaunching of the service, but automatic termination does not.
Kiran Muppala
Comment 6 2012-10-24 13:24:04 PDT
Created attachment 170458 [details] Patch
Kiran Muppala
Comment 7 2012-10-24 13:25:08 PDT
(In reply to comment #6) > Created an attachment (id=170458) [details] > Patch Replaced process suppression verbiage with auto termination, since the latter is better understood.
Alexey Proskuryakov
Comment 8 2012-10-24 16:49:58 PDT
Comment on attachment 170458 [details] Patch OK.
WebKit Review Bot
Comment 9 2012-10-24 16:56:01 PDT
Comment on attachment 170458 [details] Patch Clearing flags on attachment: 170458 Committed r132425: <http://trac.webkit.org/changeset/132425>
WebKit Review Bot
Comment 10 2012-10-24 16:56:04 PDT
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.