[webkit-reviews] review granted: [Bug 31789] Web Inspector: Introduce sidebar background on timeline panel in order to prevent it from flickering on scroll. : [Attachment 43685] [PATCH] proposed fix

[bugzilla-daemon at webkit.org]

Timothy Hatcher <timothy at hatcher.name> has granted Pavel Feldman
<pfeldman at chromium.org>'s request for review:
Bug 31789: Web Inspector: Introduce sidebar background on timeline panel in
order to prevent it from flickering on scroll.
https://bugs.webkit.org/show_bug.cgi?id=31789

Attachment 43685: [PATCH] proposed fix
https://bugs.webkit.org/attachment.cgi?id=43685&action=review

------- Additional Comments from Timothy Hatcher <timothy at hatcher.name>

> -	       this._refreshTimeout = setTimeout(this._refresh.bind(this),
100);
> +	       this._refreshTimeout = setTimeout(this._refresh.bind(this),
immediate ? 0 : 100);

A 0 delay timeout isn't immediate, it will be 10ms in Safari.

That matter?