----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/1615/#review1604 -----------------------------------------------------------
http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/selection/selection_container.js <https://reviews.apache.org/r/1615/#comment3626> How does this ever get called? Shouldn't this be called when the gadget calles removeListener? If so it should be called from your router function...right? http://svn.apache.org/repos/asf/shindig/trunk/features/src/test/javascript/features/selection/selection_test.js <https://reviews.apache.org/r/1615/#comment3627> Shouldn't there be a test for removeListener as well? - Ryan On 2011-08-23 23:06:15, Matthew Hatem wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/1615/ > ----------------------------------------------------------- > > (Updated 2011-08-23 23:06:15) > > > Review request for shindig and Ryan Baxter. > > > Summary > ------- > > https://issues.apache.org/jira/browse/SHINDIG-1591 > > > This addresses bug SHINDIG-1591. > https://issues.apache.org/jira/browse/SHINDIG-1591 > > > Diffs > ----- > > > http://svn.apache.org/repos/asf/shindig/trunk/content/samplecontainer/examples/conservcontainer/sample-actions-runner.xml > 1160436 > > http://svn.apache.org/repos/asf/shindig/trunk/content/samplecontainer/examples/conservcontainer/sample-selection-listener.xml > 1160436 > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/selection/feature.xml > 1160436 > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/selection/selection.js > 1160436 > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/selection/selection_container.js > 1160436 > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/selection/taming.js > 1160436 > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/test/javascript/features/selection/selection_test.js > 1160436 > > Diff: https://reviews.apache.org/r/1615/diff > > > Testing > ------- > > Passes all unit tests. > > > Thanks, > > Matthew > >
