Reviewers: benl_google.com,
Description:
The fastpath test for property assignment was just testing whether the
fastpath flag is truthy, which it might be for an inherited writable
property. Instead, we now test that its value is the object itself (as
the slow path test was already doing), in order to test that the own
property is writable.
Please review this at http://codereview.appspot.com/124103
Affected files:
M src/com/google/caja/parser/quasiliteral/CajitaRewriter.java
M src/com/google/caja/plugin/stages/CajaRuntimeDebuggingRewriter.java
M
tests/com/google/caja/opensocial/example-dynamic-styles-rewritten.xml
M tests/com/google/caja/opensocial/example-rewritten.xml
M tests/com/google/caja/parser/quasiliteral/CajitaRewriterTest.java
M
tests/com/google/caja/parser/quasiliteral/CommonJsRewriterTestCase.java
M
tests/com/google/caja/parser/quasiliteral/DefaultValijaRewriterTest.java