Bug 100176 - Disable auto termination of WebKit2 processes on mac
Summary: Disable auto termination of WebKit2 processes on mac
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit2 (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Kiran Muppala
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2012-10-23 18:03 PDT by Kiran Muppala
Modified: 2012-10-24 16:56 PDT (History)
4 users (show)

See Also:

Attachments
Patch (4.40 KB, patch)
2012-10-23 18:49 PDT, Kiran Muppala 		no flags Details | Formatted Diff | Diff
Patch (4.42 KB, patch)
2012-10-24 13:24 PDT, Kiran Muppala 		no flags Details | Formatted Diff | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Kiran Muppala 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.
Comment 1 Kiran Muppala 2012-10-23 18:03:24 PDT 
<rdar://problem/12561597>
Comment 2 Kiran Muppala 2012-10-23 18:49:58 PDT 
Created attachment 170288 [details]
Patch
Comment 3 Alexey Proskuryakov 2012-10-23 19:52:52 PDT 
This won't make launchd restart the service after termination, will it?
Comment 4 Kiran Muppala 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.
Comment 5 Mark Rowe (bdash) 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.
Comment 6 Kiran Muppala 2012-10-24 13:24:04 PDT 
Created attachment 170458 [details]
Patch
Comment 7 Kiran Muppala 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.
Comment 8 Alexey Proskuryakov 2012-10-24 16:49:58 PDT 
Comment on attachment 170458 [details]
Patch

OK.
Comment 9 WebKit Review Bot 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>
Comment 10 WebKit Review Bot 2012-10-24 16:56:04 PDT 
All reviewed patches have been landed.  Closing bug.