embedded experiences patch

As described in the specificication here:

http://docs.opensocial.org/display/OSD/Embedded+Experiences

Please note that this change is dependent on the View Enhancement patch submitted on 06/20

http://codereview.appspot.com/4641058/

The samples can be found in shindig-project/content/samplecontainer/examples/embeddedexperiences/

[igor1x, 2011-07-07 14:02:08]

fixed a typo in the feature code.

[igor1x, 2011-07-07 14:10:19]

without tabs this time.

[igor1x, 2011-07-08 15:09:00]

Small change in the cannonical db+unit test modification to reflect that change.

[Paul Lindner, 2011-07-11 16:05:34]

oops, forgot about these comments.. plz ignore if they're no longer relevant.

Otherwise looking good, will try to peek at this again later today.

http://codereview.appspot.com/4631090/diff/11001/features/src/main/javascript...
File
features/src/main/javascript/features/embeddedexperiences/embedded_experiences_container.js
(right):

http://codereview.appspot.com/4631090/diff/11001/features/src/main/javascript...
features/src/main/javascript/features/embeddedexperiences/embedded_experiences_container.js:28:
/**
looks like the indent went to three here.  Can you move it to 2?

http://codereview.appspot.com/4631090/diff/11001/features/src/main/javascript...
features/src/main/javascript/features/embeddedexperiences/embedded_experiences_container.js:34:
* @param {function} optional callback function
rename to opt_callback and use {function=} jsdoc.

http://codereview.appspot.com/4631090/diff/11001/features/src/main/javascript...
features/src/main/javascript/features/embeddedexperiences/embedded_experiences_container.js:53:
* @param {object} renderParams params to augment the rendering.
uppercase object

http://codereview.appspot.com/4631090/diff/11001/features/src/main/javascript...
features/src/main/javascript/features/embeddedexperiences/embedded_experiences_container.js:133:
}
nit: spacing

[rbaxter85, 2011-07-12 01:39:18]

Updated patch and moved code review to the apache code review tool.
https://reviews.apache.org/r/1085/

http://codereview.appspot.com/4631090/diff/11001/features/src/main/javascript...
File
features/src/main/javascript/features/embeddedexperiences/embedded_experiences_container.js
(right):

http://codereview.appspot.com/4631090/diff/11001/features/src/main/javascript...
features/src/main/javascript/features/embeddedexperiences/embedded_experiences_container.js:28:
/**
On 2011/07/11 16:05:34, Paul Lindner wrote:
> looks like the indent went to three here.  Can you move it to 2?

Done.

http://codereview.appspot.com/4631090/diff/11001/features/src/main/javascript...
features/src/main/javascript/features/embeddedexperiences/embedded_experiences_container.js:34:
* @param {function} optional callback function
Actually the function doesn''t even take in a function, so I removed it
completely.
On 2011/07/11 16:05:34, Paul Lindner wrote:
> rename to opt_callback and use {function=} jsdoc.

http://codereview.appspot.com/4631090/diff/11001/features/src/main/javascript...
features/src/main/javascript/features/embeddedexperiences/embedded_experiences_container.js:53:
* @param {object} renderParams params to augment the rendering.
On 2011/07/11 16:05:34, Paul Lindner wrote:
> uppercase object

Done.

http://codereview.appspot.com/4631090/diff/11001/features/src/main/javascript...
features/src/main/javascript/features/embeddedexperiences/embedded_experiences_container.js:133:
}
On 2011/07/11 16:05:34, Paul Lindner wrote:
> nit: spacing

Done.