We should only generate the static inline function isObservable when an IDL specifies GenerateIsReachable. Otherwise, this function is unused.
Created attachment 217589 [details] Patch and test
Note, I also added a space character to the right hand side of the first disjunct operator in the following line in CodeGeneratorJS.pm: if ((!$hasParent && !GetCustomIsReachable($interface))|| GetGenerateIsReachable($interface) || $codeGenerator->InheritsExtendedAttribute($interface, "ActiveDOMObject")) { if (GetGenerateIsReachable($interface)) { So that it would read: if ((!$hasParent && !GetCustomIsReachable($interface)) || GetGenerateIsReachable($interface) || $codeGenerator->InheritsExtendedAttribute($interface, "ActiveDOMObject")) { if (GetGenerateIsReachable($interface)) {
Attachment 217589 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/ChangeLog', u'Source/WebCore/bindings/scripts/CodeGeneratorJS.pm', u'Source/WebCore/bindings/scripts/test/CPP/WebDOMTestGenerateIsReachable.cpp', u'Source/WebCore/bindings/scripts/test/CPP/WebDOMTestGenerateIsReachable.h', u'Source/WebCore/bindings/scripts/test/GObject/WebKitDOMTestGenerateIsReachable.cpp', u'Source/WebCore/bindings/scripts/test/GObject/WebKitDOMTestGenerateIsReachable.h', u'Source/WebCore/bindings/scripts/test/GObject/WebKitDOMTestGenerateIsReachablePrivate.h', u'Source/WebCore/bindings/scripts/test/JS/JSTestActiveDOMObject.cpp', u'Source/WebCore/bindings/scripts/test/JS/JSTestCustomNamedGetter.cpp', u'Source/WebCore/bindings/scripts/test/JS/JSTestEventConstructor.cpp', u'Source/WebCore/bindings/scripts/test/JS/JSTestEventTarget.cpp', u'Source/WebCore/bindings/scripts/test/JS/JSTestException.cpp', u'Source/WebCore/bindings/scripts/test/JS/JSTestGenerateIsReachable.cpp', u'Source/WebCore/bindings/scripts/test/JS/JSTestGenerateIsReachable.h', u'Source/WebCore/bindings/scripts/test/JS/JSTestInterface.cpp', u'Source/WebCore/bindings/scripts/test/JS/JSTestMediaQueryListListener.cpp', u'Source/WebCore/bindings/scripts/test/JS/JSTestNamedConstructor.cpp', u'Source/WebCore/bindings/scripts/test/JS/JSTestObj.cpp', u'Source/WebCore/bindings/scripts/test/JS/JSTestOverloadedConstructors.cpp', u'Source/WebCore/bindings/scripts/test/JS/JSTestSerializedScriptValueInterface.cpp', u'Source/WebCore/bindings/scripts/test/JS/JSTestTypedefs.cpp', u'Source/WebCore/bindings/scripts/test/JS/JSattribute.cpp', u'Source/WebCore/bindings/scripts/test/JS/JSreadonly.cpp', u'Source/WebCore/bindings/scripts/test/ObjC/DOMTestGenerateIsReachable.h', u'Source/WebCore/bindings/scripts/test/ObjC/DOMTestGenerateIsReachable.mm', u'Source/WebCore/bindings/scripts/test/ObjC/DOMTestGenerateIsReachableInternal.h', u'Source/WebCore/bindings/scripts/test/TestGenerateIsReachable.idl']" exit_code: 1 Source/WebCore/bindings/scripts/test/JS/JSTestGenerateIsReachable.cpp:158: Extra space before ( in function call [whitespace/parens] [4] Total errors found: 1 in 13 files If any of these errors are false positives, please file a bug against check-webkit-style.
(In reply to comment #2) > Note, I also added a space character to the right hand side of the first disjunct operator in the following line in CodeGeneratorJS.pm: *left hand side of the first disjunct
Comment on attachment 217589 [details] Patch and test r=me
Committed r159648: <http://trac.webkit.org/changeset/159648>