diff options
| author |  mtklein <mtklein@chromium.org> | 2016-04-29 13:58:09 -0700 |
|---|---|---|
| committer |  Commit bot <commit-bot@chromium.org> | 2016-04-29 13:58:09 -0700 |
| commit | b4c899d48d4c5d8e867beb611551f7b55ec60abb (patch) | |
| tree | c22a1203e79d6d1eef676c1980636fa52f187eca /src | |
| parent | 900bd4a0463bc6471ef07a77120b413bd8f472b2 (diff) | |
remove SkWriteBuffer::getWriter32().
SkWriteBuffer exposes its lower-level implementation SkWriter32 through
this one call. It's not currently used in any interesting way:
- write_encoded_bitmap() uses it to manually re-create writeDataAsByteArray();
- unit tests use it incidentally as a quick way to read the serialized bytes.
This should be SkWriteBuffer no longer necessarily needs to have an SkWriter32.
Landing this will let us then remove SkWriter32::contiguousArray().
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1936563002
TBR=reed@google.com
Just deleting API
Review-Url: https://codereview.chromium.org/1936563002
Diffstat (limited to 'src')
| -rw-r--r-- | src/core/SkWriteBuffer.cpp | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/core/SkWriteBuffer.cpp b/src/core/SkWriteBuffer.cpp index b90a81e631..2b35b67702 100644 --- a/src/core/SkWriteBuffer.cpp +++ b/src/core/SkWriteBuffer.cpp @@ -136,8 +136,7 @@ bool SkWriteBuffer::writeToStream(SkWStream* stream) { static void write_encoded_bitmap(SkWriteBuffer* buffer, SkData* data, const SkIPoint& origin) { - buffer->writeUInt(SkToU32(data->size())); - buffer->getWriter32()->writePad(data->data(), data->size()); + buffer->writeDataAsByteArray(data); buffer->write32(origin.fX); buffer->write32(origin.fY); } |