Remove deprecated functions from Canvas in favor of the RectF version.

ui/gfx/canvas.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef UI_GFX_CANVAS_H_
 #define UI_GFX_CANVAS_H_
 
 #include <stdint.h>
 
 #include <memory>
@@ skipping 152 matching lines @@
                               const FontList& font_list,
                               SkColor text_color,
                               SkColor halo_color,
                               const Rect& display_rect,
                               int flags);
 
   // Extracts an ImageSkiaRep from the contents of this canvas.
   ImageSkiaRep ExtractImageRep() const;
 
   // Draws a dashed rectangle of the specified color.
-  // DEPRECATED in favor of the RectF version below.
-  // TODO(funkysidd): Remove this (http://crbug.com/553726)
-  void DrawDashedRect(const Rect& rect, SkColor color);
-
-  // Draws a dashed rectangle of the specified color.
   void DrawDashedRect(const RectF& rect, SkColor color);
 
   // Unscales by the image scale factor (aka device scale factor), and returns
   // that factor.  This is useful when callers want to draw directly in the
   // native scale.
   float UndoDeviceScaleFactor();
 
   // Saves a copy of the drawing state onto a stack, operating on this copy
   // until a balanced call to Restore() is made.
   void Save();
@@ skipping 44 matching lines @@
   // SkBlendMode::kSrcOver.
   void FillRect(const Rect& rect, SkColor color);
 
   // Fills |rect| with the specified |color| and |mode|.
   void FillRect(const Rect& rect, SkColor color, SkBlendMode mode);
 
   // Draws a single pixel rect in the specified region with the specified
   // color, using a transfer mode of SkBlendMode::kSrcOver.
   //
   // NOTE: if you need a single pixel line, use DrawLine.
-  // DEPRECATED in favor of the RectF version below.
-  // TODO(funkysidd): Remove this (http://crbug.com/553726)
-  void DrawRect(const Rect& rect, SkColor color);

[Matt Giuca, 2017/04/07 03:47:54]

This function seems to still be used by
chrome/browser/thumbnails/content_analysis_unittest.cc.

You will have to replace those usages with the RectF version before this can
land.

[Uzair, 2017/04/07 05:30:59]

Done.

-
-  // Draws a single pixel rect in the specified region with the specified
-  // color, using a transfer mode of SkBlendMode::kSrcOver.
-  //
-  // NOTE: if you need a single pixel line, use DrawLine.
   void DrawRect(const RectF& rect, SkColor color);
 
   // Draws a single pixel rect in the specified region with the specified
   // color and transfer mode.
   //
   // NOTE: if you need a single pixel line, use DrawLine.
   // DEPRECATED in favor of the RectF version below.
   // TODO(funkysidd): Remove this (http://crbug.com/553726)
   void DrawRect(const Rect& rect, SkColor color, SkBlendMode mode);

[Matt Giuca, 2017/04/07 03:47:54]

This one doesn't seem to have any uses so can be removed. Do that here.

[Uzair, 2017/04/07 05:30:59]

Done.

 
   // Draws a single pixel rect in the specified region with the specified
   // color and transfer mode.
   //
   // NOTE: if you need a single pixel line, use DrawLine.
   void DrawRect(const RectF& rect, SkColor color, SkBlendMode mode);
 
   // Draws the given rectangle with the given |flags| parameters.
   // DEPRECATED in favor of the RectF version below.
   // TODO(funkysidd): Remove this (http://crbug.com/553726)
@@ skipping 247 matching lines @@
   base::Optional<SkBitmap> bitmap_;
   base::Optional<cc::SkiaPaintCanvas> owned_canvas_;
   cc::PaintCanvas* canvas_;
 
   DISALLOW_COPY_AND_ASSIGN(Canvas);
 };
 
 }  // namespace gfx
 
 #endif  // UI_GFX_CANVAS_H_