RESOLVED FIXED 127226
[CodeGeneratorJS] Sort output of StructureFlags and function parameters 
https://bugs.webkit.org/show_bug.cgi?id=127226
Summary [CodeGeneratorJS] Sort output of StructureFlags and function parameters
Alberto Garcia
Reported 2014-01-18 10:29:29 PST
We're using the following code to create StructureFlags: foreach my $structureFlag (keys %structureFlags) { push(@headerContent, $structureFlag . " | "); } The problem is that keys %structureFlags returns a list in an unspecified order, which can change between Perl versions and is indeed making the bindings tests fail in one of the GTK+ bots: PASS: (JS) JSTestException.cpp FAIL: (JS) JSTestException.h --- WebCore/bindings/scripts/test/JS/JSTestException.h 2014-01-17 06:05:06.759471284 -0800 +++ /tmp/tmpGfPzJ5/JSTestException.h 2014-01-18 01:22:50.056392293 -0800 @@ -67,7 +67,7 @@ protected: JSTestException(JSC::Structure*, JSDOMGlobalObject*, PassRefPtr<TestException>); void finishCreation(JSC::VM&); - static const unsigned StructureFlags = JSC::OverridesGetOwnPropertySlot | JSC::InterceptsGetOwnPropertySlotByIndexEvenWhenLengthIsNotZero | Base::StructureFlags; + static const unsigned StructureFlags = JSC::InterceptsGetOwnPropertySlotByIndexEvenWhenLengthIsNotZero | JSC::OverridesGetOwnPropertySlot | Base::StructureFlags; }; class JSTestExceptionOwner : public JSC::WeakHandleOwner {
Attachments
Patch (17.18 KB, patch)
2014-01-18 10:32 PST, Alberto Garcia 		no flags
DetailsFormatted DiffDiff
Patch (18.75 KB, patch)
2014-01-18 11:33 PST, Alberto Garcia 		ap: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Alberto Garcia
Comment 1 2014-01-18 10:32:24 PST
Created attachment 221557 [details] Patch
Alberto Garcia
Comment 2 2014-01-18 11:30:00 PST
This also affects function parameters, I'll update the patch.
Alberto Garcia
Comment 3 2014-01-18 11:33:50 PST
Created attachment 221560 [details] Patch
Alberto Garcia
Comment 4 2014-01-18 15:26:56 PST
Committed r162280: <http://trac.webkit.org/changeset/162280>
Note You need to log in before you can comment on or make changes to this bug.