commit 4984c3c95f18eda44492a2126c9958e447f2cca8
author msarett <msarett@google.com> Thu Mar 10 05:44:43 2016 -0800
committer Commit bot <commit-bot@chromium.org> Thu Mar 10 05:44:43 2016 -0800
tree add5bdca2495968a8acc1affd10e5cfe225ff4d0
parent d6215cf4a5f416cf0b64a4fbba95c519f03fe467

Update Skia's YUV API

We should match the recently designed API in SkCodec.
https://codereview.chromium.org/1549473003/

This requires changes in Chromium as well.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1716523002

Committed: https://skia.googlesource.com/skia/+/095d31c8a0eeb5d491febf064bc3c8a44e22b94f

Review URL: https://codereview.chromium.org/1716523002

src/codec/SkCodecImageGenerator.h

diff --git a/src/codec/SkCodecImageGenerator.h b/src/codec/SkCodecImageGenerator.h
index d2c74ab..6d34223 100644
--- a/src/codec/SkCodecImageGenerator.h
+++ b/src/codec/SkCodecImageGenerator.h
@@ -26,8 +26,9 @@
     bool onGetPixels(const SkImageInfo& info, void* pixels, size_t rowBytes, SkPMColor ctable[],
             int* ctableCount) override;
 
-    bool onGetYUV8Planes(SkISize sizes[3], void* planes[3], size_t rowBytes[3],
-            SkYUVColorSpace* colorSpace) override;
+    bool onQueryYUV8(SkYUVSizeInfo*, SkYUVColorSpace*) const override;
+
+    bool onGetYUV8Planes(const SkYUVSizeInfo&, void* planes[3]) override;
 
 private:
     /*
@@ -39,11 +40,5 @@
     SkAutoTDelete<SkCodec> fCodec;
     SkAutoTUnref<SkData> fData;
 
-    // FIXME: These fields are necessary only until we change the API of SkImageGenerator
-    //        to match SkCodec.  Once the API is changed, they should be removed.
-    int fYWidth;
-    int fUWidth;
-    int fVWidth;
-
     typedef SkImageGenerator INHERITED;
 };