Revision: 10471
Author:   [email protected]
Date:     Wed Jul 27 08:45:41 2011
Log:      Remove "final" keyword from IsRenderable-overrides in Composite.

Review at http://gwt-code-reviews.appspot.com/1504802

Review by: [email protected]
http://code.google.com/p/google-web-toolkit/source/detail?r=10471

Modified:
 /trunk/user/src/com/google/gwt/user/client/ui/Composite.java

=======================================
--- /trunk/user/src/com/google/gwt/user/client/ui/Composite.java Thu Jul 21 09:08:18 2011 +++ /trunk/user/src/com/google/gwt/user/client/ui/Composite.java Wed Jul 27 08:45:41 2011
@@ -38,8 +38,6 @@
  * <h3>Example</h3>
  * {@example com.google.gwt.examples.CompositeExample}
  * </p>
- *
- * TODO(rdcastro): Remove the final qualifier from IsRenderable overrides.
  */
 public abstract class Composite extends Widget implements IsRenderable {

@@ -50,7 +48,7 @@
   private Element elementToWrap;

   @Override
-  public final void claimElement(Element element) {
+  public void claimElement(Element element) {
     if (renderable != null) {
       renderable.claimElement(element);
       setElement(widget.getElement());
@@ -60,7 +58,7 @@
   }

   @Override
-  public final void initializeClaimedElement() {
+  public void initializeClaimedElement() {
     if (renderable != null) {
       renderable.initializeClaimedElement();
     } else {
@@ -86,7 +84,7 @@
   }

   @Override
-  public final SafeHtml render(RenderableStamper stamper) {
+  public SafeHtml render(RenderableStamper stamper) {
     if (renderable != null) {
       return renderable.render(stamper);
     } else {
@@ -98,7 +96,7 @@
   }

   @Override
- public final void render(RenderableStamper stamper, SafeHtmlBuilder builder) {
+  public void render(RenderableStamper stamper, SafeHtmlBuilder builder) {
     if (renderable != null) {
       renderable.render(stamper, builder);
     } else {

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

Reply via email to