Bug 124729 – Only generate isObservable() when IDL specifies GenerateIsReachable

• Component: Bindings
• Status: RESOLVED FIXED

[Daniel Bates]

We should only generate the static inline function isObservable when an IDL specifies GenerateIsReachable. Otherwise, this function is unused.

[Daniel Bates]

Created attachment 217589 [details]
Patch and test

[Daniel Bates]

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)) {

[WebKit Commit Bot]

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.

[Daniel Bates]

(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

[Geoffrey Garen]

Comment on attachment 217589 [details]
Patch and test

r=me

[Daniel Bates]

Committed r159648: <http://trac.webkit.org/changeset/159648>