Move WebPageOverlay subclasses to use WebGraphicsContext.

chrome/renderer/webview_color_overlay.h

Index: chrome/renderer/webview_color_overlay.h
diff --git a/chrome/renderer/webview_color_overlay.h b/chrome/renderer/webview_color_overlay.h
index 3a2b9d1042e507f874350c40741f25b46d957acf..e195c4ddbfb5a592c69e509b7468bc72940cb9b9 100644
--- a/chrome/renderer/webview_color_overlay.h
+++ b/chrome/renderer/webview_color_overlay.h
@@ -7,7 +7,6 @@
 
 #include "base/basictypes.h"
 #include "base/compiler_specific.h"
-#include "third_party/WebKit/public/platform/WebRect.h"
 #include "third_party/WebKit/public/web/WebPageOverlay.h"
 #include "third_party/skia/include/core/SkColor.h"
 
@@ -23,7 +22,8 @@ class WebViewColorOverlay : public blink::WebPageOverlay {
 
  private:
   // blink::WebPageOverlay implementation:
-  virtual void paintPageOverlay(blink::WebCanvas* canvas);
+  virtual void paintPageOverlay(blink::WebGraphicsContext* context,

[Lei Zhang, 2015/02/12 20:01:21]

might as do virtual -> override while we are here.

[jbroman, 2015/02/12 20:43:59]

I've been advised in the past not to use "override" on overrides crossing
library boundaries (the superclass here is in Blink) as it makes some cross-repo
changes painful.

[jbroman, 2015/02/12 20:47:10]

see e.g. https://codereview.chromium.org/679523004

+                                const blink::WebSize& webViewSize);
 
   content::RenderView* render_view_;
   SkColor color_;