diff options
author | Herb Derby <herb@google.com> | 2018-07-26 16:54:18 -0400 |
---|---|---|
committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | 2018-07-27 03:13:56 +0000 |
commit | b935cf8e12d7371246d318f20f4ebf88e321573a (patch) | |
tree | 4af6fce01d981c24e16c5adfd92680d49199988e /src/core/SkRemoteGlyphCache.cpp | |
parent | 5f1dc76d0ceceb97796ec1626e59fe8d4ede581c (diff) |
const all the things
Having the glyph run list be const as it passes through the stack means
that future change can't be introduced in the device code that changes
behavior. Try to force all text changes into the SkGylphRun system.
Change-Id: I9412bc094c7adb8554887c725a6264af306e1d42
Reviewed-on: https://skia-review.googlesource.com/143702
Commit-Queue: Herb Derby <herb@google.com>
Reviewed-by: Ben Wagner <bungeman@google.com>
Diffstat (limited to 'src/core/SkRemoteGlyphCache.cpp')
-rw-r--r-- | src/core/SkRemoteGlyphCache.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/core/SkRemoteGlyphCache.cpp b/src/core/SkRemoteGlyphCache.cpp index 4b9239dc6b..58422627f5 100644 --- a/src/core/SkRemoteGlyphCache.cpp +++ b/src/core/SkRemoteGlyphCache.cpp @@ -239,14 +239,14 @@ public: } protected: - void drawGlyphRunList(SkGlyphRunList* glyphRunList) override { + void drawGlyphRunList(const SkGlyphRunList& glyphRunList) override { SkPaint runPaint; SkGlyphRunListIterator it(glyphRunList); for (; !it.done(); it.next()) { // applyFontToPaint() always overwrites the exact same attributes, // so it is safe to not re-seed the paint for this reason. it.applyFontToPaint(&runPaint); - this->processGlyphRun(glyphRunList->origin(), it, runPaint); + this->processGlyphRun(glyphRunList.origin(), it, runPaint); } } |