http://gwt-code-reviews.appspot.com/1546803/diff/1/user/src/com/google/gwt/user/cellview/client/RowHoverEvent.java
File user/src/com/google/gwt/user/cellview/client/RowHoverEvent.java
(right):

http://gwt-code-reviews.appspot.com/1546803/diff/1/user/src/com/google/gwt/user/cellview/client/RowHoverEvent.java#newcode57
user/src/com/google/gwt/user/cellview/client/RowHoverEvent.java:57:
public static RowHoverEvent fire(HasHandlers source, TableRowElement
hoveringRow,
This is a breaking change.  Can you add an overload that takes the Event
parameter, and pass null by default from the original version of fire().

http://gwt-code-reviews.appspot.com/1546803/diff/1/user/src/com/google/gwt/user/cellview/client/RowHoverEvent.java#newcode92
user/src/com/google/gwt/user/cellview/client/RowHoverEvent.java:92:
protected RowHoverEvent(TableRowElement hoveringRow, Event browserEvent,
boolean isUnHover) {
Same here, overload the constructor to avoid a breaking change.

http://gwt-code-reviews.appspot.com/1546803/

--
http://groups.google.com/group/Google-Web-Toolkit-Contributors

Reply via email to