[webkit-reviews] review granted: [Bug 208285] Remove initial layout throttle : [Attachment 391837] Patch

[bugzilla-daemon at webkit.org]

Antti Koivisto <koivisto at iki.fi> has granted Ben Nham <nham at apple.com>'s
request for review:
Bug 208285: Remove initial layout throttle
https://bugs.webkit.org/show_bug.cgi?id=208285

Attachment 391837: Patch

https://bugs.webkit.org/attachment.cgi?id=391837&action=review




--- Comment #3 from Antti Koivisto <koivisto at iki.fi> ---
Comment on attachment 391837
  --> https://bugs.webkit.org/attachment.cgi?id=391837
Patch

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

> Source/WebCore/page/SettingsBase.cpp:-67
> -// This amount of time must have elapsed before we will even consider
scheduling a layout without a delay.
> -// FIXME: For faster machines this value can really be lowered to 200. 250
is adequate, but a little high
> -// for dual G5s. :)
> -static const Seconds layoutScheduleThreshold = 250_ms;

My dual G5s are going to fly!