[webkit-reviews] review granted: [Bug 187108] Add configuration for automatic process pre-warming : [Attachment 346239] Patch for landing

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Jul 31 19:03:27 PDT 2018

Ryosuke Niwa <rniwa at webkit.org> has granted Ben Richards
<benton_richards at apple.com>'s request for review:
Bug 187108: Add configuration for automatic process pre-warming

Attachment 346239: Patch for landing


--- Comment #27 from Ryosuke Niwa <rniwa at webkit.org> ---
Comment on attachment 346239
  --> https://bugs.webkit.org/attachment.cgi?id=346239
Patch for landing

View in context: https://bugs.webkit.org/attachment.cgi?id=346239&action=review

> Source/WebKit/ChangeLog:25
> +	   (WebKit::WebProcessPool::setMaximumNumberOfProcesses): Condition
changed so that warmInitialProcess doesn't result in a crash if called before
this function.

warmInitialProcess itself doesn't crash but calling setMaximumNumberOfProcesses

> Tools/TestWebKitAPI/Tests/WebKitCocoa/SetMaximumPrewarmedProcessCount.mm:2
> + * Copyright (C) 2017 Apple Inc. All rights reserved.

This should be 2018.

More information about the webkit-reviews mailing list