a118d40c85a4b53fe824bd5e4231114982485434 - gwt

commit: a118d40c85a4b53fe824bd5e4231114982485434
[log]
author: t.broyer@gmail.com <t.broyer@gmail.com@8db76d5a-ed1c-0410-87a9-c151d255dfc7>
Sun Sep 19 00:22:16 2010 +0000
committer: t.broyer@gmail.com <t.broyer@gmail.com@8db76d5a-ed1c-0410-87a9-c151d255dfc7>
Sun Sep 19 00:22:16 2010 +0000
tree: e8060e713902fd0ebbeb3ebbc58224fc08f305c6
parent: c7f99dd71e96f4db276ebe7a3b624017c877334f [diff]

Following up on https://jira.springsource.org/browse/ROO-1276, here's the
proposed alternate design, separating the "keep history and places in sync" and
"map places to/from history tokens" concerns into a concrete PlaceHistoryHandler
class and a PlaceHistoryMapper interface whose implementation can be generated
based on annotations.

This patch superseeds http://gwt-code-reviews.appspot.com/824801/show and
integrates the changes from http://gwt-code-reviews.appspot.com/827801/show (if
it weren't the handler -> mapper renaming, they could have been kept separate).

Derived from http://gwt-code-reviews.appspot.com/845802, but without the
ActivityMapperGenerator bits.

Review by: rjrjr@google.com
Patch by: t.broyer@gmail.com


git-svn-id: https://google-web-toolkit.googlecode.com/svn/trunk@8817 8db76d5a-ed1c-0410-87a9-c151d255dfc7

user/src/com/google/gwt/app/App.gwt.xml[diff]
user/src/com/google/gwt/app/place/AbstractPlaceHistoryHandler.java[Deleted - diff]
user/src/com/google/gwt/app/place/PlaceHistoryHandler.java[diff]
user/src/com/google/gwt/app/place/PlaceHistoryMapper.java[Copied from user/src/com/google/gwt/app/place/PlaceHistoryHandlerWithFactory.java - diff]
user/src/com/google/gwt/app/place/PlaceHistoryMapperWithFactory.java[Renamed from user/src/com/google/gwt/app/place/PlaceHistoryHandlerWithFactory.java - diff]
user/src/com/google/gwt/app/place/Prefix.java[diff]
user/src/com/google/gwt/app/place/WithTokenizers.java[diff]
user/src/com/google/gwt/app/place/impl/AbstractPlaceHistoryMapper.java[Added - diff]
user/src/com/google/gwt/app/rebind/MostToLeastDerivedPlaceTypeComparator.java[Added - diff]
user/src/com/google/gwt/app/rebind/PlaceHistoryGeneratorContext.java[diff]
user/src/com/google/gwt/app/rebind/PlaceHistoryMapperGenerator.java[Renamed from user/src/com/google/gwt/app/rebind/PlaceHistoryHandlerGenerator.java - diff]
user/test/com/google/gwt/app/AppJreSuite.java[diff]
user/test/com/google/gwt/app/AppSuite.gwt.xml[Deleted - diff]
user/test/com/google/gwt/app/AppSuite.java[diff]
user/test/com/google/gwt/app/place/AbstractPlaceHistoryHandlerTest.java[Deleted - diff]
user/test/com/google/gwt/app/place/PlaceHistoryHandlerTest.java[Added - diff]
user/test/com/google/gwt/app/place/impl/PlaceHistoryMapperGeneratorTest.java[Renamed from user/test/com/google/gwt/app/place/PlaceHistoryHandlerGeneratorTest.java - diff]
user/test/com/google/gwt/app/place/testplacemappers/NoFactory.java[Renamed from user/test/com/google/gwt/app/place/testplacehandler/NoFactory.java - diff]
user/test/com/google/gwt/app/place/testplacemappers/WithFactory.java[Renamed from user/test/com/google/gwt/app/place/testplacehandler/WithFactory.java - diff]
user/test/com/google/gwt/app/place/testplaces/Place1.java[diff]
user/test/com/google/gwt/app/place/testplaces/Place2.java[diff]
user/test/com/google/gwt/app/place/testplaces/Place3.java[diff]
user/test/com/google/gwt/app/place/testplaces/Place4.java[diff]
user/test/com/google/gwt/app/place/testplaces/Place5.java[Copied from user/src/com/google/gwt/app/place/PlaceHistoryHandlerWithFactory.java - diff]
user/test/com/google/gwt/app/place/testplaces/Tokenizer2.java[diff]
user/test/com/google/gwt/app/place/testplaces/Tokenizer3.java[diff]
user/test/com/google/gwt/app/place/testplaces/Tokenizer4.java[diff]
user/test/com/google/gwt/app/place/testplaces/TokenizerFactory.java[diff]
user/test/com/google/gwt/app/rebind/EmptyMockJavaResource.java[Added - diff]
user/test/com/google/gwt/app/rebind/MostToLeastDerivedPlaceTypeComparatorTest.java[Added - diff]
user/test/com/google/gwt/app/rebind/PlaceHistoryGeneratorContextTest.java[Added - diff]
user/test/com/google/gwt/app/rebind/RealJavaResource.java[Added - diff]
user/test/com/google/gwt/editor/rebind/model/EditorModelTest.java[diff]

33 files changed

tree: e8060e713902fd0ebbeb3ebbc58224fc08f305c6