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

Side by Side Diff: src/codec/SkMaskSwizzler.h

Issue 1407603003: Fix SkSwizzler bug (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Response to comments Created 5 years, 2 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 unified diff | Download patch
« no previous file with comments | « no previous file | src/codec/SkMaskSwizzler.cpp » ('j') | src/codec/SkMaskSwizzler.cpp » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright 2015 Google Inc. 2 * Copyright 2015 Google Inc.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license that can be 4 * Use of this source code is governed by a BSD-style license that can be
5 * found in the LICENSE file. 5 * found in the LICENSE file.
6 */ 6 */
7 #ifndef SkMaskSwizzler_DEFINED 7 #ifndef SkMaskSwizzler_DEFINED
8 #define SkMaskSwizzler_DEFINED 8 #define SkMaskSwizzler_DEFINED
9 9
10 #include "SkMasks.h" 10 #include "SkMasks.h"
(...skipping 38 matching lines...) Expand 10 before | Expand all | Expand 10 after
49 /* 49 /*
50 * Row procedure used for swizzle 50 * Row procedure used for swizzle
51 */ 51 */
52 typedef SkSwizzler::ResultAlpha (*RowProc)( 52 typedef SkSwizzler::ResultAlpha (*RowProc)(
53 void* dstRow, const uint8_t* srcRow, int width, 53 void* dstRow, const uint8_t* srcRow, int width,
54 SkMasks* masks, uint32_t startX, uint32_t sampleX); 54 SkMasks* masks, uint32_t startX, uint32_t sampleX);
55 55
56 /* 56 /*
57 * Constructor for mask swizzler 57 * Constructor for mask swizzler
58 */ 58 */
59 SkMaskSwizzler(int width, SkMasks* masks, RowProc proc); 59 SkMaskSwizzler(int width, SkMasks* masks, RowProc proc);
scroggo 2015/10/16 13:36:16 subsetWidth?
msarett 2015/10/16 17:36:52 Done.
60 60
61 int onSetSampleX(int) override; 61 int onSetSampleX(int) override;
62 62
63 SkMasks* fMasks; // unowned 63 SkMasks* fMasks; // unowned
64 const RowProc fRowProc; 64 const RowProc fRowProc;
65 65
66 // FIXME: Can this class share more with SkSwizzler? These variables are all the same. 66 // FIXME: Can this class share more with SkSwizzler? These variables are all the same.
67 const int fSrcWidth; // Width of the source - i.e. before any s ampling. 67 const int fSubsetWidth; // Width of the subset of source before an y sampling.
68 int fDstWidth; // Width of dst, which may differ with sam pling. 68 int fDstWidth; // Width of dst, which may differ with sam pling.
69 int fSampleX; 69 int fSampleX;
70 int fX0; 70 int fX0;
71 }; 71 };
72 72
73 #endif 73 #endif
OLDNEW
« no previous file with comments | « no previous file | src/codec/SkMaskSwizzler.cpp » ('j') | src/codec/SkMaskSwizzler.cpp » ('J')

Powered by Google App Engine
This is Rietveld 408576698