Revision: 10273
Author: [email protected]
Date: Fri Jun 3 09:52:12 2011
Log: Make private two new methods accidentally exposed in
ActivityManager
Review at http://gwt-code-reviews.appspot.com/1449811
Review by: [email protected]
http://code.google.com/p/google-web-toolkit/source/detail?r=10273
Modified:
/trunk/user/src/com/google/gwt/activity/shared/ActivityManager.java
=======================================
--- /trunk/user/src/com/google/gwt/activity/shared/ActivityManager.java Fri
Jun 3 08:57:30 2011
+++ /trunk/user/src/com/google/gwt/activity/shared/ActivityManager.java Fri
Jun 3 09:52:12 2011
@@ -181,7 +181,25 @@
}
}
- public Throwable tryStart() {
+ private Activity getNextActivity(PlaceChangeEvent event) {
+ if (display == null) {
+ /*
+ * Display may have been nulled during PlaceChangeEvent dispatch.
Don't
+ * bother the mapper, just return a null to ensure we shut down the
+ * current activity
+ */
+ return null;
+ }
+ return mapper.getActivity(event.getNewPlace());
+ }
+
+ private void showWidget(IsWidget view) {
+ if (display != null) {
+ display.setWidget(view);
+ }
+ }
+
+ private Throwable tryStart() {
Throwable caughtOnStart = null;
try {
/* Wrap the actual display with a per-call instance
@@ -196,7 +214,7 @@
return caughtOnStart;
}
- public Throwable tryStopOrCancel(boolean stop) {
+ private Throwable tryStopOrCancel(boolean stop) {
Throwable caughtOnStop = null;
try {
if (stop) {
@@ -215,24 +233,6 @@
}
return caughtOnStop;
}
-
- private Activity getNextActivity(PlaceChangeEvent event) {
- if (display == null) {
- /*
- * Display may have been nulled during PlaceChangeEvent dispatch.
Don't
- * bother the mapper, just return a null to ensure we shut down the
- * current activity
- */
- return null;
- }
- return mapper.getActivity(event.getNewPlace());
- }
-
- private void showWidget(IsWidget view) {
- if (display != null) {
- display.setWidget(view);
- }
- }
private void updateHandlers(boolean activate) {
if (activate) {
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
