Bug 134448

Summary: Reduce memory required for js/typedarray-zero-size.js
Product: WebKit Reporter: Michael Saboff <msaboff>
Component: Tools / TestsAssignee: Michael Saboff <msaboff>
Status: RESOLVED FIXED    
Severity: Normal CC: bunhere, cdumez, commit-queue, gyuyoung.kim, sergio
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
darin: review+
Patch with loop count of 4000 mhahnenberg: review+

Description Michael Saboff 2014-06-30 10:11:34 PDT
LayoutTests/js/script-tests/typedarray-zero-size.js uses ~190MB which can cause havoc on devices with small amounts of memory.  This is especially true when running JSC stress tests where multiple instances of the test run concurrently.  The test can be modified to still test for the original bug fixed in <http://trac.webkit.org/changeset/158583>, while using less memory.
Comment 1 Michael Saboff 2014-06-30 10:14:26 PDT
Created attachment 234079 [details]
Patch

Verified that this catches the original issue by backing out the code change in <http://trac.webkit.org/changeset/158583>
Comment 2 Michael Saboff 2014-06-30 10:37:39 PDT
Committed r170586: <http://trac.webkit.org/changeset/170586>
Comment 3 Michael Saboff 2014-06-30 11:19:00 PDT
The first patch fixed the memory use issue on most devices.  Still fails on an older iPad.  Going to reduce the loop count.
Comment 4 Michael Saboff 2014-06-30 11:20:23 PDT
Created attachment 234082 [details]
Patch with loop count of 4000

With a loop count of 4000, we still catch the original issue and it works on all the smaller memory devices I tested.
Comment 5 Mark Hahnenberg 2014-06-30 11:21:50 PDT
Comment on attachment 234082 [details]
Patch with loop count of 4000

r=me
Comment 6 Michael Saboff 2014-06-30 11:22:54 PDT
Committed r170591: <http://trac.webkit.org/changeset/170591>