Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(301)

Unified Diff: Source/WebKit/chromium/src/WebScrollbarThemePainter.cpp

Issue 14550002: Making GraphicsContext the owner of PlatformContext. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Fixing tests and addressing comments. Created 7 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Source/WebKit/chromium/src/WebScrollbarThemePainter.cpp
diff --git a/Source/WebKit/chromium/src/WebScrollbarThemePainter.cpp b/Source/WebKit/chromium/src/WebScrollbarThemePainter.cpp
index 16601147eb442b718bdb5dcdb4a246affbb67f8d..11e08ad6a90aa747a3b0ba7b43164d79dc9be847 100644
--- a/Source/WebKit/chromium/src/WebScrollbarThemePainter.cpp
+++ b/Source/WebKit/chromium/src/WebScrollbarThemePainter.cpp
@@ -48,81 +48,71 @@ void WebScrollbarThemePainter::paintScrollbarBackground(WebCanvas* canvas, const
SkRect clip = SkRect::MakeXYWH(rect.x, rect.y, rect.width, rect.height);
canvas->clipRect(clip);
- PlatformContextSkia platformContext(canvas);
- platformContext.setDrawingToImageBuffer(true);
- GraphicsContext context(&platformContext);
+ GraphicsContext context(canvas);
+ context.platformContext()->setDrawingToImageBuffer(true);
m_theme->paintScrollbarBackground(&context, m_scrollbar);
}
void WebScrollbarThemePainter::paintTrackBackground(WebCanvas* canvas, const WebRect& rect)
{
- PlatformContextSkia platformContext(canvas);
- platformContext.setDrawingToImageBuffer(true);
- GraphicsContext context(&platformContext);
+ GraphicsContext context(canvas);
+ context.platformContext()->setDrawingToImageBuffer(true);
m_theme->paintTrackBackground(&context, m_scrollbar, IntRect(rect));
}
void WebScrollbarThemePainter::paintBackTrackPart(WebCanvas* canvas, const WebRect& rect)
{
- PlatformContextSkia platformContext(canvas);
- platformContext.setDrawingToImageBuffer(true);
- GraphicsContext context(&platformContext);
+ GraphicsContext context(canvas);
+ context.platformContext()->setDrawingToImageBuffer(true);
m_theme->paintTrackPiece(&context, m_scrollbar, IntRect(rect), WebCore::BackTrackPart);
}
void WebScrollbarThemePainter::paintForwardTrackPart(WebCanvas* canvas, const WebRect& rect)
{
- PlatformContextSkia platformContext(canvas);
- platformContext.setDrawingToImageBuffer(true);
- GraphicsContext context(&platformContext);
+ GraphicsContext context(canvas);
+ context.platformContext()->setDrawingToImageBuffer(true);
m_theme->paintTrackPiece(&context, m_scrollbar, IntRect(rect), WebCore::ForwardTrackPart);
}
void WebScrollbarThemePainter::paintBackButtonStart(WebCanvas* canvas, const WebRect& rect)
{
- PlatformContextSkia platformContext(canvas);
- platformContext.setDrawingToImageBuffer(true);
- GraphicsContext context(&platformContext);
+ GraphicsContext context(canvas);
+ context.platformContext()->setDrawingToImageBuffer(true);
m_theme->paintButton(&context, m_scrollbar, IntRect(rect), WebCore::BackButtonStartPart);
}
void WebScrollbarThemePainter::paintBackButtonEnd(WebCanvas* canvas, const WebRect& rect)
{
- PlatformContextSkia platformContext(canvas);
- platformContext.setDrawingToImageBuffer(true);
- GraphicsContext context(&platformContext);
+ GraphicsContext context(canvas);
+ context.platformContext()->setDrawingToImageBuffer(true);
m_theme->paintButton(&context, m_scrollbar, IntRect(rect), WebCore::BackButtonEndPart);
}
void WebScrollbarThemePainter::paintForwardButtonStart(WebCanvas* canvas, const WebRect& rect)
{
- PlatformContextSkia platformContext(canvas);
- platformContext.setDrawingToImageBuffer(true);
- GraphicsContext context(&platformContext);
+ GraphicsContext context(canvas);
+ context.platformContext()->setDrawingToImageBuffer(true);
m_theme->paintButton(&context, m_scrollbar, IntRect(rect), WebCore::ForwardButtonStartPart);
}
void WebScrollbarThemePainter::paintForwardButtonEnd(WebCanvas* canvas, const WebRect& rect)
{
- PlatformContextSkia platformContext(canvas);
- platformContext.setDrawingToImageBuffer(true);
- GraphicsContext context(&platformContext);
+ GraphicsContext context(canvas);
+ context.platformContext()->setDrawingToImageBuffer(true);
m_theme->paintButton(&context, m_scrollbar, IntRect(rect), WebCore::ForwardButtonEndPart);
}
void WebScrollbarThemePainter::paintTickmarks(WebCanvas* canvas, const WebRect& rect)
{
- PlatformContextSkia platformContext(canvas);
- platformContext.setDrawingToImageBuffer(true);
- GraphicsContext context(&platformContext);
+ GraphicsContext context(canvas);
+ context.platformContext()->setDrawingToImageBuffer(true);
m_theme->paintTickmarks(&context, m_scrollbar, IntRect(rect));
}
void WebScrollbarThemePainter::paintThumb(WebCanvas* canvas, const WebRect& rect)
{
- PlatformContextSkia platformContext(canvas);
- platformContext.setDrawingToImageBuffer(true);
- GraphicsContext context(&platformContext);
+ GraphicsContext context(canvas);
+ context.platformContext()->setDrawingToImageBuffer(true);
m_theme->paintThumb(&context, m_scrollbar, IntRect(rect));
}

Powered by Google App Engine
This is Rietveld 408576698