Bug 139639 - cssom/non-subpixel-scroll-top-left-values.html is very slow on Yosemite in debug
Summary: cssom/non-subpixel-scroll-top-left-values.html is very slow on Yosemite in debug
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: CSS (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-12-14 22:35 PST by Alexey Proskuryakov
Modified: 2014-12-15 13:54 PST (History)
4 users (show)

See Also:

Attachments
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 Alexey Proskuryakov 2014-12-14 22:35:02 PST 
cssom/non-subpixel-scroll-top-left-values.html takes 35 seconds on Yosemite WK1 debug tester, and then it times out. Strangely, there is no data from other bots, which is supposed to mean that the test runs very quickly (in under 1 second). Not sure if I believe that.

Locally, I see it take 29 seconds when run separately, so it would take a lot more with other tests and hyper-threading.
Comment 1 Alexey Proskuryakov 2014-12-14 22:39:37 PST 
Marked as slow in r177273.
Comment 2 Simon Fraser (smfr) 2014-12-15 13:42:14 PST 
It makes 225 divs. Not sure why it would be this slow.
Comment 3 zalan 2014-12-15 13:47:34 PST 
(In reply to comment #0)
> cssom/non-subpixel-scroll-top-left-values.html takes 35 seconds on Yosemite
> WK1 debug tester, and then it times out. Strangely, there is no data from
> other bots, which is supposed to mean that the test runs very quickly (in
> under 1 second). Not sure if I believe that.
> 
> Locally, I see it take 29 seconds when run separately, so it would take a
> lot more with other tests and hyper-threading.

Locally, it takes about 2 seconds when run separately. I don't see why it should take any longer than that.
Comment 4 Simon Fraser (smfr) 2014-12-15 13:49:59 PST 
It's very slow in debug because of lots of time in assertion-only code (all the ASSERT(willBeComposited == needsToBeComposited(layer))).
Comment 5 Simon Fraser (smfr) 2014-12-15 13:51:31 PST 
Lots of time under FrameView::scrollableAreaSetChanged too, even in release, which we should fix.
Comment 6 Simon Fraser (smfr) 2014-12-15 13:54:14 PST 
Filed bug 139651.