Bug 197244 – Add SPI callbacks for before and after module execution

• Status: RESOLVED
• Resolution: FIXED
• Component: JavaScriptCore

[Saam Barati]

...

[Radar WebKit Bug Importer]

<rdar://problem/50180511>

[Saam Barati]

Created attachment 368183 [details]
patch

[EWS Watchlist]

Attachment 368183 [details] did not pass style-queue:


ERROR: Source/JavaScriptCore/API/tests/testapi.mm:2033:  Tests for true/false, null/non-null, and zero/non-zero should all be done without equality comparisons.  [readability/comparison_to_zero] [5]
ERROR: Source/JavaScriptCore/API/tests/testapi.mm:2034:  Tests for true/false, null/non-null, and zero/non-zero should all be done without equality comparisons.  [readability/comparison_to_zero] [5]
Total errors found: 2 in 7 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Yusuke Suzuki]

Comment on attachment 368183 [details]
patch

View in context: https://bugs.webkit.org/attachment.cgi?id=368183&action=review

r=me. BTW, this use case is actually why we have moduleLoaderEvaluate hook in ModuleLoader APIs. WebCore does the almost same thing to tell the inspector about module evaluation :)

> Source/JavaScriptCore/API/JSAPIGlobalObject.mm:249
> +        String moduleKey = key.toWTFString(exec);
> +        RETURN_IF_EXCEPTION(scope, { });

In JSContext / JSScript module loading API desing, the key is always URL String (not Symbol), is my understanding correct? If so, it is fine.

[Saam Barati]

(In reply to Yusuke Suzuki from comment #4)
> Comment on attachment 368183 [details]
> patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=368183&action=review
> 
> r=me. BTW, this use case is actually why we have moduleLoaderEvaluate hook
> in ModuleLoader APIs. WebCore does the almost same thing to tell the
> inspector about module evaluation :)
Nice.

> 
> > Source/JavaScriptCore/API/JSAPIGlobalObject.mm:249
> > +        String moduleKey = key.toWTFString(exec);
> > +        RETURN_IF_EXCEPTION(scope, { });
> 
> In JSContext / JSScript module loading API desing, the key is always URL
> String (not Symbol), is my understanding correct? If so, it is fine.

That's correct.

[WebKit Commit Bot]

Comment on attachment 368183 [details]
patch

Clearing flags on attachment: 368183

Committed r244620: <https://trac.webkit.org/changeset/244620>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.