LGTM

On Fri, May 27, 2011 at 3:34 AM, Felix Lee <[email protected]> wrote:

>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/794/
> -----------------------------------------------------------
>
> Review request for shindig.
>
>
> Summary
> -------
>
> opensocial-reference/taming.js tries to whitelist some JsonPerson methods,
> which fails because JsonPerson isn't whitelisted yet.
> These methods are already being whitelisted in opensocial-base/taming.js,
> which is also where JsonPerson is whitelisted.
>
> This patch deletes the broken taming.
>
>
> Diffs
> -----
>
>  /trunk/features/src/main/javascript/features/opensocial-reference/taming.js
> 1128124
>
> Diff: https://reviews.apache.org/r/794/diff
>
>
> Testing
> -------
>
> SocialActivitiesWorld.xml demo works in sample container with Caja enabled,
> running in Chrome.
>
>
> Thanks,
>
> Felix
>
>

Reply via email to