Following on bug 146060, we could do some improvements. >> Source/WebCore/bindings/js/JSDOMPromise.h:62 >> +template<class JSClassName> > >The type here is the JavaScript class, not the JavaScript class name. I suggest the name WrapperClass here instead of JSClassName. Also, we almost always use typename rather >than class in templates, although I’m not entirely sure why. > >> Source/WebCore/bindings/js/JSDOMPromise.h:63 >> +inline JSC::JSValue callPromiseFunction(JSC::ExecState& state, JSClassName& jsObject, JSC::EncodedJSValue promiseFunction(JSC::ExecState*, JSClassName*, >JSC::JSPromiseDeferred*)) > >A typical name used in bindings for what we are calling jsObject here is “wrapper”
Created attachment 255483 [details] Patch
Given the modifications are already suggested as part of a review, I do not think a review is needed. But just in case, I will wait for tomorrow to cq it.
Comment on attachment 255483 [details] Patch Clearing flags on attachment: 255483 Committed r185919: <http://trac.webkit.org/changeset/185919>
All reviewed patches have been landed. Closing bug.