Bug 134554

Summary: fix webgl crash
Product: WebKit Reporter: Alex Christensen <achristensen>
Component: WebGLAssignee: Alex Christensen <achristensen>
Severity: Normal CC: buildbot, bunhere, cdumez, commit-queue, dino, gyuyoung.kim, kondapallykalyan, noam, rniwa, roger_fong, sergio
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
URL: https://www.khronos.org/registry/webgl/sdk/tests/conformance/glsl/bugs/long-expressions-should-not-crash.html
Description Flags
dino: review+, buildbot: commit-queue-
Archive of layout-test-results from webkit-ews-16 for mac-mountainlion-wk2 none

Description Alex Christensen 2014-07-02 13:22:16 PDT
A khronos test crashes without this change.
Comment 1 Alex Christensen 2014-07-02 13:24:35 PDT
Created attachment 234274 [details]
Comment 2 Dean Jackson 2014-07-02 13:27:08 PDT
Comment on attachment 234274 [details]

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

> Source/WebCore/ChangeLog:7
> +
> +        Reviewed by NOBODY (OOPS!).

Mention the khronos test that exercises this.
Comment 3 Build Bot 2014-07-02 14:33:32 PDT
Comment on attachment 234274 [details]

Attachment 234274 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.appspot.com/results/5273111880531968

New failing tests:
Comment 4 Build Bot 2014-07-02 14:33:36 PDT
Created attachment 234279 [details]
Archive of layout-test-results from webkit-ews-16 for mac-mountainlion-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: webkit-ews-16  Port: mac-mountainlion-wk2  Platform: Mac OS X 10.8.5
Comment 5 Alex Christensen 2014-07-02 14:34:50 PDT
This was committed to http://trac.webkit.org/changeset/170718
I'm pretty sure this did not cause that failure.