cc: Remove WebLayerImpl::invalidateRect(WebFloatRect)

cc/blink/web_layer_impl_fixed_bounds.h

Index: cc/blink/web_layer_impl_fixed_bounds.h
diff --git a/cc/blink/web_layer_impl_fixed_bounds.h b/cc/blink/web_layer_impl_fixed_bounds.h
index 52f54cfd3b36fb90fb7c82701bb5602f64a783ca..fa88d6b6054848c11f1a0bf1815b760d7a458685 100644
--- a/cc/blink/web_layer_impl_fixed_bounds.h
+++ b/cc/blink/web_layer_impl_fixed_bounds.h
@@ -19,18 +19,18 @@ namespace cc_blink {
 class WebLayerImplFixedBounds : public WebLayerImpl {
  public:
   CC_BLINK_EXPORT WebLayerImplFixedBounds();
-  CC_BLINK_EXPORT explicit WebLayerImplFixedBounds(scoped_refptr<cc::Layer>);
+  CC_BLINK_EXPORT explicit WebLayerImplFixedBounds(
+      scoped_refptr<cc::Layer> layer);
   virtual ~WebLayerImplFixedBounds();
 
   // WebLayerImpl overrides.
-  virtual void invalidateRect(const blink::WebFloatRect& rect) override;
   virtual void invalidateRect(const blink::WebRect& rect) override;
   virtual void setTransformOrigin(
-      const blink::WebFloatPoint3D& transform_origin);
-  virtual void setBounds(const blink::WebSize& bounds);
-  virtual blink::WebSize bounds() const;
+      const blink::WebFloatPoint3D& transform_origin) override;

[danakj, 2014/10/14 18:00:56]

I left the override on invalidateRect and added them to the others. these are
overriding WebLayerImpl which is on the chromium side so they should be there.
clang plugin doesn't enforce this tho since it doesn't notice the chromium side
overrode the blink side. But these break when you change the blink side of the
codebase if they are missing.

+  virtual void setBounds(const blink::WebSize& bounds) override;
+  virtual blink::WebSize bounds() const override;
   virtual void setTransform(const SkMatrix44& transform);

[enne (OOO), 2014/10/14 19:27:27]

This doesn't need an override?

[danakj, 2014/10/14 19:28:21]

oops missed

-  virtual SkMatrix44 transform() const;
+  virtual SkMatrix44 transform() const override;
 
   CC_BLINK_EXPORT void SetFixedBounds(gfx::Size bounds);