Bug 111625 – [V8] Preprocess constant values to avoid a "static_cast<signed int>" in CodeGeneratorV8.

• Component: WebCore Misc.
• Status: RESOLVED FIXED

[Mike West]

When generating code for constants in IDL files, we're currently pushing the values through "static_cast<signed int>" in order to ensure that values like "0xFFFFFFFF" are properly handled as "-1". There's no reason to do this work in C when Perl is perfectly capable of manipulating text.

The FIXME at http://trac.webkit.org/browser/trunk/Source/WebCore/bindings/scripts/CodeGeneratorV8.pm?rev=144976#L2933 has been around since 2009. Now's as good a time as any to take care of it.

[Mike West]

Created attachment 191832 [details]
Patch

[Mike West]

Starting to dabble in bindings. Jochen, Dan, mind taking a look?

[WebKit Review Bot]

Comment on attachment 191832 [details]
Patch

Attachment 191832 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://webkit-commit-queue.appspot.com/results/17027132

New failing tests:
fast/canvas/webgl/css-webkit-canvas-repaint.html
fast/canvas/webgl/error-reporting.html
fast/canvas/webgl/gl-getstring.html
fast/canvas/webgl/copy-tex-image-and-sub-image-2d.html
fast/canvas/webgl/drawingbuffer-test.html
fast/canvas/webgl/gl-pixelstorei.html
fast/canvas/webgl/constants.html
http/tests/cache/subresource-failover-to-network.html
fast/canvas/webgl/functions-returning-strings.html
fast/canvas/webgl/context-attributes-alpha-depth-stencil-antialias.html
fast/canvas/webgl/buffer-data-array-buffer.html
fast/canvas/webgl/gl-enable-enum-test.html
fast/canvas/webgl/get-active-test.html
fast/canvas/webgl/context-lost-restored.html
fast/canvas/webgl/context-release-upon-reload.html
fast/canvas/webgl/buffer-bind-test.html
fast/canvas/webgl/gl-get-calls.html
fast/canvas/webgl/draw-arrays-out-of-bounds.html
fast/canvas/webgl/gl-bind-attrib-location-test.html
fast/canvas/webgl/framebuffer-test.html
fast/canvas/webgl/canvas-zero-size.html
fast/canvas/webgl/context-lost.html
fast/canvas/webgl/attrib-location-length-limits.html
fast/canvas/webgl/bad-arguments-test.html
compositing/webgl/webgl-reflection.html
fast/canvas/webgl/gl-enum-tests.html
fast/canvas/webgl/draw-elements-out-of-bounds.html
fast/canvas/webgl/canvas-2d-webgl-texture.html
fast/canvas/webgl/css-webkit-canvas.html
fast/canvas/webgl/gl-getshadersource.html

[Mike West]

Created attachment 191852 [details]
Patch

[Kentaro Hara]

Comment on attachment 191852 [details]
Patch

ok

[Mike West]

Comment on attachment 191852 [details]
Patch

Thanks.

[WebKit Review Bot]

Comment on attachment 191852 [details]
Patch

Clearing flags on attachment: 191852

Committed r145043: <http://trac.webkit.org/changeset/145043>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.

[Csaba Osztrogonác]

(In reply to comment #7)
> (From update of attachment 191852 [details])
> Clearing flags on attachment: 191852
> 
> Committed r145043: <http://trac.webkit.org/changeset/145043>

It broke the bindings generation tests. Could you fix it?

[Kentaro Hara]

(In reply to comment #9)
> It broke the bindings generation tests. Could you fix it?

Sorry every week... Fixed in r145060.