[Cleanup] Follow Up of Pinch/Zoom Infrastucture & Plumbing CL

public/platform/WebLayer.h

 /*
  * Copyright (C) 2011 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
  * 2.  Redistributions in binary form must reproduce the above copyright
@@ skipping 155 matching lines @@
     // This is not a requirement of the scroll parent.
     virtual void setClipParent(WebLayer*) = 0;
 
     // Scrolling
     virtual void setScrollPosition(WebPoint) = 0;
     virtual WebPoint scrollPosition() const = 0;
 
     virtual WebSize maxScrollPosition() const = 0;
 
     // To set a WebLayer as scrollable we must specify the corresponding clip layer.
-    // TODO(wjmaclean) Make this pure virtual once https://codereview.chromium.org/23983047 lands.
-    virtual void setScrollClipLayer(WebLayer*) { }
+    virtual void setScrollClipLayer(WebLayer*) = 0;

[wjmaclean, 2014/01/31 13:38:25]

nit: Don't need a blank line after this one.

[deepak.sa, 2014/02/01 09:15:24]

Done.

     virtual bool scrollable() const = 0;
 
     virtual void setUserScrollable(bool horizontal, bool vertical) = 0;
     virtual bool userScrollableHorizontal() const = 0;
     virtual bool userScrollableVertical() const = 0;
 
     virtual void setHaveWheelEventHandlers(bool) = 0;
     virtual bool haveWheelEventHandlers() const = 0;
 
     virtual void setShouldScrollOnMainThread(bool) = 0;
@@ skipping 21 matching lines @@
     virtual void setScrollClient(WebLayerScrollClient*) = 0;
 
     // Forces this layer to use a render surface. There is no benefit in doing
     // so, but this is to facilitate benchmarks and tests.
     virtual void setForceRenderSurface(bool) = 0;
 
     // True if the layer is not part of a tree attached to a WebLayerTreeView.
     virtual bool isOrphan() const = 0;
 
     virtual void setWebLayerClient(WebLayerClient*) = 0;
-
-    // TODO(wjmaclean) Remove next two lines once https://codereview.chromium.org/23983047 lands.
-    virtual void setMaxScrollPosition(WebSize) { }
-    virtual void setScrollable(bool) { }
 };
 
 } // namespace blink
 
 #endif // WebLayer_h