----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2616/#review3405 -----------------------------------------------------------
Ship it! I only reviewed the changes to the actions feature. LFTM. - Matthew On 2011-11-18 21:56:17, Ryan Baxter wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/2616/ > ----------------------------------------------------------- > > (Updated 2011-11-18 21:56:17) > > > Review request for shindig, Matthew Hatem and Igor Belakovskiy. > > > Summary > ------- > > I have added additional lifecycle handlers. My primary use case here is to > be able to time the different events that take place as a gadget is rendered, > but these additional lifecycle handlers may be used for other things as well. > > > This addresses bug SHINDIG-1653. > https://issues.apache.org/jira/browse/SHINDIG-1653 > > > Diffs > ----- > > > http://svn.apache.org/repos/asf/shindig/trunk/content/samplecontainer/examples/commoncontainer/assembler.js > 1203746 > > http://svn.apache.org/repos/asf/shindig/trunk/content/samplecontainer/examples/commoncontainer/viewController.js > 1203746 > > http://svn.apache.org/repos/asf/shindig/trunk/extras/src/main/javascript/features-extras/org.openajax.hub-2.0.5/iframe.js > 1203746 > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/actions/actions_container.js > 1203746 > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/container.gadget/gadget_holder.js > 1203746 > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/container.gadget/gadget_site.js > 1203746 > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/container.util/constant.js > 1203746 > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/container/container.js > 1203746 > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/opensearch/opensearch.js > 1203746 > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/shared-script-frame/shared-script-frame-container.js > 1203746 > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/test/javascript/features/container/gadget_holder_test.js > 1203746 > > Diff: https://reviews.apache.org/r/2616/diff > > > Testing > ------- > > Modified sample container to log this information and updated unit tests. > > > Thanks, > > Ryan > >
