Summary: | Drop [UsePointersEvenForNonNullableObjectArguments] from several Canvas interfaces | ||
---|---|---|---|
Product: | WebKit | Reporter: | Chris Dumez <cdumez> |
Component: | Bindings | Assignee: | Chris Dumez <cdumez> |
Status: | RESOLVED FIXED | ||
Severity: | Normal | CC: | buildbot, commit-queue, darin, dino, esprehn+autocc, graouts, gyuyoung.kim, kondapallykalyan, rniwa, youennf |
Priority: | P2 | ||
Version: | WebKit Nightly Build | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Attachments: |
Description
Chris Dumez
2016-04-19 21:52:24 PDT
Created attachment 276802 [details]
Patch
Comment on attachment 276802 [details] Patch Attachment 276802 [details] did not pass mac-ews (mac): Output: http://webkit-queues.webkit.org/results/1189227 New failing tests: canvas/philip/tests/2d.drawImage.null.html Created attachment 276803 [details]
Archive of layout-test-results from ews103 for mac-yosemite
The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews103 Port: mac-yosemite Platform: Mac OS X 10.10.5
Comment on attachment 276802 [details] Patch Attachment 276802 [details] did not pass ios-sim-ews (ios-simulator-wk2): Output: http://webkit-queues.webkit.org/results/1189238 New failing tests: canvas/philip/tests/2d.drawImage.null.html Created attachment 276804 [details]
Archive of layout-test-results from ews121 for ios-simulator-wk2
The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews121 Port: ios-simulator-wk2 Platform: Mac OS X 10.10.5
Comment on attachment 276802 [details] Patch Attachment 276802 [details] did not pass mac-debug-ews (mac): Output: http://webkit-queues.webkit.org/results/1189249 New failing tests: canvas/philip/tests/2d.drawImage.null.html Created attachment 276806 [details]
Archive of layout-test-results from ews114 for mac-yosemite
The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews114 Port: mac-yosemite Platform: Mac OS X 10.10.5
Comment on attachment 276802 [details] Patch Attachment 276802 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.webkit.org/results/1189369 New failing tests: canvas/philip/tests/2d.drawImage.null.html Created attachment 276807 [details]
Archive of layout-test-results from ews107 for mac-yosemite-wk2
The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews107 Port: mac-yosemite-wk2 Platform: Mac OS X 10.10.5
Created attachment 276827 [details]
Patch
Comment on attachment 276827 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=276827&action=review Looks like this patch has more cases of null raising a different exception now than it was before. Would be nice to cover those in some existing regression test. > Source/WebCore/html/canvas/WebGLDebugShaders.idl:31 > + [StrictTypeChecking] DOMString? getTranslatedShaderSource(WebGLShader? shader); No FIXME? Why is the shader nullable? Comment on attachment 276827 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=276827&action=review >> Source/WebCore/html/canvas/WebGLDebugShaders.idl:31 >> + [StrictTypeChecking] DOMString? getTranslatedShaderSource(WebGLShader? shader); > > No FIXME? Why is the shader nullable? I did not find the corresponding spec but I'll give it another try. Comment on attachment 276827 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=276827&action=review >>> Source/WebCore/html/canvas/WebGLDebugShaders.idl:31 >>> + [StrictTypeChecking] DOMString? getTranslatedShaderSource(WebGLShader? shader); >> >> No FIXME? Why is the shader nullable? > > I did not find the corresponding spec but I'll give it another try. Ok, found it: https://www.khronos.org/registry/webgl/extensions/WEBGL_debug_shaders/ Looks like it should not be nullable indeed. I'll add a FIXME Created attachment 276831 [details]
Patch
Comment on attachment 276831 [details] Patch Clearing flags on attachment: 276831 Committed r199776: <http://trac.webkit.org/changeset/199776> All reviewed patches have been landed. Closing bug. |