The latest patch set parses both sourceURL and sourceMappingURL, and allows
them
in the middle of the script too (the later one overrides the previous one, if
there are several instances of the same value).
Great, thank you!

I'm not sure if you'd need more plumbing than this, for example,
source_mapping_url is not plumbed via StackFrame. Is it enough that Script has
the source_mapping_url accessor?
Should be enough. I can write up blink-side plumbing so that we could check that
inspector tests are passing as well.

Do you plan to remove the older javascript-based sourceURL parsing?
This should be probably done in a separate patch so that v8 roll doesn't break
the build but other than that I don't think it is still needed.


https://codereview.chromium.org/316173002/

--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to