Reviewers: Mads Ager, Vitaly,
Message:
Mads and Vitaly,
may you have a look?
This patch requires some adjustments in V8 bindings for layout tests to
pass,
but those changes appear reasonable and save to one rebaseline can go in
even
without this patch.
When this patch gets in better shape, I'll commit CL for WebKit before this
change hits the trunk.
I haven't added additional checks yet (ones we discussed): I have almost
working
patch which unifies pending and scheduled exceptions and will do it as part
of
this work. However, I'd rather stage this patch for some time on DevChannel
before messing with the things.
Description:
Make exception thrown via v8 public API propagate to v8::TryCatch as JS
thrown
exceptions do.
Correctly process failures which can be returned by Object::GetProperty
when performing GetRealNamedProperty* queries.
Callback properties can produce exceptions so we need to wrap access to them
into exception checks. However, despite of many other methods with
exception
checks, property access doesn't mandatroy go via JavaScript and hence we
need to inject code to propagate exception to public API TryCatch handlers.
Please review this at http://codereview.chromium.org/6397011/
SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge
Affected files:
M src/api.cc
M src/messages.cc
M src/top.h
M src/top.cc
M test/cctest/test-api.cc
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
