-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/18941/
-----------------------------------------------------------

(Updated March 8, 2014, 12:42 p.m.)


Review request for wave, Ali Lown, Pablo Ojanguren, and Yuri Zelikov.


Changes
-------

Just removed some unnecessary changes in build.xml.


Bugs: WAVE-384
    https://issues.apache.org/jira/browse/WAVE-384


Repository: wave


Description
-------

This patch add the GWT superdev mode to 'waveharness' and 'editor' that now 
works again. A refactorization is needed because some new functionalities had 
broken these GWT dev tools (WAVE-384).

Note: GWT superdev mode in the full client (ant superdev-gwt) does not work yet 
because we need to replace socketio-java GWT dependency with an alternative 
(like atmosphere) or patched it with
  
https://github.com/vjrj/Socket.IO-Java/commit/f7010aee1f5cc5adea0aab3f27e48c37d1b10b5e
I think is more interesting the first option and seems that pablojan is working 
on it, because socketio-java update to jetty 9 websocket (#WAVE-405) version is 
not well documented:
https://github.com/jetty-project/jetty-documentation/issues/12
Anyway, when the replacement is ready, we can just uncomment the xsiframe 
linker in WebClientDev.gwt.xml and Editor.gwt.xml.


Diffs (updated)
-----

  build.xml 0a1b23c 
  src/org/waveprotocol/box/webclient/WebClientDev.gwt.xml da32dd7 
  src/org/waveprotocol/box/webclient/client/ClientEvents.java 8b8e411 
  src/org/waveprotocol/box/webclient/client/DebugMessagePanel.java def986e 
  src/org/waveprotocol/box/webclient/client/HistoryChangeListener.java ddcdb5d 
  src/org/waveprotocol/box/webclient/client/HistoryProvider.java f207b83 
  src/org/waveprotocol/box/webclient/client/RemoteWaveViewService.java 8d4ff15 
  src/org/waveprotocol/box/webclient/client/SavedStateIndicator.java 3ee5c74 
  src/org/waveprotocol/box/webclient/client/SimpleWaveStore.java 83484c4 
  src/org/waveprotocol/box/webclient/client/StagesProvider.java dfca652 
  src/org/waveprotocol/box/webclient/client/WaveWebSocketClient.java 3bb39a2 
  src/org/waveprotocol/box/webclient/client/WebClient.java d3e3b49 
  src/org/waveprotocol/box/webclient/client/WindowTitleHandler.java ff1654f 
  src/org/waveprotocol/box/webclient/client/events/DebugMessageEvent.java 
6245489 
  
src/org/waveprotocol/box/webclient/client/events/DebugMessageEventHandler.java 
57f4659 
  src/org/waveprotocol/box/webclient/client/events/Log.java af132fe 
  src/org/waveprotocol/box/webclient/client/events/NetworkStatusEvent.java 
e495cc0 
  
src/org/waveprotocol/box/webclient/client/events/NetworkStatusEventHandler.java 
034d062 
  src/org/waveprotocol/box/webclient/client/events/WaveCreationEvent.java 
ec59ccc 
  
src/org/waveprotocol/box/webclient/client/events/WaveCreationEventHandler.java 
3ddf1a0 
  src/org/waveprotocol/box/webclient/client/events/WaveOpenEvent.java b5dcd75 
  src/org/waveprotocol/box/webclient/client/events/WaveOpenEventHandler.java 
f440d29 
  src/org/waveprotocol/box/webclient/client/events/WaveSelectionEvent.java 
a86469d 
  
src/org/waveprotocol/box/webclient/client/events/WaveSelectionEventHandler.java 
a35242c 
  src/org/waveprotocol/box/webclient/search/SimpleSearch.java c79af80 
  src/org/waveprotocol/box/webclient/search/WaveBasedDigest.java 8c04db8 
  src/org/waveprotocol/box/webclient/search/WaveContext.java c8deb2a 
  src/org/waveprotocol/box/webclient/search/WaveStore.java 2290737 
  src/org/waveprotocol/wave/client/StageTwo.java 557e838 
  src/org/waveprotocol/wave/client/common/util/UserAgentStaticProperties.java 
0bca984 
  
src/org/waveprotocol/wave/client/doodad/attachment/AttachmentManagerProvider.java
 PRE-CREATION 
  src/org/waveprotocol/wave/client/editor/Editor.gwt.xml 78f3c26 
  src/org/waveprotocol/wave/client/events/ClientEvents.java PRE-CREATION 
  src/org/waveprotocol/wave/client/events/DebugMessageEvent.java PRE-CREATION 
  src/org/waveprotocol/wave/client/events/DebugMessageEventHandler.java 
PRE-CREATION 
  src/org/waveprotocol/wave/client/events/Log.java PRE-CREATION 
  src/org/waveprotocol/wave/client/events/NetworkStatusEvent.java PRE-CREATION 
  src/org/waveprotocol/wave/client/events/NetworkStatusEventHandler.java 
PRE-CREATION 
  src/org/waveprotocol/wave/client/events/WaveCreationEvent.java PRE-CREATION 
  src/org/waveprotocol/wave/client/events/WaveCreationEventHandler.java 
PRE-CREATION 
  src/org/waveprotocol/wave/client/events/WaveOpenEvent.java PRE-CREATION 
  src/org/waveprotocol/wave/client/events/WaveOpenEventHandler.java 
PRE-CREATION 
  src/org/waveprotocol/wave/client/events/WaveSelectionEvent.java PRE-CREATION 
  src/org/waveprotocol/wave/client/events/WaveSelectionEventHandler.java 
PRE-CREATION 
  src/org/waveprotocol/wave/client/testing/UndercurrentHarness.gwt.xml e48fd59 
  src/org/waveprotocol/wave/client/testing/UndercurrentHarness.java 22f7df6 
  
src/org/waveprotocol/wave/client/wavepanel/impl/edit/ParticipantController.java 
21995a6 
  src/org/waveprotocol/wave/model/conversation/TitleHelper.java 9402d67 
  src/org/waveprotocol/wave/model/document/WaveContext.java PRE-CREATION 

Diff: https://reviews.apache.org/r/18941/diff/


Testing
-------

Run:
ant waveharness-superdev
Add the bookmarks in: http://localhost:9876/
and play with
http://localhost:9876/waveharness/UndercurrentHarness.html
so some code changes (for instance in UndercurrentHarness.java), click in "Dev 
mode on" in the bookmarks and recompile the module.

Same with:
ant editor-superdev


Thanks,

Vicente J. Ruiz Jurado

Reply via email to