r51884 broke things
Created attachment 44552 [details] fix compile errors
Attachment 44552 [details] did not pass style-queue: Failed to run "WebKitTools/Scripts/check-webkit-style" exit_code: 1 WebCore/bindings/v8/custom/V8WebGLRenderingContextCustom.cpp:643: A case label should not be indented, but line up with its switch statement. [whitespace/indent] [4] WebCore/bindings/v8/custom/V8WebGLRenderingContextCustom.cpp:673: A case label should not be indented, but line up with its switch statement. [whitespace/indent] [4] WebCore/bindings/v8/custom/V8WebGLRenderingContextCustom.cpp:721: A case label should not be indented, but line up with its switch statement. [whitespace/indent] [4] WebCore/bindings/v8/custom/V8WebGLRenderingContextCustom.cpp:747: A case label should not be indented, but line up with its switch statement. [whitespace/indent] [4] WebCore/bindings/v8/custom/V8WebGLRenderingContextCustom.cpp:841: A case label should not be indented, but line up with its switch statement. [whitespace/indent] [4] WebCore/bindings/v8/custom/V8WebGLRenderingContextCustom.cpp:866: A case label should not be indented, but line up with its switch statement. [whitespace/indent] [4] Total errors found: 6
bah... i misssed another compile error... new patch coming soon
Created attachment 44555 [details] Fix compile errors
Attachment 44555 [details] did not pass style-queue: Failed to run "WebKitTools/Scripts/check-webkit-style" exit_code: 1 WebCore/bindings/v8/custom/V8WebGLRenderingContextCustom.cpp:643: A case label should not be indented, but line up with its switch statement. [whitespace/indent] [4] WebCore/bindings/v8/custom/V8WebGLRenderingContextCustom.cpp:673: A case label should not be indented, but line up with its switch statement. [whitespace/indent] [4] WebCore/bindings/v8/custom/V8WebGLRenderingContextCustom.cpp:721: A case label should not be indented, but line up with its switch statement. [whitespace/indent] [4] WebCore/bindings/v8/custom/V8WebGLRenderingContextCustom.cpp:747: A case label should not be indented, but line up with its switch statement. [whitespace/indent] [4] WebCore/bindings/v8/custom/V8WebGLRenderingContextCustom.cpp:841: A case label should not be indented, but line up with its switch statement. [whitespace/indent] [4] WebCore/bindings/v8/custom/V8WebGLRenderingContextCustom.cpp:866: A case label should not be indented, but line up with its switch statement. [whitespace/indent] [4] Total errors found: 6
Ok, this is not cool. Peterson, please compile your patches before landing.
Comment on attachment 44555 [details] Fix compile errors ok.
Comment on attachment 44555 [details] Fix compile errors Clearing flags on attachment: 44555 Committed r51915: <http://trac.webkit.org/changeset/51915>
All reviewed patches have been landed. Closing bug.