...
Created attachment 226366 [details] work in progress
Created attachment 226369 [details] it compiles Not done yet though.
Created attachment 226376 [details] starting to work
Created attachment 226382 [details] almost done
Created attachment 226393 [details] the patch
Attachment 226393 [details] did not pass style-queue: ERROR: Source/JavaScriptCore/bytecode/PolymorphicGetByIdList.h:104: The parameter name "stubInfo" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/JavaScriptCore/bytecode/StructureStubInfo.h:51: enum members should use InterCaps with an initial capital letter. [readability/enum_casing] [4] ERROR: Source/JavaScriptCore/bytecode/PolymorphicGetByIdList.cpp:41: Wrong number of spaces before statement. (expected: 8) [whitespace/indent] [4] ERROR: Source/JavaScriptCore/bytecode/PolymorphicGetByIdList.cpp:42: Wrong number of spaces before statement. (expected: 8) [whitespace/indent] [4] ERROR: Source/JavaScriptCore/bytecode/PolymorphicGetByIdList.cpp:43: Wrong number of spaces before statement. (expected: 8) [whitespace/indent] [4] ERROR: Source/JavaScriptCore/bytecode/PolymorphicGetByIdList.cpp:44: Wrong number of spaces before statement. (expected: 8) [whitespace/indent] [4] Total errors found: 6 in 16 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 226393 [details] the patch r=me
Landed in http://trac.webkit.org/changeset/165459
(In reply to comment #8) > Landed in http://trac.webkit.org/changeset/165459 It caused regression on ARM Thumb2 - https://bugs.webkit.org/show_bug.cgi?id=130134
(In reply to comment #9) > (In reply to comment #8) > > Landed in http://trac.webkit.org/changeset/165459 > > It caused regression on ARM Thumb2 - https://bugs.webkit.org/show_bug.cgi?id=130134 Same regression occured on 32 bit Apple Mac build too. But https://bugs.webkit.org/show_bug.cgi?id=130203 fixed 94 failures from the 110 , only 16 failures remains.
*** Bug 129590 has been marked as a duplicate of this bug. ***