----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8916/#review15237 -----------------------------------------------------------
Andrew, this looks like another good feature for Wave. (Since we seem to have more business interest outside of the English-speaking world than in it). As far as the code goes, it appears to be mostly fine. There are a few minor suggestions I have made, but this is mostly just formatting changes. You appear to have automatically generated a lot of the later files, but left the default template header on. A lot of my comments below are simply regarding trailing whitespace. I suggest you setup your editor to remove it automatically. ./src/org/waveprotocol/box/server/account/HumanAccountDataImpl.java <https://reviews.apache.org/r/8916/#comment32850> Why did you remove this precondition? If it is deliberate to allow null participant id (is this useful?), then the javadoc above needs to be updated. If it is not deliberate, please add it back. ./src/org/waveprotocol/box/server/account/HumanAccountDataImpl.java <https://reviews.apache.org/r/8916/#comment32849> Trailing space. ./src/org/waveprotocol/box/server/persistence/protos/account-store.proto <https://reviews.apache.org/r/8916/#comment32851> Indentation here should be consistent. ./src/org/waveprotocol/box/server/rpc/WaveClientServlet.java <https://reviews.apache.org/r/8916/#comment32852> trailing space. ./src/org/waveprotocol/box/server/rpc/WaveClientServlet.java <https://reviews.apache.org/r/8916/#comment32853> Rather than stacking these if's (which is making it harder to follow the logic directly) can we flatten it to single-depth, and simply 'return' if not met. ./src/org/waveprotocol/box/server/util/URLUtil.java <https://reviews.apache.org/r/8916/#comment32858> Can we have the Apache header here instead? ./src/org/waveprotocol/box/server/util/URLUtil.java <https://reviews.apache.org/r/8916/#comment32854> Trailing whitespace. ./src/org/waveprotocol/box/server/util/URLUtil.java <https://reviews.apache.org/r/8916/#comment32855> Trailing whitespace. ./src/org/waveprotocol/box/server/util/URLUtil.java <https://reviews.apache.org/r/8916/#comment32856> Trailing whitespace. ./src/org/waveprotocol/box/server/util/URLUtil.java <https://reviews.apache.org/r/8916/#comment32857> Trailing whitespace. ./src/org/waveprotocol/box/webclient/WebClientProd.gwt.xml <https://reviews.apache.org/r/8916/#comment32860> whitespace. ./src/org/waveprotocol/box/webclient/WebClientProd.gwt.xml <https://reviews.apache.org/r/8916/#comment32859> whitespace. ./src/org/waveprotocol/box/webclient/client/LocaleService.java <https://reviews.apache.org/r/8916/#comment32861> Apache header please. ./src/org/waveprotocol/box/webclient/client/RemoteLocaleService.java <https://reviews.apache.org/r/8916/#comment32862> This comment seems out from the rest of the code. Perhaps 'Manage locale settings on the server.' would be better. ./src/org/waveprotocol/box/webclient/client/Session.java <https://reviews.apache.org/r/8916/#comment32863> whitespace. ./src/org/waveprotocol/box/webclient/client/WebClient.java <https://reviews.apache.org/r/8916/#comment32864> whitespace. ./src/org/waveprotocol/box/webclient/client/WebClient.java <https://reviews.apache.org/r/8916/#comment32865> whitespace. ./src/org/waveprotocol/box/webclient/client/WebClient.java <https://reviews.apache.org/r/8916/#comment32866> whitespace. ./src/org/waveprotocol/box/webclient/client/WebClient.java <https://reviews.apache.org/r/8916/#comment32867> I don't like having this 'magic' string here. Could you at least move it to a commented constant at the top of the class. ./src/org/waveprotocol/box/webclient/client/i18n/SavedStateMessages.java <https://reviews.apache.org/r/8916/#comment32868> All of these need to be changed to Apache headers rather than this default header. ./src/org/waveprotocol/box/webclient/client/i18n/WebClientMessages.java <https://reviews.apache.org/r/8916/#comment32870> Why are you setting the default message here as well? This is duplicating the message between this file and the properties file. ./src/org/waveprotocol/box/webclient/client/i18n/WebClientMessages.java <https://reviews.apache.org/r/8916/#comment32869> whitespace. ./src/org/waveprotocol/box/webclient/search/SearchPresenter.java <https://reviews.apache.org/r/8916/#comment32871> whitespace. ./src/org/waveprotocol/box/webclient/search/i18n/SearchPresenterMessages_fr.properties <https://reviews.apache.org/r/8916/#comment32872> whitespace. ./src/org/waveprotocol/box/webclient/search/i18n/SearchPresenterMessages_fr.properties <https://reviews.apache.org/r/8916/#comment32873> ./src/org/waveprotocol/box/webclient/widget/error/i18n/ErrorMessages.java <https://reviews.apache.org/r/8916/#comment32874> whitespace in this file. ./src/org/waveprotocol/wave/client/StageThree.java <https://reviews.apache.org/r/8916/#comment32875> whitespace. ./src/org/waveprotocol/wave/client/wavepanel/impl/toolbar/LinkerHelper.java <https://reviews.apache.org/r/8916/#comment32876> whitespace. ./src/org/waveprotocol/wave/client/wavepanel/impl/toolbar/attachment/i18n/AttachmentMessages.java <https://reviews.apache.org/r/8916/#comment32877> whitespace in this file. ./src/org/waveprotocol/wave/client/wavepanel/impl/toolbar/gadget/GadgetSelectorWidget.java <https://reviews.apache.org/r/8916/#comment32878> whitespace. ./src/org/waveprotocol/wave/client/wavepanel/impl/toolbar/gadget/GadgetSelectorWidget.ui.xml <https://reviews.apache.org/r/8916/#comment32879> whitespace. ./src/org/waveprotocol/wave/client/wavepanel/impl/toolbar/gadget/i18n/GadgetCategoryMessages.java <https://reviews.apache.org/r/8916/#comment32880> whitespace in this file. ./src/org/waveprotocol/wave/client/wavepanel/impl/toolbar/gadget/i18n/GadgetSelectorMessages.java <https://reviews.apache.org/r/8916/#comment32881> whitespace here too. ./src/org/waveprotocol/wave/client/wavepanel/impl/toolbar/i18n/LinkerMessages.java <https://reviews.apache.org/r/8916/#comment32882> whitespace. ./src/org/waveprotocol/wave/client/wavepanel/impl/toolbar/i18n/ToolbarMessages_fr.properties <https://reviews.apache.org/r/8916/#comment32883> whitespace. ./src/org/waveprotocol/wave/client/wavepanel/view/dom/full/BlipLinkPopupWidget.ui.xml <https://reviews.apache.org/r/8916/#comment32884> whitespace. ./src/org/waveprotocol/wave/client/wavepanel/view/dom/full/BlipMetaViewBuilder.java <https://reviews.apache.org/r/8916/#comment32885> whitespace. ./src/org/waveprotocol/wave/client/wavepanel/view/dom/full/WavePanelResourceLoader.java <https://reviews.apache.org/r/8916/#comment32886> whitespace. ./src/org/waveprotocol/wave/client/wavepanel/view/dom/full/WavePanelResourceLoader.java <https://reviews.apache.org/r/8916/#comment32887> and here. ./src/org/waveprotocol/wave/client/wavepanel/view/dom/full/i18n/BlipLinkMessages.java <https://reviews.apache.org/r/8916/#comment32888> whitespace. ./src/org/waveprotocol/wave/client/wavepanel/view/dom/full/i18n/BlipLinkMessages_ru.properties <https://reviews.apache.org/r/8916/#comment32889> whitespace. Why is there a line between each translation in this file only? ./src/org/waveprotocol/wave/client/wavepanel/view/dom/full/i18n/BlipMessages_fr.properties <https://reviews.apache.org/r/8916/#comment32890> empty line? ./src/org/waveprotocol/wave/client/wavepanel/view/dom/full/i18n/ParticipantMessages.java <https://reviews.apache.org/r/8916/#comment32891> whitespace. ./src/org/waveprotocol/wave/client/wavepanel/view/dom/full/i18n/ParticipantMessages_en.properties <https://reviews.apache.org/r/8916/#comment32892> empty line? ./src/org/waveprotocol/wave/client/wavepanel/view/dom/full/i18n/ParticipantMessages_sl.properties <https://reviews.apache.org/r/8916/#comment32893> empty line? ./src/org/waveprotocol/wave/client/widget/profile/ProfilePopupWidget.ui.xml <https://reviews.apache.org/r/8916/#comment32894> whitespace. Thanks. Ali - Ali Lown On Jan. 10, 2013, 12:02 p.m., Andrew Kaplanov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/8916/ > ----------------------------------------------------------- > > (Updated Jan. 10, 2013, 12:02 p.m.) > > > Review request for wave. > > > Description > ------- > > Realizes the possibility of switching and storing the user's language. > Most phrases have been translated into several languages. > > > Diffs > ----- > > > ./proto_src/org/waveprotocol/box/server/persistence/protos/ProtoAccountStoreData.java > 1430452 > ./src/org/waveprotocol/box/server/ServerMain.java 1430452 > ./src/org/waveprotocol/box/server/account/HumanAccountData.java 1430452 > ./src/org/waveprotocol/box/server/account/HumanAccountDataImpl.java 1430452 > ./src/org/waveprotocol/box/server/gxp/TopBar.gxp 1430452 > ./src/org/waveprotocol/box/server/gxp/WaveClientPage.gxp 1430452 > > ./src/org/waveprotocol/box/server/persistence/protos/ProtoAccountDataSerializer.java > 1430452 > ./src/org/waveprotocol/box/server/persistence/protos/account-store.proto > 1430452 > ./src/org/waveprotocol/box/server/rpc/LocaleServlet.java PRE-CREATION > ./src/org/waveprotocol/box/server/rpc/WaveClientServlet.java 1430452 > ./src/org/waveprotocol/box/server/util/URLUtil.java PRE-CREATION > ./src/org/waveprotocol/box/webclient/WebClientProd.gwt.xml 1430452 > ./src/org/waveprotocol/box/webclient/client/LocaleService.java PRE-CREATION > ./src/org/waveprotocol/box/webclient/client/RemoteLocaleService.java > PRE-CREATION > ./src/org/waveprotocol/box/webclient/client/SavedStateIndicator.java > 1430452 > ./src/org/waveprotocol/box/webclient/client/Session.java 1430452 > ./src/org/waveprotocol/box/webclient/client/WebClient.java 1430452 > ./src/org/waveprotocol/box/webclient/client/i18n/SavedStateMessages.java > PRE-CREATION > > ./src/org/waveprotocol/box/webclient/client/i18n/SavedStateMessages_en.properties > PRE-CREATION > > ./src/org/waveprotocol/box/webclient/client/i18n/SavedStateMessages_ru.properties > PRE-CREATION > ./src/org/waveprotocol/box/webclient/client/i18n/SessionMessages.java > PRE-CREATION > > ./src/org/waveprotocol/box/webclient/client/i18n/SessionMessages_en.properties > PRE-CREATION > > ./src/org/waveprotocol/box/webclient/client/i18n/SessionMessages_fr.properties > PRE-CREATION > > ./src/org/waveprotocol/box/webclient/client/i18n/SessionMessages_ru.properties > PRE-CREATION > ./src/org/waveprotocol/box/webclient/client/i18n/WebClientMessages.java > PRE-CREATION > > ./src/org/waveprotocol/box/webclient/client/i18n/WebClientMessages_en.properties > PRE-CREATION > > ./src/org/waveprotocol/box/webclient/client/i18n/WebClientMessages_fr.properties > PRE-CREATION > > ./src/org/waveprotocol/box/webclient/client/i18n/WebClientMessages_ru.properties > PRE-CREATION > > ./src/org/waveprotocol/box/webclient/client/i18n/WebClientMessages_sl.properties > PRE-CREATION > ./src/org/waveprotocol/box/webclient/search/DigestDomImpl.java 1430452 > ./src/org/waveprotocol/box/webclient/search/SearchPresenter.java 1430452 > ./src/org/waveprotocol/box/webclient/search/SearchWidget.ui.xml 1430452 > ./src/org/waveprotocol/box/webclient/search/i18n/DigestDomMessages.java > PRE-CREATION > > ./src/org/waveprotocol/box/webclient/search/i18n/DigestDomMessages_en.properties > PRE-CREATION > > ./src/org/waveprotocol/box/webclient/search/i18n/DigestDomMessages_fr.properties > PRE-CREATION > > ./src/org/waveprotocol/box/webclient/search/i18n/DigestDomMessages_ru.properties > PRE-CREATION > > ./src/org/waveprotocol/box/webclient/search/i18n/SearchPresenterMessages.java > PRE-CREATION > > ./src/org/waveprotocol/box/webclient/search/i18n/SearchPresenterMessages_en.properties > PRE-CREATION > > ./src/org/waveprotocol/box/webclient/search/i18n/SearchPresenterMessages_fr.properties > PRE-CREATION > > ./src/org/waveprotocol/box/webclient/search/i18n/SearchPresenterMessages_ru.properties > PRE-CREATION > > ./src/org/waveprotocol/box/webclient/search/i18n/SearchPresenterMessages_sl.properties > PRE-CREATION > ./src/org/waveprotocol/box/webclient/search/i18n/SearchWidgetMessages.java > PRE-CREATION > > ./src/org/waveprotocol/box/webclient/search/i18n/SearchWidgetMessages_en.properties > PRE-CREATION > > ./src/org/waveprotocol/box/webclient/search/i18n/SearchWidgetMessages_ru.properties > PRE-CREATION > > ./src/org/waveprotocol/box/webclient/widget/error/ErrorIndicatorWidget.ui.xml > 1430452 > ./src/org/waveprotocol/box/webclient/widget/error/i18n/ErrorMessages.java > PRE-CREATION > > ./src/org/waveprotocol/box/webclient/widget/error/i18n/ErrorMessages_en.properties > PRE-CREATION > > ./src/org/waveprotocol/box/webclient/widget/error/i18n/ErrorMessages_fr.properties > PRE-CREATION > > ./src/org/waveprotocol/box/webclient/widget/error/i18n/ErrorMessages_ru.properties > PRE-CREATION > ./src/org/waveprotocol/box/webclient/widget/loading/LoadingIndicator.ui.xml > 1430452 > > ./src/org/waveprotocol/box/webclient/widget/loading/i18n/LoadingMessages.java > PRE-CREATION > > ./src/org/waveprotocol/box/webclient/widget/loading/i18n/LoadingMessages_en.properties > PRE-CREATION > > ./src/org/waveprotocol/box/webclient/widget/loading/i18n/LoadingMessages_fr.properties > PRE-CREATION > > ./src/org/waveprotocol/box/webclient/widget/loading/i18n/LoadingMessages_ru.properties > PRE-CREATION > ./src/org/waveprotocol/wave/client/StageThree.java 1430452 > > ./src/org/waveprotocol/wave/client/wavepanel/event/EventDispatcherPanel.java > 1430452 > > ./src/org/waveprotocol/wave/client/wavepanel/event/EventHandlerRegistry.java > 1430452 > ./src/org/waveprotocol/wave/client/wavepanel/event/WaveChangeHandler.java > PRE-CREATION > ./src/org/waveprotocol/wave/client/wavepanel/impl/edit/ActionsImpl.java > 1430452 > > ./src/org/waveprotocol/wave/client/wavepanel/impl/edit/ParticipantController.java > 1430452 > > ./src/org/waveprotocol/wave/client/wavepanel/impl/edit/i18n/ActionMessages.java > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/impl/edit/i18n/ActionMessages_en.properties > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/impl/edit/i18n/ActionMessages_fr.properties > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/impl/edit/i18n/ActionMessages_ru.properties > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/impl/edit/i18n/ParticipantMessages.java > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/impl/edit/i18n/ParticipantMessages_en.properties > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/impl/edit/i18n/ParticipantMessages_fr.properties > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/impl/edit/i18n/ParticipantMessages_ru.properties > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/impl/edit/i18n/ParticipantMessages_sl.properties > PRE-CREATION > ./src/org/waveprotocol/wave/client/wavepanel/impl/menu/MenuController.java > 1430452 > > ./src/org/waveprotocol/wave/client/wavepanel/impl/menu/i18n/MenuMessages.java > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/impl/menu/i18n/MenuMessages_en.properties > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/impl/menu/i18n/MenuMessages_fr.properties > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/impl/menu/i18n/MenuMessages_ru.properties > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/impl/menu/i18n/MenuMessages_sl.properties > PRE-CREATION > ./src/org/waveprotocol/wave/client/wavepanel/impl/toolbar/LinkerHelper.java > 1430452 > ./src/org/waveprotocol/wave/client/wavepanel/impl/toolbar/ViewToolbar.java > 1430452 > > ./src/org/waveprotocol/wave/client/wavepanel/impl/toolbar/attachment/AttachmentPopupWidget.ui.xml > 1430452 > > ./src/org/waveprotocol/wave/client/wavepanel/impl/toolbar/attachment/i18n/AttachmentMessages.java > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/impl/toolbar/attachment/i18n/AttachmentMessages_en.properties > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/impl/toolbar/attachment/i18n/AttachmentMessages_fr.properties > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/impl/toolbar/attachment/i18n/AttachmentMessages_ru.properties > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/impl/toolbar/gadget/GadgetSelectorWidget.java > 1430452 > > ./src/org/waveprotocol/wave/client/wavepanel/impl/toolbar/gadget/GadgetSelectorWidget.ui.xml > 1430452 > > ./src/org/waveprotocol/wave/client/wavepanel/impl/toolbar/gadget/i18n/GadgetCategoryMessages.java > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/impl/toolbar/gadget/i18n/GadgetCategoryMessages_en.properties > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/impl/toolbar/gadget/i18n/GadgetCategoryMessages_fr.properties > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/impl/toolbar/gadget/i18n/GadgetCategoryMessages_ru.properties > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/impl/toolbar/gadget/i18n/GadgetSelectorMessages.java > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/impl/toolbar/gadget/i18n/GadgetSelectorMessages_en.properties > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/impl/toolbar/gadget/i18n/GadgetSelectorMessages_fr.properties > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/impl/toolbar/gadget/i18n/GadgetSelectorMessages_ru.properties > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/impl/toolbar/i18n/LinkerMessages.java > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/impl/toolbar/i18n/LinkerMessages_en.properties > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/impl/toolbar/i18n/LinkerMessages_fr.properties > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/impl/toolbar/i18n/LinkerMessages_ru.properties > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/impl/toolbar/i18n/ToolbarMessages.java > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/impl/toolbar/i18n/ToolbarMessages_en.properties > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/impl/toolbar/i18n/ToolbarMessages_fr.properties > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/impl/toolbar/i18n/ToolbarMessages_ru.properties > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/impl/toolbar/i18n/ToolbarMessages_sl.properties > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/view/dom/full/BlipLinkPopupWidget.ui.xml > 1430452 > > ./src/org/waveprotocol/wave/client/wavepanel/view/dom/full/BlipMetaViewBuilder.java > 1430452 > > ./src/org/waveprotocol/wave/client/wavepanel/view/dom/full/ParticipantsViewBuilder.java > 1430452 > > ./src/org/waveprotocol/wave/client/wavepanel/view/dom/full/ReplyBoxViewBuilder.java > 1430452 > > ./src/org/waveprotocol/wave/client/wavepanel/view/dom/full/WavePanelResourceLoader.java > 1430452 > > ./src/org/waveprotocol/wave/client/wavepanel/view/dom/full/i18n/BlipLinkMessages.java > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/view/dom/full/i18n/BlipLinkMessages_en.properties > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/view/dom/full/i18n/BlipLinkMessages_fr.properties > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/view/dom/full/i18n/BlipLinkMessages_ru.properties > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/view/dom/full/i18n/BlipMessages.java > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/view/dom/full/i18n/BlipMessages_en.properties > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/view/dom/full/i18n/BlipMessages_fr.properties > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/view/dom/full/i18n/BlipMessages_ru.properties > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/view/dom/full/i18n/BlipMessages_sl.properties > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/view/dom/full/i18n/ParticipantMessages.java > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/view/dom/full/i18n/ParticipantMessages_en.properties > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/view/dom/full/i18n/ParticipantMessages_fr.properties > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/view/dom/full/i18n/ParticipantMessages_ru.properties > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/view/dom/full/i18n/ParticipantMessages_sl.properties > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/view/dom/full/i18n/ReplyBoxMessages.java > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/view/dom/full/i18n/ReplyBoxMessages_en.properties > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/view/dom/full/i18n/ReplyBoxMessages_fr.properties > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/view/dom/full/i18n/ReplyBoxMessages_ru.properties > PRE-CREATION > > ./src/org/waveprotocol/wave/client/wavepanel/view/dom/full/i18n/ReplyBoxMessages_sl.properties > PRE-CREATION > ./src/org/waveprotocol/wave/client/widget/profile/ProfilePopupWidget.java > 1430452 > ./src/org/waveprotocol/wave/client/widget/profile/ProfilePopupWidget.ui.xml > 1430452 > > ./src/org/waveprotocol/wave/client/widget/profile/i18n/ProfilePopupMessages.java > PRE-CREATION > > ./src/org/waveprotocol/wave/client/widget/profile/i18n/ProfilePopupMessages_en.properties > PRE-CREATION > > ./src/org/waveprotocol/wave/client/widget/profile/i18n/ProfilePopupMessages_fr.properties > PRE-CREATION > > ./src/org/waveprotocol/wave/client/widget/profile/i18n/ProfilePopupMessages_ru.properties > PRE-CREATION > > ./src/org/waveprotocol/wave/client/widget/profile/i18n/ProfilePopupMessages_sl.properties > PRE-CREATION > > ./test/org/waveprotocol/wave/client/wavepanel/view/dom/full/BlipViewBuilderTest.java > 1430452 > > ./test/org/waveprotocol/wave/client/wavepanel/view/dom/full/ReplyBoxBuilderTest.java > 1430452 > > ./test/org/waveprotocol/wave/client/wavepanel/view/dom/full/RootThreadViewBuilderTest.java > 1430452 > > ./test/org/waveprotocol/wave/client/wavepanel/view/dom/full/UiBuilderTestHelper.java > 1430452 > ./war/static/earth.png UNKNOWN > > Diff: https://reviews.apache.org/r/8916/diff/ > > > Testing > ------- > > > Thanks, > > Andrew Kaplanov > >
