Reviewers: amitmanjhi,

Description:
Breaks the circular dependencies between app, user, valuestore and
requestfactory. Does so by eliminating the app.util package by:

* moving IsWidget to com.google.gwt.user.client.ui
* moving everything else, all of which is rendering and parsing
related, to a new package, com.google.gwt.input

and by moving the WriteOperations enum out of the RequestFactory
interface and into ValueStore.

Also pares down the inherits lines in ValueStore and RequestFactory to
make it clearer exactly what those modules require from the rest of
GWT.

Review by: amitman...@google.com

Please review this at http://gwt-code-reviews.appspot.com/522803/show

Affected files:
  M /bikeshed/src/com/google/gwt/app/client/CellListPlacePickerView.java
  M /bikeshed/src/com/google/gwt/app/place/Activity.java
  M /bikeshed/src/com/google/gwt/app/place/ActivityManager.java
  M /bikeshed/src/com/google/gwt/app/place/PlacePicker.java
  M /bikeshed/src/com/google/gwt/app/place/PlacePickerView.java
  D /bikeshed/src/com/google/gwt/app/util/BooleanParser.java
  D /bikeshed/src/com/google/gwt/app/util/BooleanRenderer.java
  D /bikeshed/src/com/google/gwt/app/util/DateTimeFormatRenderer.java
  D /bikeshed/src/com/google/gwt/app/util/DoubleParser.java
  D /bikeshed/src/com/google/gwt/app/util/DoubleRenderer.java
  D /bikeshed/src/com/google/gwt/app/util/IntegerParser.java
  D /bikeshed/src/com/google/gwt/app/util/IntegerRenderer.java
  D /bikeshed/src/com/google/gwt/app/util/IsWidget.java
  D /bikeshed/src/com/google/gwt/app/util/LongParser.java
  D /bikeshed/src/com/google/gwt/app/util/LongRenderer.java
  D /bikeshed/src/com/google/gwt/app/util/ParseException.java
  D /bikeshed/src/com/google/gwt/app/util/Parser.java
  D /bikeshed/src/com/google/gwt/app/util/PassthroughParser.java
  D /bikeshed/src/com/google/gwt/app/util/PassthroughRenderer.java
  D /bikeshed/src/com/google/gwt/app/util/Renderer.java
  D /bikeshed/src/com/google/gwt/app/util/package.html
  A /bikeshed/src/com/google/gwt/input/Input.gwt.xml
  A /bikeshed/src/com/google/gwt/input/shared/BooleanParser.java
  A /bikeshed/src/com/google/gwt/input/shared/BooleanRenderer.java
  A /bikeshed/src/com/google/gwt/input/shared/DateTimeFormatRenderer.java
  A /bikeshed/src/com/google/gwt/input/shared/DoubleParser.java
  A /bikeshed/src/com/google/gwt/input/shared/DoubleRenderer.java
  A /bikeshed/src/com/google/gwt/input/shared/IntegerParser.java
  A /bikeshed/src/com/google/gwt/input/shared/IntegerRenderer.java
  A /bikeshed/src/com/google/gwt/input/shared/LongParser.java
  A /bikeshed/src/com/google/gwt/input/shared/LongRenderer.java
  A /bikeshed/src/com/google/gwt/input/shared/ParseException.java
  A /bikeshed/src/com/google/gwt/input/shared/Parser.java
  A /bikeshed/src/com/google/gwt/input/shared/PassthroughParser.java
  A /bikeshed/src/com/google/gwt/input/shared/PassthroughRenderer.java
  A /bikeshed/src/com/google/gwt/input/shared/Renderer.java
  M /bikeshed/src/com/google/gwt/requestfactory/RequestFactory.gwt.xml
M /bikeshed/src/com/google/gwt/requestfactory/client/impl/RequestFactoryJsonImpl.java M /bikeshed/src/com/google/gwt/requestfactory/rebind/RequestFactoryGenerator.java M /bikeshed/src/com/google/gwt/requestfactory/server/RequestFactoryServlet.java M /bikeshed/src/com/google/gwt/requestfactory/server/SampleDataPopulator.java
  M /bikeshed/src/com/google/gwt/requestfactory/shared/RequestFactory.java
M /bikeshed/src/com/google/gwt/requestfactory/shared/impl/RequestDataManager.java
  M /bikeshed/src/com/google/gwt/sample/expenses/gwt/client/Scaffold.java
M /bikeshed/src/com/google/gwt/sample/expenses/gwt/client/ScaffoldMobile.java M /bikeshed/src/com/google/gwt/sample/expenses/gwt/request/EmployeeRecordChanged.java M /bikeshed/src/com/google/gwt/sample/expenses/gwt/request/ExpenseRecordChanged.java M /bikeshed/src/com/google/gwt/sample/expenses/gwt/request/ReportRecordChanged.java M /bikeshed/src/com/google/gwt/sample/expenses/gwt/ui/ScaffoldListPlaceRenderer.java M /bikeshed/src/com/google/gwt/sample/expenses/gwt/ui/report/ReportListView.java
  M /bikeshed/src/com/google/gwt/sample/expenses/server/DataGenerator.java
  M /bikeshed/src/com/google/gwt/user/User.gwt.xml
  M /bikeshed/src/com/google/gwt/user/client/ui/DoubleBox.java
  M /bikeshed/src/com/google/gwt/user/client/ui/IntegerBox.java
  A /bikeshed/src/com/google/gwt/user/client/ui/IsWidget.java
  M /bikeshed/src/com/google/gwt/user/client/ui/LongBox.java
  M /bikeshed/src/com/google/gwt/user/client/ui/TextBoxBase.java
  M /bikeshed/src/com/google/gwt/user/client/ui/ValueBox.java
  M /bikeshed/src/com/google/gwt/user/client/ui/ValueBoxBase.java
  M /bikeshed/src/com/google/gwt/valuestore/ValueStore.gwt.xml
M /bikeshed/src/com/google/gwt/valuestore/client/DeltaValueStoreJsonImpl.java
  M /bikeshed/src/com/google/gwt/valuestore/client/ValueStoreJsonImpl.java
  M /bikeshed/src/com/google/gwt/valuestore/shared/RecordChangedEvent.java
  A /bikeshed/src/com/google/gwt/valuestore/shared/WriteOperation.java
  M /bikeshed/src/com/google/gwt/valuestore/shared/impl/RecordSchema.java
M /bikeshed/src/com/google/gwt/valuestore/ui/AbstractRecordListActivity.java
  M /bikeshed/src/com/google/gwt/valuestore/ui/PropertyColumn.java
  M /bikeshed/src/com/google/gwt/valuestore/ui/RecordDetailsView.java
  M /bikeshed/src/com/google/gwt/valuestore/ui/RecordEditView.java
  M /bikeshed/src/com/google/gwt/valuestore/ui/RecordListView.java
  M /bikeshed/test/com/google/gwt/app/place/ActivityManagerTest.java


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

Reply via email to