> On 2011-12-04 18:55:14, Ryan Baxter wrote: > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/container/container.js, > > line 262 > > <https://reviews.apache.org/r/2994/diff/4/?file=61735#file61735line262> > > > > Rename this method if we are calling it for URL sites as well
I want to wait to do that in another changeset with the changes to align the two site types under a common parent class. > On 2011-12-04 18:55:14, Ryan Baxter wrote: > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/container/container.js, > > line 610 > > <https://reviews.apache.org/r/2994/diff/4/?file=61735#file61735line610> > > > > So can I now call this for URL sites as well? If so we should rename > > the method No, there's another method for url sites with a different name. > On 2011-12-04 18:55:14, Ryan Baxter wrote: > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/container/container.js, > > line 622 > > <https://reviews.apache.org/r/2994/diff/4/?file=61735#file61735line622> > > > > Rename this method because it is no longer for just GadgetSites. Yeah, since this method is new, I'm fine with that. > On 2011-12-04 18:55:14, Ryan Baxter wrote: > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/container/container.js, > > line 912 > > <https://reviews.apache.org/r/2994/diff/4/?file=61735#file61735line912> > > > > Are we sure there are no other assumptions about the sites_ array > > containing only GadgetSites? It looks like everything is accounted for now. Most calls just iterate over the map looking for a site with an id or iframeId of a particular value. The refresh tokens call has been scoped to only operate on gadget sites. - Dan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2994/#review3609 ----------------------------------------------------------- On 2011-12-02 20:40:53, Dan Dumont wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/2994/ > ----------------------------------------------------------- > > (Updated 2011-12-02 20:40:53) > > > Review request for shindig, Ryan Baxter, Jason Chiang, Matthew Hatem, and > Stanton Sievers. > > > Summary > ------- > > As a general rule, any time anyone is writing a feature using RPC, those > endpoints should never expect to get a function (or something that contains a > function) as an argument. > The serialization that happens when running with locked domains will null the > function argument when (and only when) in locked domains (so jsunit tests > will pass). > > The check is a bit to expensive to perform for each rpc call so I think > people are just going to have to stay sharp. > > Side note: > The ugly code you'll run into to get the gadget holder from a site is > temporary. I want to go in later and refactor gadget sites, gadget holders, > url sites, and url holders to inherit from a common parent an remove a bunch > of the duplication and make the methods make more sense. > > > This addresses bug SHINDIG-1664. > https://issues.apache.org/jira/browse/SHINDIG-1664 > > > Diffs > ----- > > > http://svn.apache.org/repos/asf/shindig/trunk/content/samplecontainer/examples/conservcontainer/sample-actions-voip.xml > 1209641 > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/actions/actions.js > 1209641 > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/actions/actions_container.js > 1209641 > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/actions/feature.xml > 1209641 > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/container.gadget/gadget_site.js > 1209641 > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/container.url/url_site.js > 1209641 > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/container/container.js > 1209641 > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/container/feature.xml > 1209641 > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/open-views/feature.xml > 1209641 > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/open-views/viewenhancements-container.js > 1209641 > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/open-views/viewenhancements.js > 1209641 > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/selection/feature.xml > 1209641 > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/selection/selection.js > 1209641 > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/selection/selection_container.js > 1209641 > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/test/javascript/features/actions/actions_test.js > 1209641 > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/test/javascript/features/container.url/url_site_test.js > 1209641 > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/test/javascript/features/container/gadget_site_test.js > 1209641 > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/test/javascript/features/open-views/viewEnhancements-test.js > 1209641 > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/test/javascript/features/selection/selection_test.js > 1209641 > > Diff: https://reviews.apache.org/r/2994/diff > > > Testing > ------- > > Updated jsunit tests, they all pass. > > Tested with some gadgets to exercise the open-views, selection, and actions > api. > > > Thanks, > > Dan > >
