----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5810/#review8953 -----------------------------------------------------------
trunk/features/src/main/javascript/features/rpc/rpc.js <https://reviews.apache.org/r/5810/#comment19041> Is this also going to cause an issue in the scenario you described? This is the only other use of window.top I see. - Stanton Sievers On July 6, 2012, 6:11 p.m., Henry Saputra wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/5810/ > ----------------------------------------------------------- > > (Updated July 6, 2012, 6:11 p.m.) > > > Review request for shindig. > > > Description > ------- > > The Shindig container could be rendered inside parent iframe and in this > scenario, the rpc check for isChild (almost the same as check fro isGadget) > does not work well because its actually act as the real container even though > its put inside another iframe. > > The proposed fix to add check for url param "parent" as well as the check for > window instance. > > > Diffs > ----- > > trunk/features/src/main/javascript/features/rpc/rpc.js 1356941 > > Diff: https://reviews.apache.org/r/5810/diff/ > > > Testing > ------- > > Unit tests still pass. Common container renders gadgets and rpc works as > expected. > > > Thanks, > > Henry Saputra > >
