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

Unified Diff: third_party/WebKit/Source/platform/graphics/GraphicsContext.cpp

Issue 2739533003: Revert of Make cc/paint have concrete types (Closed)
Patch Set: Created 3 years, 9 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: third_party/WebKit/Source/platform/graphics/GraphicsContext.cpp
diff --git a/third_party/WebKit/Source/platform/graphics/GraphicsContext.cpp b/third_party/WebKit/Source/platform/graphics/GraphicsContext.cpp
index 980e8c38a082fa0d07127988f64b3a1419eabca4..6b8397a0c3a16442320670088c393b5e010650e7 100644
--- a/third_party/WebKit/Source/platform/graphics/GraphicsContext.cpp
+++ b/third_party/WebKit/Source/platform/graphics/GraphicsContext.cpp
@@ -82,7 +82,7 @@
if (contextDisabled()) {
DEFINE_STATIC_LOCAL(SkCanvas*, nullSkCanvas,
(SkMakeNullCanvas().release()));
- DEFINE_STATIC_LOCAL(PaintCanvas, nullCanvas, (nullSkCanvas));
+ DEFINE_STATIC_LOCAL(PaintCanvasPassThrough, nullCanvas, (nullSkCanvas));
m_canvas = &nullCanvas;
}
}
@@ -267,7 +267,7 @@
return;
DCHECK(!m_canvas);
- m_canvas = m_paintRecorder.beginRecording(bounds);
+ m_canvas = m_paintRecorder.beginRecording(bounds, nullptr);
if (m_hasMetaData)
m_canvas->getMetaData() = m_metaData;
}
@@ -276,7 +276,7 @@
sk_sp<PaintRecord> createEmptyPaintRecord() {
PaintRecorder recorder;
- recorder.beginRecording(SkRect::MakeEmpty());
+ recorder.beginRecording(SkRect::MakeEmpty(), nullptr);
return recorder.finishRecordingAsPicture();
}
@@ -560,7 +560,7 @@
PaintFlags flags;
flags.setAntiAlias(true);
flags.setColor(color);
- flags.setStyle(PaintFlags::kStroke_Style);
+ flags.setStyle(SkPaint::kStroke_Style);
flags.setStrokeWidth(kH * 1 / 2);
PaintRecorder recorder;
@@ -645,9 +645,9 @@
PaintFlags flags;
flags.setAntiAlias(true);
- flags.setShader(WrapSkShader(
- MakePaintShaderRecord(sk_ref_sp(marker), SkShader::kRepeat_TileMode,
- SkShader::kClamp_TileMode, &localMatrix, nullptr)));
+ flags.setShader(WrapSkShader(SkShader::MakePictureShader(
+ sk_ref_sp(marker), SkShader::kRepeat_TileMode, SkShader::kClamp_TileMode,
+ &localMatrix, nullptr)));
// Apply the origin translation as a global transform. This ensures that the
// shader local matrix depends solely on zoom => Skia can reuse the same
@@ -1249,7 +1249,7 @@
DCHECK(m_canvas);
sk_sp<SkData> url(SkData::MakeWithCString(link.getString().utf8().data()));
- PaintCanvasAnnotateRectWithURL(m_canvas, destRect, url.get());
+ SkAnnotateRectWithURL(m_canvas, destRect, url.get());
}
void GraphicsContext::setURLFragmentForRect(const String& destName,
@@ -1259,7 +1259,7 @@
DCHECK(m_canvas);
sk_sp<SkData> skDestName(SkData::MakeWithCString(destName.utf8().data()));
- PaintCanvasAnnotateLinkToDestination(m_canvas, rect, skDestName.get());
+ SkAnnotateLinkToDestination(m_canvas, rect, skDestName.get());
}
void GraphicsContext::setURLDestinationLocation(const String& name,
@@ -1269,7 +1269,7 @@
DCHECK(m_canvas);
sk_sp<SkData> skName(SkData::MakeWithCString(name.utf8().data()));
- PaintCanvasAnnotateNamedDestination(
+ SkAnnotateNamedDestination(
m_canvas, SkPoint::Make(location.x(), location.y()), skName.get());
}

Powered by Google App Engine
This is Rietveld 408576698