Revision: 9366
Author: [email protected]
Date: Tue Dec  7 08:36:25 2010
Log: Cherry picking CL/18454817 from trunk into the release branch
Public: Cherry picking issue 1187801 / svn 9358 into the release branch.

http://code.google.com/p/google-web-toolkit/source/detail?r=9366

Modified:
 /releases/2.1/user/src/com/google/gwt/event/shared/HandlerManager.java
 /releases/2.1/user/src/com/google/gwt/user/client/ui/TabPanel.java
 /releases/2.1/user/src/com/google/gwt/user/client/ui/Widget.java

=======================================
--- /releases/2.1/user/src/com/google/gwt/event/shared/HandlerManager.java Wed Oct 13 06:40:20 2010 +++ /releases/2.1/user/src/com/google/gwt/event/shared/HandlerManager.java Tue Dec 7 08:36:25 2010
@@ -19,12 +19,16 @@

 /**
* Manager responsible for adding handlers to event sources and firing those
- * handlers on passed in events. Primitive ancestor of {...@link EventBus},
- * and used at the core of {com.google.gwt.user.client.ui.Widget}.
- *
- * @deprecated use {...@link SimpleEventBus}.
+ * handlers on passed in events. Primitive ancestor of {...@link EventBus}, and
+ * used at the core of {com.google.gwt.user.client.ui.Widget}.
+ * <p>
+ * While widget authors should continue to use
+ * {...@link com.google.gwt.user.client.ui.Widget#addDomHandler(EventHandler, com.google.gwt.event.dom.client.DomEvent.Type)}
+ * and
+ * {...@link com.google.gwt.user.client.ui.Widget#addHandler(EventHandler, Type)}, + * application developers are strongly discouraged from using a HandlerManager
+ * instance as a global event dispatch mechanism.
  */
-...@deprecated
 public class HandlerManager implements HasHandlers {

   private final SimpleEventBus eventBus;
@@ -50,6 +54,7 @@
    * @param source the event source
    * @param fireInReverseOrder true to fire handlers in reverse order
    */
+  @SuppressWarnings("deprecation")
   public HandlerManager(Object source, boolean fireInReverseOrder) {
     eventBus = new SimpleEventBus(fireInReverseOrder);
     this.source = source;
@@ -114,6 +119,7 @@
    * @param type the handler's event type
    * @return the given handler
    */
+  @SuppressWarnings("deprecation")
public <H extends EventHandler> H getHandler(GwtEvent.Type<H> type, int index) {
     return eventBus.getHandler(type, index);
   }
@@ -124,6 +130,7 @@
    * @param type the event type
    * @return the number of registered handlers
    */
+  @SuppressWarnings("deprecation")
   public int getHandlerCount(Type<?> type) {
     return eventBus.getHandlerCount(type);
   }
@@ -134,6 +141,7 @@
    * @param e the event type
    * @return whether the given event type is handled
    */
+  @SuppressWarnings("deprecation")
   public boolean isEventHandled(Type<?> e) {
     return eventBus.isEventHandled(e);
   }
=======================================
--- /releases/2.1/user/src/com/google/gwt/user/client/ui/TabPanel.java Wed Oct 13 06:40:20 2010 +++ /releases/2.1/user/src/com/google/gwt/user/client/ui/TabPanel.java Tue Dec 7 08:36:25 2010
@@ -58,18 +58,13 @@
  * <h3>Example</h3>
  * {...@example com.google.gwt.examples.TabPanelExample}
  * </p>
- *
- * @deprecated Use {...@link TabLayoutPanel} instead, but understand that it is - * not a drop in replacement for this class. It requires standards - * mode, and is most easily used under a {...@link RootLayoutPanel} (as
- *             opposed to a {...@link RootPanel}
  *
  * @see TabLayoutPanel
  */

 // Cannot do anything about tab panel implementing TabListener until next
 // release
-...@deprecated
+...@suppresswarnings("deprecation")
 public class TabPanel extends Composite implements TabListener,
     SourcesTabEvents, HasWidgets, HasAnimation, IndexedPanel.ForIsWidget,
     HasBeforeSelectionHandlers<Integer>, HasSelectionHandlers<Integer> {
=======================================
--- /releases/2.1/user/src/com/google/gwt/user/client/ui/Widget.java Thu Oct 14 11:27:30 2010 +++ /releases/2.1/user/src/com/google/gwt/user/client/ui/Widget.java Tue Dec 7 08:36:25 2010
@@ -55,7 +55,6 @@
    */
   int eventsToSink;
   private boolean attached;
-  @SuppressWarnings("deprecation")
   private HandlerManager handlerManager;
   private Object layoutData;
   private Widget parent;
@@ -74,7 +73,6 @@
    * @param handler the handler
    * @return {...@link HandlerRegistration} used to remove the handler
    */
-  @SuppressWarnings("deprecation")
   public final <H extends EventHandler> HandlerRegistration addDomHandler(
       final H handler, DomEvent.Type<H> type) {
     assert handler != null : "handler must not be null";
@@ -91,7 +89,6 @@
    * @param handler the handler
    * @return {...@link HandlerRegistration} used to remove the handler
    */
-  @SuppressWarnings("deprecation")
   public final <H extends EventHandler> HandlerRegistration addHandler(
       final H handler, GwtEvent.Type<H> type) {
     return ensureHandlers().addHandler(type, handler);
@@ -101,7 +98,6 @@
     return this;
   }

-  @SuppressWarnings("deprecation")
   public void fireEvent(GwtEvent<?> event) {
     if (handlerManager != null) {
       handlerManager.fireEvent(event);
@@ -228,7 +224,6 @@
    *
    * @return the {...@link HandlerManager} you want to use
    */
-  @SuppressWarnings("deprecation")
protected com.google.gwt.event.shared.HandlerManager createHandlerManager() {
     return new com.google.gwt.event.shared.HandlerManager(this);
   }
@@ -272,7 +267,6 @@
    * @param type the event type
    * @return the number of registered handlers
    */
-  @SuppressWarnings("deprecation")
   protected int getHandlerCount(Type<?> type) {
return handlerManager == null ? 0 : handlerManager.getHandlerCount(type);
   }
@@ -396,13 +390,11 @@
    *
    * @return the handler manager
    * */
-  @SuppressWarnings("deprecation")
   com.google.gwt.event.shared.HandlerManager ensureHandlers() {
     return handlerManager == null ? handlerManager = createHandlerManager()
         : handlerManager;
   }

-  @SuppressWarnings("deprecation")
   com.google.gwt.event.shared.HandlerManager getHandlerManager() {
     return handlerManager;
   }

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

Reply via email to