----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2616/ -----------------------------------------------------------
(Updated 2011-11-18 21:12:15.796911)
Review request for shindig.
Changes
-------
Updated approach using lifecycle listeners.
Summary
-------
We already have a callback function for navigating gadgets in the common
container which can be used to print timing information. Problem is if you
preload the gadget before navigating the timing information during navigation
is 0. It would be helpful to print timing information for preloading as well.
This addresses bug SHINDIG-1653.
https://issues.apache.org/jira/browse/SHINDIG-1653
Diffs (updated)
-----
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
