> On 2012-01-18 20:42:51, Michael MacFadden wrote: > > src/org/waveprotocol/wave/client/doodad/title/TitleAnnotationHandler.java, > > line 50 > > <https://reviews.apache.org/r/3530/diff/1/?file=69698#file69698line50> > > > > This was made public. But I don't see that it us used anywhere outside > > of this class.
done - Yuri ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3530/#review4442 ----------------------------------------------------------- On 2012-01-19 20:11:38, Yuri Zelikov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/3530/ > ----------------------------------------------------------- > > (Updated 2012-01-19 20:11:38) > > > Review request for wave, Michael MacFadden and Ali Lown. > > > Summary > ------- > > Sets the current wave title also as the browser window title. > > > Diffs > ----- > > src/org/waveprotocol/box/webclient/client/StagesProvider.java 8a5355a > src/org/waveprotocol/wave/client/StageThree.java f31ed3b > src/org/waveprotocol/wave/client/doodad/title/TitleAnnotationHandler.java > e8b13f7 > src/org/waveprotocol/wave/client/scroll/ProxyScrollPanel.java 27e5da7 > src/org/waveprotocol/wave/client/wavepanel/WavePanel.java d1944b7 > src/org/waveprotocol/wave/client/wavepanel/impl/WavePanelImpl.java 282387e > src/org/waveprotocol/wave/client/wavepanel/impl/edit/EditSession.java > b4f141b > > src/org/waveprotocol/wave/client/wavepanel/impl/focus/FocusFramePresenter.java > 1351987 > > src/org/waveprotocol/wave/client/wavepanel/impl/title/WindowTitleHandler.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/3530/diff > > > Testing > ------- > > Tested locally, verified that the wave title also set as browser window title. > > > Thanks, > > Yuri > >
