> On 2011-06-21 15:45:22, Paul Lindner wrote: > > trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/uri/DefaultIframeUriManager.java, > > line 136 > > <https://reviews.apache.org/r/939/diff/2/?file=21409#file21409line136> > > > > set host = '//' here and then append to it. > >
Good suggestion Paul, will make the change. - Henry ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/939/#review869 ----------------------------------------------------------- On 2011-06-21 17:57:51, Henry Saputra wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/939/ > ----------------------------------------------------------- > > (Updated 2011-06-21 17:57:51) > > > Review request for shindig. > > > Summary > ------- > > Remove scheme from default test domain/host name in the config/container.js. > > Also make the gadgets.uri.iframe.unlockedDomain is free from // and add it in > the default Iframe URL manager to avoid opaque URL. > > > Diffs > ----- > > trunk/config/container.js 1137864 > > trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/uri/DefaultIframeUriManager.java > 1137864 > > trunk/java/gadgets/src/test/java/org/apache/shindig/gadgets/uri/DefaultIframeUriManagerTest.java > 1137864 > > Diff: https://reviews.apache.org/r/939/diff > > > Testing > ------- > > Modify unit test. > > > Thanks, > > Henry > >
