05dd251e5e135626d170b9e77eb64729bd482169 |
|
05-Jan-2016 |
reed <reed@google.com> |
take gr-context parameter to refEncoded, indicating a desire for only gpu-specific formats Prime motivator: - we always call refEncoded on the generator when trying to upload - we call it *before* we ask for raster or YUV - for blink, this call can be very slow, as they have to cons-up their SkData the first time (and grab a mutex to do it) - this parameter will indicate to them that we're only interested in gpu formats, which they will know if they have. BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1556333004 Review URL: https://codereview.chromium.org/1556333004
/external/skia/src/image/SkImage_Raster.cpp
|
db7dc09e526e815ade242e3dcb066693c42c8bda |
|
18-Dec-2015 |
scroggo <scroggo@google.com> |
Remove SkImageMinRowBytes As stated in the comments in crrev.com/1379193002, this method name is misleading. It returns a larger number than the minimum rowbytes - it increases to the next four-byte alignment. This has the effect that the one place that calls it does not support 565 (which is not already four-byte aligned), but does not serve any other purpose. Remove it and the only call-site. BUG=skia:4396 GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1528383004 Review URL: https://codereview.chromium.org/1528383004
/external/skia/src/image/SkImage_Raster.cpp
|
095530389d8211895309107a4f74beefacce638c |
|
23-Nov-2015 |
reed <reed@google.com> |
scaling API on SkPixmap BUG=skia:4481 Review URL: https://codereview.chromium.org/1463373002
/external/skia/src/image/SkImage_Raster.cpp
|
045802dbb7202b52f5fd2758d725f39c156a6165 |
|
20-Oct-2015 |
bsalomon <bsalomon@google.com> |
Rewrite GrTextureMaker to disentangle bitmap case from base class and give GPU object a say in what copying needs to be done. Committed: https://skia.googlesource.com/skia/+/fcffaf22d697f06f903c3193308f9dc54a959f79 Committed: https://skia.googlesource.com/skia/+/1a197ea31e0aac7ea312e9a6c0d9f5df626b0350 Review URL: https://codereview.chromium.org/1409163002
/external/skia/src/image/SkImage_Raster.cpp
|
bc0bcc08b390e430f66710ddfbc47da39ab841d9 |
|
19-Oct-2015 |
Brian Salomon <bsalomon@google.com> |
Revert recent CLs around GrTextureMaker/GrTextureParamsAdjuster Review URL: https://codereview.chromium.org/1411353002 .
/external/skia/src/image/SkImage_Raster.cpp
|
1a197ea31e0aac7ea312e9a6c0d9f5df626b0350 |
|
19-Oct-2015 |
bsalomon <bsalomon@google.com> |
Rewrite GrTextureMaker to disentangle bitmap case from base class and give GPU object a say in what copying needs to be done. Committed: https://skia.googlesource.com/skia/+/fcffaf22d697f06f903c3193308f9dc54a959f79 Review URL: https://codereview.chromium.org/1409163002
/external/skia/src/image/SkImage_Raster.cpp
|
91957941ce2fe9457babe7f83514e4599089d411 |
|
16-Oct-2015 |
egdaniel <egdaniel@google.com> |
Revert of Rewrite GrTextureMaker to disentangle bitmap case from base class and give GPU object a say in what… (patchset #4 id:60001 of https://codereview.chromium.org/1409163002/ ) Reason for revert: breaking nanobench Original issue's description: > Rewrite GrTextureMaker to disentangle bitmap case from base class and give GPU object a say in what copying needs to be done. > > Committed: https://skia.googlesource.com/skia/+/fcffaf22d697f06f903c3193308f9dc54a959f79 TBR=reed@google.com,bsalomon@google.com NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/1409923003
/external/skia/src/image/SkImage_Raster.cpp
|
fcffaf22d697f06f903c3193308f9dc54a959f79 |
|
16-Oct-2015 |
bsalomon <bsalomon@google.com> |
Rewrite GrTextureMaker to disentangle bitmap case from base class and give GPU object a say in what copying needs to be done. Review URL: https://codereview.chromium.org/1409163002
/external/skia/src/image/SkImage_Raster.cpp
|
afa95e270c64c9777647b6c58b796750ced57c39 |
|
12-Oct-2015 |
bsalomon <bsalomon@google.com> |
Remove image usage type enum. Use GrTextureParams instead. BUG=skia: Review URL: https://codereview.chromium.org/1404433002
/external/skia/src/image/SkImage_Raster.cpp
|
af3fbfca0ab706230c33e46af620e0044a56c83e |
|
04-Oct-2015 |
reed <reed@chromium.org> |
SkImage doesn't use props, so don't need to store it BUG=skia: TBR=bsalomon Review URL: https://codereview.chromium.org/1372153006
/external/skia/src/image/SkImage_Raster.cpp
|
856e9d921462136da8562f8f122d42e114cd4710 |
|
30-Sep-2015 |
reed <reed@google.com> |
Revert[4] of add ImageShader, sharing code with its Bitmap cousin Now with GrTextureMaker subclasses to handle npot usage. This reverts commit 476506d070dbc59b158acc1a00c34bff95ab2968. BUG=skia: Review URL: https://codereview.chromium.org/1370223002
/external/skia/src/image/SkImage_Raster.cpp
|
c56c6ef3ce6447f39894649d535acb1606595d31 |
|
28-Sep-2015 |
halcanary <halcanary@google.com> |
SkImage::NewFromBitmap works with Index8 BUG=skia:4390 Review URL: https://codereview.chromium.org/1377473002
/external/skia/src/image/SkImage_Raster.cpp
|
476506d070dbc59b158acc1a00c34bff95ab2968 |
|
28-Sep-2015 |
reed <reed@google.com> |
Revert of Revert[2] of add ImageShader, sharing code with its Bitmap cousin (patchset #11 id:200001 of https://codereview.chromium.org/1352293002/ ) Reason for revert: guess: Need to handle NPOT support in shader case Original issue's description: > Revert[2] of add ImageShader, sharing code with its Bitmap cousin > > Adjustment from previous land : runtime check for npot-support when texture is requested for tiling. > > (patchset #10 id:180001 of https://codereview.chromium.org/1342113002/ ) > > This reverts commit f2608513626264459a00388537175600b515cae2. > > BUG=skia:4365 > > Committed: https://skia.googlesource.com/skia/+/fe05707c356d2a8c2c69222591d4cafbd456b4e5 TBR=fmalita@chromium.org,robertphillips@google.com,bsalomon@google.com,reed@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia:4365 Review URL: https://codereview.chromium.org/1375673002
/external/skia/src/image/SkImage_Raster.cpp
|
fe05707c356d2a8c2c69222591d4cafbd456b4e5 |
|
25-Sep-2015 |
reed <reed@google.com> |
Revert[2] of add ImageShader, sharing code with its Bitmap cousin Adjustment from previous land : runtime check for npot-support when texture is requested for tiling. (patchset #10 id:180001 of https://codereview.chromium.org/1342113002/ ) This reverts commit f2608513626264459a00388537175600b515cae2. BUG=skia:4365 Review URL: https://codereview.chromium.org/1352293002
/external/skia/src/image/SkImage_Raster.cpp
|
7b6945bc4e639d7cc4a49b84d492690f8e865566 |
|
24-Sep-2015 |
reed <reed@google.com> |
remove unused (by the outside) SkImage::newSurface, and simplify newImage -> newSubset BUG=skia: Review URL: https://codereview.chromium.org/1364443002
/external/skia/src/image/SkImage_Raster.cpp
|
f2608513626264459a00388537175600b515cae2 |
|
18-Sep-2015 |
tomhudson <tomhudson@google.com> |
Revert of add ImageShader, sharing code with its Bitmap cousin (patchset #10 id:180001 of https://codereview.chromium.org/1342113002/ ) Reason for revert: Failing ImageNewShaderTest on both Android (Tegra3 GPU) and iOS bots. e.g. /Users/chrome-bot/buildbot/skiabot-ipad4-000/build/slave/workdir/build/skia/tests/ ImageNewShaderTest.cpp:24 0 == memcmp(bm1.getPixels(), bm2.getPixels(), bm1.getSize()) ImageNewShaderTest.cpp:95 0xFFFF0000 == bmt.getColor(0, y) ImageNewShaderTest.cpp:98 0xFFDEDEDE == bmt.getColor(x, y) ImageNewShaderTest.cpp:98 0xFFDEDEDE == bmt.getColor(x, y) ImageNewShaderTest.cpp:98 0xFFDEDEDE == bmt.getColor(x, y) ImageNewShaderTest.cpp:98 0xFFDEDEDE == bmt.getColor(x, y) ImageNewShaderTest.cpp:95 0xFFFF0000 == bmt.getColor(0, y) ... Original issue's description: > add ImageShader, sharing code with its Bitmap cousin > > This is done by having abstracted the BitmapShaderContext to take a BitmapProvider, instead of just a bitmap. This allows us to share all of that code between SkBitmap and SkImage, since both are valid providers. > > It also means that we can simplify SkImage_Base to not need a virtual for onNewShader, since ALL images can uniformly be turned into a shader now. > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/0b93e3149d2cb30860c51f9f3204ae811d9a97ca TBR=fmalita@chromium.org,bsalomon@google.com,robertphillips@google.com,reed@google.com NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/1355863002
/external/skia/src/image/SkImage_Raster.cpp
|
0b93e3149d2cb30860c51f9f3204ae811d9a97ca |
|
18-Sep-2015 |
reed <reed@google.com> |
add ImageShader, sharing code with its Bitmap cousin This is done by having abstracted the BitmapShaderContext to take a BitmapProvider, instead of just a bitmap. This allows us to share all of that code between SkBitmap and SkImage, since both are valid providers. It also means that we can simplify SkImage_Base to not need a virtual for onNewShader, since ALL images can uniformly be turned into a shader now. BUG=skia: Review URL: https://codereview.chromium.org/1342113002
/external/skia/src/image/SkImage_Raster.cpp
|
746f31f18037c12f9fb79fbfb0cc0bbafc204484 |
|
16-Sep-2015 |
reed <reed@google.com> |
have raster-image return itself as a texture This tickles skbug.com/4351, but we don't know how to fix that yet, so we think this CL is ok. BUG=skia: Review URL: https://codereview.chromium.org/1348023004
/external/skia/src/image/SkImage_Raster.cpp
|
2bcab827872420762061f31200bedbd14e00b2c7 |
|
14-Sep-2015 |
reed <reed@google.com> |
remove code from SK_SUPPORT_LEGACY_NEWFROMGENERATOR, eliminates caller of deprecated SkInstallDiscardablePixelRef BUG=skia: Review URL: https://codereview.chromium.org/1344663002
/external/skia/src/image/SkImage_Raster.cpp
|
efd50daa2883ccadbed4bcffc789a76b24536a60 |
|
14-Sep-2015 |
reed <reed@google.com> |
impl preroll for all image backends BUG=skia: Review URL: https://codereview.chromium.org/1341043002
/external/skia/src/image/SkImage_Raster.cpp
|
85d9178832f4a64c9d80ffb14cb9dab4fe0fa54a |
|
10-Sep-2015 |
reed <reed@google.com> |
Use SkImageCacherator in SkImages Possible follow-up changes to consider 1. Roll SkImage_Raster and _Gpu into _Generator, where the generator (or cacherator) is backed by a pre-existing texture or raster. 2. Evolve SkImageUsageType into a verb requiring stretching, and have the caller (common code) digest the caps() and usage, so that subclasses are just told what to do (stretch or not) 3. Common code/utility to convert an unstretched texture into a stretch one (and cache it) if the generator can only make an unstretched one. BUG=skia: Review URL: https://codereview.chromium.org/1282363002
/external/skia/src/image/SkImage_Raster.cpp
|
a10bb86da82449b8756da5d00384993351f56d1d |
|
10-Sep-2015 |
reed <reed@google.com> |
make a shallow-copy so we don't get racy trying to lock/unlock our private bitmap BUG= 529995 NOTREECHECKS=True Review URL: https://codereview.chromium.org/1327313002
/external/skia/src/image/SkImage_Raster.cpp
|
86e90fafe17321964d975e9a6e2f242df280e74e |
|
01-Sep-2015 |
reed <reed@google.com> |
add preroll to image, for chrome's warmup pass BUG=skia: Review URL: https://codereview.chromium.org/1301373007
/external/skia/src/image/SkImage_Raster.cpp
|
96fcdcc219d2a0d3579719b84b28bede76efba64 |
|
27-Aug-2015 |
halcanary <halcanary@google.com> |
Style Change: NULL->nullptr DOCS_PREVIEW= https://skia.org/?cl=1316233002 Review URL: https://codereview.chromium.org/1316233002
/external/skia/src/image/SkImage_Raster.cpp
|
385fe4d4b62d7d1dd76116dd570df3290a2f487b |
|
26-Aug-2015 |
halcanary <halcanary@google.com> |
Style Change: SkNEW->new; SkDELETE->delete DOCS_PREVIEW= https://skia.org/?cl=1316123003 Review URL: https://codereview.chromium.org/1316123003
/external/skia/src/image/SkImage_Raster.cpp
|
ddbbddabefe865cc52e4b89a9d45455b987129bd |
|
20-Aug-2015 |
fmalita <fmalita@chromium.org> |
SkImage method for detecting lazy decoding BUG=skia:4224 R=reed@google.com Review URL: https://codereview.chromium.org/1305453007
/external/skia/src/image/SkImage_Raster.cpp
|
02d91d187324cd0f014a78eab1ca09ec71a9b356 |
|
31-Jul-2015 |
reed <reed@chromium.org> |
lock pixels in image when bitmap is immutable and not-lazy BUG=skia: Review URL: https://codereview.chromium.org/1266143003
/external/skia/src/image/SkImage_Raster.cpp
|
4ed3dd613e03ff2188c5061d07e9502cf040a2b1 |
|
31-Jul-2015 |
reed <reed@chromium.org> |
fix empty image constructor fBitmap was not fully constructed yet BUG=skia: TBR= Review URL: https://codereview.chromium.org/1264773004
/external/skia/src/image/SkImage_Raster.cpp
|
80c772b2a46aea7efe0632e580fbc8233ff2a190 |
|
31-Jul-2015 |
reed <reed@google.com> |
unify pixelref and image ID space, so we can share IDs when we share pixels I view this as a performance opportunity, not a feature or bug fix per-se. BUG=skia: Review URL: https://codereview.chromium.org/1266883002
/external/skia/src/image/SkImage_Raster.cpp
|
26e0e587f76f2a9338652c100f835c2377c908d3 |
|
29-Jul-2015 |
reed <reed@google.com> |
SkImage_Raster's pixels are always immutable. To make this work, we tag their pixelrefs as temporarily immutable, allowing ourselves to restore the pixels to mutability only when the image drops away. This should allow us to wobble back and forth between writing to the Surface and reading from the Image without a COW, with the Surface seeing mutable pixels and the Image seeing immutable pixels. The big idea is, Image doesn't need forever-immutable pixels, it just needs pixels that are immutable as long as it's alive. BUG=skia: patch from issue 804523002 at patchset 40001 (http://crrev.com/804523002#ps40001) Review URL: https://codereview.chromium.org/1254383006
/external/skia/src/image/SkImage_Raster.cpp
|
9a5d1ab54d52a912bb3ac9f74ee01bba079639e5 |
|
27-Jul-2015 |
fmalita <fmalita@chromium.org> |
Make peekPixels() usable with raster surface snapshots SkSurface_Raster snapshots do not lock their backing bitmaps when the pixel ref is shared - they only lock on deep-copy. But since for raster surfaces the pixels are always in memory, I think it would be OK to also lock in the former case. This allows for optimized (zero-copy) reads of raster surface snapshot data. R=reed@google.com Review URL: https://codereview.chromium.org/1256993002
/external/skia/src/image/SkImage_Raster.cpp
|
ea561bf055bb803f4c10ca323ea60a9d94da7956 |
|
10-Jul-2015 |
fmalita <fmalita@chromium.org> |
Fix SkImage::asLegacyBitmap() rowBytes assert When reusing the pixel ref, we need to observe its rowBytes. R=reed@google.com Review URL: https://codereview.chromium.org/1227413002
/external/skia/src/image/SkImage_Raster.cpp
|
3c06511e910524bf9456c532fe20d1b2956e7d97 |
|
08-Jul-2015 |
reed <reed@google.com> |
Add image->bitmap BUG=skia: patch from issue 1212163012 at patchset 1 (http://crrev.com/1212163012#ps1) Review URL: https://codereview.chromium.org/1208993017
/external/skia/src/image/SkImage_Raster.cpp
|
5617900063f69754de62b8b12d32f6e36df14104 |
|
07-Jul-2015 |
reed <reed@chromium.org> |
add SkImage::NewFromBitmap BUG=skia: Review URL: https://codereview.chromium.org/1217053003
/external/skia/src/image/SkImage_Raster.cpp
|
759373a9fe73a7883d007254193084a541714b40 |
|
04-Jul-2015 |
reed <reed@chromium.org> |
add colortable param to newrastercopy BUG=skia: TBR= NOTREECHECKS=True Review URL: https://codereview.chromium.org/1222683004
/external/skia/src/image/SkImage_Raster.cpp
|
871872f3f247f6b699617f6d9ef50ef5da6fbe74 |
|
22-Jun-2015 |
reed <reed@google.com> |
change old picture serialization to really handle images BUG=skia:3965 Review URL: https://codereview.chromium.org/1199473002
/external/skia/src/image/SkImage_Raster.cpp
|
de49988bc24977f97bb1da8c17ceee5432fcc8e7 |
|
18-Jun-2015 |
reed <reed@google.com> |
add callbacks to Images that wrap client-provided content BUG=skia: Review URL: https://codereview.chromium.org/1169553003
/external/skia/src/image/SkImage_Raster.cpp
|
a85d4d0814818e4ddabb9237da209d61d6cd5854 |
|
06-May-2015 |
reed <reed@google.com> |
Make drawImage a virtual on SkDevice This reverts commit 06dad4b92f1fcb593bc686f9537d3e0d866b2241. BUG=485243 Review URL: https://codereview.chromium.org/1129843003
/external/skia/src/image/SkImage_Raster.cpp
|
06dad4b92f1fcb593bc686f9537d3e0d866b2241 |
|
06-May-2015 |
reed <reed@google.com> |
Revert of Revert of Revert of Make drawImage a virtual on SkDevice (patchset #1 id:1 of https://codereview.chromium.org/1126273002/) Reason for revert: still breaking webglconformance Original issue's description: > Revert of Revert of Make drawImage a virtual on SkDevice (patchset #1 id:1 of https://codereview.chromium.org/1124003002/) > > Reason for revert: > retry now that some fixes (onDrawImage overrides) have landed in chrome/blink > > Original issue's description: > > Revert of Make drawImage a virtual on SkDevice (patchset #4 id:60001 of https://codereview.chromium.org/1122643005/) > > > > Reason for revert: > > speculative to see if it unblocks DEPS roll > > > > Original issue's description: > > > Make drawImage a virtual on SkDevice > > > > > > Now with patch for SkDeferredCanvas > > > > > > This reverts commit 119468b71f8f4f45657ab30ead331be665de5a57. > > > > > > BUG=skia: > > > > > > Committed: https://skia.googlesource.com/skia/+/14fe8fd3e53b5e988aac189a8bc3ed28904d85c8 > > > > TBR=robertphillips@google.com,mtklein@google.com,reed@google.com > > NOPRESUBMIT=true > > NOTREECHECKS=true > > NOTRY=true > > BUG=skia: > > > > Committed: https://skia.googlesource.com/skia/+/3538e3bfe2e00bc1b5b48d977fa7adff64d8c96b > > TBR=robertphillips@google.com,mtklein@google.com,reed@chromium.org > NOPRESUBMIT=true > NOTREECHECKS=true > NOTRY=true > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/620ba3afe09d4173c87537040fe50c1c1895fb1a TBR=robertphillips@google.com,mtklein@google.com,reed@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/1127993004
/external/skia/src/image/SkImage_Raster.cpp
|
620ba3afe09d4173c87537040fe50c1c1895fb1a |
|
06-May-2015 |
reed <reed@google.com> |
Revert of Revert of Make drawImage a virtual on SkDevice (patchset #1 id:1 of https://codereview.chromium.org/1124003002/) Reason for revert: retry now that some fixes (onDrawImage overrides) have landed in chrome/blink Original issue's description: > Revert of Make drawImage a virtual on SkDevice (patchset #4 id:60001 of https://codereview.chromium.org/1122643005/) > > Reason for revert: > speculative to see if it unblocks DEPS roll > > Original issue's description: > > Make drawImage a virtual on SkDevice > > > > Now with patch for SkDeferredCanvas > > > > This reverts commit 119468b71f8f4f45657ab30ead331be665de5a57. > > > > BUG=skia: > > > > Committed: https://skia.googlesource.com/skia/+/14fe8fd3e53b5e988aac189a8bc3ed28904d85c8 > > TBR=robertphillips@google.com,mtklein@google.com,reed@google.com > NOPRESUBMIT=true > NOTREECHECKS=true > NOTRY=true > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/3538e3bfe2e00bc1b5b48d977fa7adff64d8c96b TBR=robertphillips@google.com,mtklein@google.com,reed@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/1126273002
/external/skia/src/image/SkImage_Raster.cpp
|
3538e3bfe2e00bc1b5b48d977fa7adff64d8c96b |
|
06-May-2015 |
reed <reed@chromium.org> |
Revert of Make drawImage a virtual on SkDevice (patchset #4 id:60001 of https://codereview.chromium.org/1122643005/) Reason for revert: speculative to see if it unblocks DEPS roll Original issue's description: > Make drawImage a virtual on SkDevice > > Now with patch for SkDeferredCanvas > > This reverts commit 119468b71f8f4f45657ab30ead331be665de5a57. > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/14fe8fd3e53b5e988aac189a8bc3ed28904d85c8 TBR=robertphillips@google.com,mtklein@google.com,reed@google.com NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/1124003002
/external/skia/src/image/SkImage_Raster.cpp
|
14fe8fd3e53b5e988aac189a8bc3ed28904d85c8 |
|
05-May-2015 |
reed <reed@google.com> |
Make drawImage a virtual on SkDevice Now with patch for SkDeferredCanvas This reverts commit 119468b71f8f4f45657ab30ead331be665de5a57. BUG=skia: Review URL: https://codereview.chromium.org/1122643005
/external/skia/src/image/SkImage_Raster.cpp
|
119468b71f8f4f45657ab30ead331be665de5a57 |
|
05-May-2015 |
reed <reed@google.com> |
Revert of Make drawImage a virtual on SkDevice (patchset #3 id:40001 of https://codereview.chromium.org/11188… (patchset #2 id:20001 of https://codereview.chromium.org/1122813002/) Reason for revert: did not update DeferredCanvas Original issue's description: > Make drawImage a virtual on SkDevice (patchset #3 id:40001 of https://codereview.chromium.org/1118823004/)" > > Fixed serialization from prev CL > > This reverts commit 973d1f1f60f2710a268a3165aa4f3460502d4be5. > > BUG=skia:3803 > TBR= > > Committed: https://skia.googlesource.com/skia/+/5392785080001fe737fac9e5801fc2127a78d4fb TBR= NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia:3803 Review URL: https://codereview.chromium.org/1123743006
/external/skia/src/image/SkImage_Raster.cpp
|
5392785080001fe737fac9e5801fc2127a78d4fb |
|
04-May-2015 |
reed <reed@google.com> |
Make drawImage a virtual on SkDevice (patchset #3 id:40001 of https://codereview.chromium.org/1118823004/)" Fixed serialization from prev CL This reverts commit 973d1f1f60f2710a268a3165aa4f3460502d4be5. BUG=skia:3803 TBR= Review URL: https://codereview.chromium.org/1122813002
/external/skia/src/image/SkImage_Raster.cpp
|
973d1f1f60f2710a268a3165aa4f3460502d4be5 |
|
04-May-2015 |
reed <reed@google.com> |
Revert of make drawImage a virtual on SkDevice (patchset #3 id:40001 of https://codereview.chromium.org/1118823004/) Reason for revert: need to make serialization work for pictures, now that images are "real" Original issue's description: > make drawImage a virtual on SkDevice > > default impl will turn the Image into a (raster) bitmap and call drawBitmap. > > BUG=skia:3803 > > Committed: https://skia.googlesource.com/skia/+/f603fb3d0e5fa57c3f228d66945bef198b001135 TBR=bsalomon@google.com,robertphillips@google.com NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia:3803 Review URL: https://codereview.chromium.org/1122803002
/external/skia/src/image/SkImage_Raster.cpp
|
f603fb3d0e5fa57c3f228d66945bef198b001135 |
|
04-May-2015 |
reed <reed@google.com> |
make drawImage a virtual on SkDevice default impl will turn the Image into a (raster) bitmap and call drawBitmap. BUG=skia:3803 Review URL: https://codereview.chromium.org/1118823004
/external/skia/src/image/SkImage_Raster.cpp
|
7831a4bbb46a919316efc7c70808dcaaf8ee629a |
|
27-Apr-2015 |
tfarina <tfarina@chromium.org> |
Update more directories under src/ to follow C++11 style rule for {virtual,override}. The Google style guide states that only one of {virtual,override,final} should be used for each declaration, since override implies virtual and final implies both virtual and override. The entries were found using the following command line: $ find src/ -iname "*.h" -o -iname "*.cpp" | xargs pcregrep -M "[^\n/]+virtual\ [^;{]+\ [a-zA-Z0-9_]+\([^;{]+\ override[ \n]*[;{]" The regex was a courtesy of nick@chromium.org BUG=None R=mtklein@google.com NOPRESUBMIT=true Review URL: https://codereview.chromium.org/1086143003
/external/skia/src/image/SkImage_Raster.cpp
|
36352bf5e38f45a70ee4f4fc132a38048d38206d |
|
26-Mar-2015 |
mtklein <mtklein@chromium.org> |
C++11 override should now be supported by all of {bots,Chrome,Android,Mozilla} NOPRESUBMIT=true BUG=skia: DOCS_PREVIEW= https://skia.org/?cl=1037793002 Review URL: https://codereview.chromium.org/1037793002
/external/skia/src/image/SkImage_Raster.cpp
|
73953e7d00f0a75a98e985ceed88059d35945327 |
|
24-Feb-2015 |
kkinnunen <kkinnunen@nvidia.com> |
Make SkNewImageFromBitmap take pixel ref origin into account Make SkNewImageFromBitmap take pixel ref origin into account. BUG=skia:3388 Review URL: https://codereview.chromium.org/951483002
/external/skia/src/image/SkImage_Raster.cpp
|
9e447c08deb30b26a2101cb20b8a04aa230f09c4 |
|
15-Feb-2015 |
reed <reed@chromium.org> |
remove unneeded preroll BUG=skia: TBR=robertphilips Review URL: https://codereview.chromium.org/927443006
/external/skia/src/image/SkImage_Raster.cpp
|
96a857ef5a40b3d0751ec9ebaaa96d9f20b82b21 |
|
25-Jan-2015 |
reed <reed@chromium.org> |
initial preroll api BUG=skia: Review URL: https://codereview.chromium.org/855473002
/external/skia/src/image/SkImage_Raster.cpp
|
7983bf914a6df040befe2aafb3657c469b678257 |
|
23-Jan-2015 |
reed <reed@google.com> |
more reinforcement that images never have zero width or height BUG=skia: Review URL: https://codereview.chromium.org/869743002
/external/skia/src/image/SkImage_Raster.cpp
|
72c9faab45124e08c85f70ca38536914862d947c |
|
09-Jan-2015 |
mtklein <mtklein@chromium.org> |
Fix up all the easy virtual ... SK_OVERRIDE cases. This fixes every case where virtual and SK_OVERRIDE were on the same line, which should be the bulk of cases. We'll have to manually clean up the rest over time unless I level up in regexes. for f in (find . -type f); perl -p -i -e 's/virtual (.*)SK_OVERRIDE/\1SK_OVERRIDE/g' $f; end BUG=skia: Review URL: https://codereview.chromium.org/806653007
/external/skia/src/image/SkImage_Raster.cpp
|
5965c8ae4ee960275da4bc40189bdba85aab8b5e |
|
08-Jan-2015 |
reed <reed@google.com> |
add ImageGenerator::NewFromData to porting layer BUG=skia:3275 Review URL: https://codereview.chromium.org/834633006
/external/skia/src/image/SkImage_Raster.cpp
|
b2497c2d945c0a5d770865b026e2d5947bf37c91 |
|
31-Dec-2014 |
reed <reed@chromium.org> |
change API contract: disallow zero-sized images or surfaces BUG=skia: Review URL: https://codereview.chromium.org/830033003
/external/skia/src/image/SkImage_Raster.cpp
|
b4f629cd688b680eba21a5a08daeaf82bbe65b28 |
|
25-Dec-2014 |
reed <reed@chromium.org> |
fix images in multi-thread by doing shallow-copies BUG=skia: TBR=mtklein@google.com Review URL: https://codereview.chromium.org/791473006
/external/skia/src/image/SkImage_Raster.cpp
|
96472deea70169396b8e1f576e470138f55fdb1f |
|
10-Dec-2014 |
reed <reed@google.com> |
dd readPixels to SkImage patch from issue 789673007 at patchset 1 (http://crrev.com/789673007#ps1) BUG=skia: Review URL: https://codereview.chromium.org/793723002
/external/skia/src/image/SkImage_Raster.cpp
|
4af267b11964d4a8acdb232ac46094c84d890e88 |
|
21-Nov-2014 |
reed <reed@google.com> |
add SkImage::newSurface BUG=skia: Review URL: https://codereview.chromium.org/741763002
/external/skia/src/image/SkImage_Raster.cpp
|
65151754b9fdb6a968d7307764c20655d1b680a0 |
|
16-Oct-2014 |
piotaixr <piotaixr@chromium.org> |
Override SkCanvas::drawImage() in SkRecorder. BUG=skia:2947 Review URL: https://codereview.chromium.org/610003002
/external/skia/src/image/SkImage_Raster.cpp
|
5ceff913cfd458c653358db7f60e995f505bce84 |
|
26-Sep-2014 |
piotaixr <piotaixr@chromium.org> |
Rename SkImage::draw(_, SkRect, SkRect) in SkImage::drawRect() BUG=skia:2947 R=reed@google.com, junov@chromium.org Author: piotaixr@chromium.org Review URL: https://codereview.chromium.org/605843002
/external/skia/src/image/SkImage_Raster.cpp
|
a29b5d8430ada72bc73b1e6e1b8f09e4b046b2ff |
|
22-Sep-2014 |
piotaixr <piotaixr@chromium.org> |
Cleaning includes in SkImage-related files BUG=skia:2948 R=junov@chromium.org, tfarina@chromium.org, reed@google.com Author: piotaixr@chromium.org Review URL: https://codereview.chromium.org/586853003
/external/skia/src/image/SkImage_Raster.cpp
|
e5ea500d4714a7d84de2bf913e81be3b65d2de68 |
|
03-Sep-2014 |
reed <reed@google.com> |
Hide fields in SkImageInfo R=rmistry@google.com TBR=bsalomon Author: reed@google.com Review URL: https://codereview.chromium.org/536003002
/external/skia/src/image/SkImage_Raster.cpp
|
d2a3522503ca0c39829f1bb41f01201d1affdaf6 |
|
19-Aug-2014 |
piotaixr <piotaixr@chromium.org> |
Add the method isOpaque() to SkImage BUG=skia:2766 R=junov@chromium.org, halcanary@google.com, scroggo@google.com, reed@google.com, bsalomon@google.com Author: piotaixr@chromium.org Review URL: https://codereview.chromium.org/406673003
/external/skia/src/image/SkImage_Raster.cpp
|
ea4673fde65f15d6f1ca77e24ced7348c4914517 |
|
18-Aug-2014 |
halcanary <halcanary@google.com> |
SkImage::NewFromGenerator(SkImageGenerator*), and a unit test. R=reed@google.com Author: halcanary@google.com Review URL: https://codereview.chromium.org/465823003
/external/skia/src/image/SkImage_Raster.cpp
|
78ee93f5454c93136efff5970553ac6105950247 |
|
12-Aug-2014 |
halcanary <halcanary@google.com> |
Revert of SkImage::NewFromGenerator(SkImageGenerator*), and a unit test. (https://codereview.chromium.org/465823003/) Reason for revert: Mac bots are failing Original issue's description: > SkImage::NewFromGenerator(SkImageGenerator*), and a unit test. > > Committed: https://skia.googlesource.com/skia/+/186f7b04956a1742f185a4ca69b44b52bc50e7fc R=reed@google.com TBR=reed@google.com NOTREECHECKS=true NOTRY=true Author: halcanary@google.com Review URL: https://codereview.chromium.org/453723003
/external/skia/src/image/SkImage_Raster.cpp
|
186f7b04956a1742f185a4ca69b44b52bc50e7fc |
|
12-Aug-2014 |
halcanary <halcanary@google.com> |
SkImage::NewFromGenerator(SkImageGenerator*), and a unit test. R=reed@google.com Author: halcanary@google.com Review URL: https://codereview.chromium.org/465823003
/external/skia/src/image/SkImage_Raster.cpp
|
b2edec2c396f9be3e527179bd0e5c866b59ae795 |
|
12-Aug-2014 |
halcanary <halcanary@google.com> |
Revert of SkImage_Codec is Lazy (https://codereview.chromium.org/460823002/) Reason for revert: breaks chromium - no SkDecodingImageGenerator::Create Original issue's description: > SkImage_Codec is Lazy > > Committed: https://skia.googlesource.com/skia/+/e36f499110da8c2e2aa05227bee6deb967309ead R=reed@google.com TBR=reed@google.com NOTREECHECKS=true NOTRY=true Author: halcanary@google.com Review URL: https://codereview.chromium.org/461043002
/external/skia/src/image/SkImage_Raster.cpp
|
e36f499110da8c2e2aa05227bee6deb967309ead |
|
12-Aug-2014 |
halcanary <halcanary@google.com> |
SkImage_Codec is Lazy R=reed@google.com Author: halcanary@google.com Review URL: https://codereview.chromium.org/460823002
/external/skia/src/image/SkImage_Raster.cpp
|
8572fc01ac4f6bdcf173b05417776abc55f729c1 |
|
11-Aug-2014 |
reed <reed@google.com> |
mark all SkImage methods const, so we can make it thread-safe BUG=skia: R=mtklein@google.com, halcanary@google.com Author: reed@google.com Review URL: https://codereview.chromium.org/453613003
/external/skia/src/image/SkImage_Raster.cpp
|
76d5b477c92ef076374fc7ea9d551c798656bebd |
|
23-Jul-2014 |
piotaixr <piotaixr@chromium.org> |
Now able to set the localMatrix when creating a SkShader from a SkImage BUG=skia:2771 R=junov@chromium.org, reed@chromium.org, bsalomon@chromium.org, bsalomon@google.com Author: piotaixr@chromium.org Review URL: https://codereview.chromium.org/409653003
/external/skia/src/image/SkImage_Raster.cpp
|
cef04f818820e59fb534eb226f967f72497845d3 |
|
14-Jul-2014 |
piotaixr <piotaixr@chromium.org> |
Deleted SkImage::newShaderClamp: not used, not implemented. Implemented SkImage::newShader. BUG=skia:2701, 344804 R=junov@chromium.org, reed@chromium.org, bsalomon@chromium.org, reed@google.com Author: piotaixr@chromium.org Review URL: https://codereview.chromium.org/345463009
/external/skia/src/image/SkImage_Raster.cpp
|
a3264e53ee3f3c5d6a2c813df7e44b5b96d207f2 |
|
30-May-2014 |
commit-bot@chromium.org <commit-bot@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81> |
Revert "Revert of setConfig -> setInfo (https://codereview.chromium.org/308683005/)" This reverts commit eecaea4148805834f223681f70b6488ceba12d09. R=robertphillips@google.com, scroggo@google.com Author: reed@google.com Review URL: https://codereview.chromium.org/301283003 git-svn-id: http://skia.googlecode.com/svn/trunk@14989 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/image/SkImage_Raster.cpp
|
d0f824cfbd8951bb948b19d7ff5dce028ce557a5 |
|
29-May-2014 |
commit-bot@chromium.org <commit-bot@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81> |
Revert of setConfig -> setInfo (https://codereview.chromium.org/308683005/) Reason for revert: broke all Windows bots Original issue's description: > setConfig -> setInfo > > BUG=skia: > > Committed: http://code.google.com/p/skia/source/detail?r=14978 R=robertphillips@google.com, reed@google.com TBR=reed@google.com, robertphillips@google.com NOTREECHECKS=true NOTRY=true BUG=skia: Author: epoger@google.com Review URL: https://codereview.chromium.org/302053002 git-svn-id: http://skia.googlecode.com/svn/trunk@14979 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/image/SkImage_Raster.cpp
|
986d681f3e7a7eb6febd9df9fe21a8d277ba2296 |
|
29-May-2014 |
commit-bot@chromium.org <commit-bot@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81> |
setConfig -> setInfo BUG=skia: R=robertphillips@google.com Author: reed@google.com Review URL: https://codereview.chromium.org/308683005 git-svn-id: http://skia.googlecode.com/svn/trunk@14978 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/image/SkImage_Raster.cpp
|
00f8d6c75d22ce8f95f932c5b101354b196fa0df |
|
29-May-2014 |
commit-bot@chromium.org <commit-bot@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81> |
Revert "Revert of add colortable support to imagegenerator (https://codereview.chromium.org/304443003/)" Fix is to add colortable param to installPixels() This reverts commit 924205aaf2e0c3c65dda13e0eaccde3e7b2a5c40. BUG=skia: R=scroggo@google.com, reed@chromium.org Author: reed@google.com Review URL: https://codereview.chromium.org/300263005 git-svn-id: http://skia.googlecode.com/svn/trunk@14958 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/image/SkImage_Raster.cpp
|
466f5f3e44e703ca58b43ac1c4ac3bfa0e1ff024 |
|
27-May-2014 |
commit-bot@chromium.org <commit-bot@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81> |
remove SkBitmap::asImageInfo BUG=skia: R=scroggo@google.com, halcanary@google.com Author: reed@google.com Review URL: https://codereview.chromium.org/305483005 git-svn-id: http://skia.googlecode.com/svn/trunk@14909 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/image/SkImage_Raster.cpp
|
8a2ad3cae710f05cca57e48dd1732d575dba2dc7 |
|
23-Feb-2014 |
commit-bot@chromium.org <commit-bot@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81> |
add new copyTo version to SkBitmap, which takes SkColorType BUG=skia: R=scroggo@google.com, halcanary@google.com, bsalomon@google.com Author: reed@google.com Review URL: https://codereview.chromium.org/171723007 git-svn-id: http://skia.googlecode.com/svn/trunk@13553 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/image/SkImage_Raster.cpp
|
2776e01d2c8ae6a23718cd4f4d9a4bf368a8e46b |
|
06-Feb-2014 |
reed@google.com <reed@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> |
return NULL not false for a void* failure git-svn-id: http://skia.googlecode.com/svn/trunk@13341 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/image/SkImage_Raster.cpp
|
4f7c61583b16e3056cf3350fcef42dcc6d3483b7 |
|
06-Feb-2014 |
reed@google.com <reed@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> |
add ways to peer into an image to get its pixels BUG=skia: R=bsalomon@google.com, scroggo@google.com Review URL: https://codereview.chromium.org/155763004 git-svn-id: http://skia.googlecode.com/svn/trunk@13339 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/image/SkImage_Raster.cpp
|
999da9c5e45fb533efe8782c9096794b9ad1c1b3 |
|
06-Feb-2014 |
reed@google.com <reed@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> |
fix refcounting in SkImage::NewRasterData git-svn-id: http://skia.googlecode.com/svn/trunk@13338 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/image/SkImage_Raster.cpp
|
e13af711d4ff9031c9ed3054a4c33a56a0c62e1f |
|
13-Jan-2014 |
commit-bot@chromium.org <commit-bot@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81> |
Revert of https://codereview.chromium.org/132643007/ Reason for revert: broken tests TBR= NOTREECHECKS=true NOTRY=true BUG= Author: reed@google.com Review URL: https://codereview.chromium.org/134843008 git-svn-id: http://skia.googlecode.com/svn/trunk@13058 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/image/SkImage_Raster.cpp
|
f0b56e74485d59465194b21f89fa3bc3a7962ac6 |
|
13-Jan-2014 |
reed@google.com <reed@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> |
Revert "Revert of https://codereview.chromium.org/129423002/" fixed new api references in unit tests This reverts commit ffc0058e1fbcbd69617e1f41b2dce5b5765ff99e. BUG= Review URL: https://codereview.chromium.org/132643007 git-svn-id: http://skia.googlecode.com/svn/trunk@13057 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/image/SkImage_Raster.cpp
|
9b06ba4c91484d09565ff4a572d8c5af15dd429e |
|
13-Jan-2014 |
commit-bot@chromium.org <commit-bot@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81> |
Revert of https://codereview.chromium.org/129423002/ Reason for revert: broke tests (compile) R=scroggo@google.com, halcanary@google.com TBR=halcanary@google.com, scroggo@google.com NOTREECHECKS=true NOTRY=true BUG= Author: reed@google.com Review URL: https://codereview.chromium.org/137133003 git-svn-id: http://skia.googlecode.com/svn/trunk@13056 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/image/SkImage_Raster.cpp
|
dd9ea9262cb61b545fcf414fbde677eb2b62fee4 |
|
13-Jan-2014 |
reed@google.com <reed@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> |
add SkBitmap::installPixelRef() BUG= R=scroggo@google.com Review URL: https://codereview.chromium.org/129423002 git-svn-id: http://skia.googlecode.com/svn/trunk@13055 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/image/SkImage_Raster.cpp
|
f1901788d8753332c5748a30e8d14e290c155bd1 |
|
08-Jan-2014 |
commit-bot@chromium.org <commit-bot@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81> |
use SkImageInfo directly, instead of convert to Config BUG= R=scroggo@google.com Author: reed@google.com Review URL: https://codereview.chromium.org/128953003 git-svn-id: http://skia.googlecode.com/svn/trunk@12977 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/image/SkImage_Raster.cpp
|
672588b684d484dce6ae251e9e163e4a46924322 |
|
08-Jan-2014 |
reed@google.com <reed@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> |
change offset to xy for pixelref subsetting BUG= R=scroggo@google.com Review URL: https://codereview.chromium.org/105893012 git-svn-id: http://skia.googlecode.com/svn/trunk@12958 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/image/SkImage_Raster.cpp
|
1bed687f6b8fc67336f0f5d6fb5a5b38dd0fdff9 |
|
02-Jan-2014 |
halcanary@google.com <halcanary@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> |
Add a release procedure to SkMallocPixelRef; remove SkDataPixelRef This works in a way that is similar to SkData. SkMallocPixelRef::NewWithProc Motivation: Chrome has a ETC1PixelRef which calls delete[] on the pixles on destruction. There is no reason for them to almost duplicate our class, when we can provide them a more flexible class. Example use: static void delete_uint8_proc(void* ptr, void*) { delete[] static_cast<uint8_t>(ptr); } SkPixelRef* new_delete_pixref(const SkImageInfo& info, SkColorTable* ctable) { size_t rb = info.minRowBytes(); return SkMallocPixelRef::NewWithProc( info, rb, ctable, new uint8_t[info.getSafeSize(rb)], delete_uint8_proc, NULL); } SkMallocPixelRef::NewWithData Motivation: This allows up to eliminate SkDataPixelRef. We modified SkImage_Raster to use MallocPixelRef rather than SkDataPixlRef. Also: Unit tests in tests/MallocPixelRefTest. BUG= R=reed@google.com Review URL: https://codereview.chromium.org/106883006 git-svn-id: http://skia.googlecode.com/svn/trunk@12861 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/image/SkImage_Raster.cpp
|
bf790232f6d94b54239dbc210d8beee7411ca458 |
|
13-Dec-2013 |
reed@google.com <reed@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> |
Update all callsites to use info for pixelrefs #define SK_SUPPORT_LEGACY_PIXELREF_CONSTRUCTOR in chrome to keep old API signature (for now) BUG= R=scroggo@google.com Review URL: https://codereview.chromium.org/100723005 git-svn-id: http://skia.googlecode.com/svn/trunk@12677 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/image/SkImage_Raster.cpp
|
0daa1adb03b4b1fc11d854cb7754416ac05a31e8 |
|
13-Dec-2013 |
robertphillips@google.com <robertphillips@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> |
Reverting r12665 & r12666 (Remove duplicate impl for SkImageInfo flattening) due to Chromium/Blink compilation errors https://codereview.chromium.org/112603003/ git-svn-id: http://skia.googlecode.com/svn/trunk@12667 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/image/SkImage_Raster.cpp
|
93b40594b78ab40170b5a7fa0811fa95ab8fc30f |
|
13-Dec-2013 |
reed@google.com <reed@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> |
Remvoe duplicate impl for SkImageInfo flattening Add onNewLockPixels This reverts commit bb8eff6a70c52b7644391cfd4f4d21bf7294a6bf. BUG= Review URL: https://codereview.chromium.org/111323005 git-svn-id: http://skia.googlecode.com/svn/trunk@12665 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/image/SkImage_Raster.cpp
|
398337b3a5d66a7db5d36a3b5182e4407e2292f7 |
|
11-Dec-2013 |
reed@google.com <reed@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> |
Revert "remvoe duplicate impl for SkImageInfo flattening" Reason: breaks chrome_mac_tests which still have non-imageinfo constructors This reverts commit a06b8cf60b39bda93e9ef1a73579007b2b930d29. BUG= Review URL: https://codereview.chromium.org/103033005 git-svn-id: http://skia.googlecode.com/svn/trunk@12631 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/image/SkImage_Raster.cpp
|
f3aead2829fa515f790631ee65784a7b9b073343 |
|
11-Dec-2013 |
reed@google.com <reed@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> |
remvoe duplicate impl for SkImageInfo flattening Revert "Revert "PixelRef now returns (nearly) everything that is currently in SkBitmap. The goal is to refactor bitmap later to remove redundancy, and more interestingly, remove the chance for a disconnect between the actual (pixelref) rowbytes and config, and the one claimed by the bitmap."""""" This reverts commit eabd6b2ed4e494b323c08f32358f45950a0368c3. BUG= Review URL: https://codereview.chromium.org/108773003 git-svn-id: http://skia.googlecode.com/svn/trunk@12624 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/image/SkImage_Raster.cpp
|
6a32add491cc736aab95561d137909cd7c5d65ef |
|
11-Dec-2013 |
reed@google.com <reed@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> |
Revert "PixelRef now returns (nearly) everything that is currently in SkBitmap. The goal is to refactor bitmap later to remove redundancy, and more interestingly, remove the chance for a disconnect between the actual (pixelref) rowbytes and config, and the one claimed by the bitmap.""""" This reverts commit d08eca87a0bef10112a211de540f89656a80b86a. BUG= Review URL: https://codereview.chromium.org/108303003 git-svn-id: http://skia.googlecode.com/svn/trunk@12623 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/image/SkImage_Raster.cpp
|
6965a0a2df9d35cd0a25e1738f0388272d03f399 |
|
11-Dec-2013 |
reed@google.com <reed@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> |
PixelRef now returns (nearly) everything that is currently in SkBitmap. The goal is to refactor bitmap later to remove redundancy, and more interestingly, remove the chance for a disconnect between the actual (pixelref) rowbytes and config, and the one claimed by the bitmap."""" BUG= Review URL: https://codereview.chromium.org/110503003 git-svn-id: http://skia.googlecode.com/svn/trunk@12622 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/image/SkImage_Raster.cpp
|
2ebc10dd12a6fe55fda37a9bd5b1e0c1c1e08232 |
|
06-Dec-2013 |
reed@google.com <reed@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> |
Revert "Revert "Revert "PixelRef now returns (nearly) everything that is currently in SkBitmap. The goal is to refactor bitmap later to remove redundancy, and more interestingly, remove the chance for a disconnect between the actual (pixelref) rowbytes and config, and the one claimed by the bitmap.""" This reverts commit 5f035e90d6dea0139a4f204b634e7b7b3b4976d7. Reverting because it breaks chrome/blink due to new SkPixelRef constructor arg. BUG= Review URL: https://codereview.chromium.org/108993002 git-svn-id: http://skia.googlecode.com/svn/trunk@12551 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/image/SkImage_Raster.cpp
|
473f0aa2bb218e50fce5e19063f8c8fdaf57fad4 |
|
06-Dec-2013 |
reed@google.com <reed@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> |
Revert "Revert "PixelRef now returns (nearly) everything that is currently in SkBitmap. The goal is to refactor bitmap later to remove redundancy, and more interestingly, remove the chance for a disconnect between the actual (pixelref) rowbytes and config, and the one claimed by the bitmap."" This reverts commit 4174afb18a9746bbad2a06c0ec2d4ad35f72d790. BUG= Review URL: https://codereview.chromium.org/108723003 git-svn-id: http://skia.googlecode.com/svn/trunk@12547 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/image/SkImage_Raster.cpp
|
5b132b28da61e89cdcb84e95f30a58f8184739fc |
|
06-Dec-2013 |
reed@google.com <reed@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> |
Revert "PixelRef now returns (nearly) everything that is currently in SkBitmap. The goal is to refactor bitmap later to remove redundancy, and more interestingly, remove the chance for a disconnect between the actual (pixelref) rowbytes and config, and the one claimed by the bitmap." This reverts commit 154e08b2f5904ef533da694e3510befcb9a3f3e2. revert due to warnings Review URL: https://codereview.chromium.org/108513003 git-svn-id: http://skia.googlecode.com/svn/trunk@12538 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/image/SkImage_Raster.cpp
|
3e89524e747b513986abfeeea00b6fac79593f26 |
|
06-Dec-2013 |
reed@google.com <reed@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> |
PixelRef now returns (nearly) everything that is currently in SkBitmap. The goal is to refactor bitmap later to remove redundancy, and more interestingly, remove the chance for a disconnect between the actual (pixelref) rowbytes and config, and the one claimed by the bitmap. R=mtklein@google.com, scroggo@google.com Review URL: https://codereview.chromium.org/68973005 git-svn-id: http://skia.googlecode.com/svn/trunk@12537 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/image/SkImage_Raster.cpp
|
2bd8b8100529c96c81c30f749f672f4caf775b04 |
|
01-Nov-2013 |
reed@google.com <reed@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> |
move SkImage::ColorType into SkColorType objective -- move clients over to SkImage tasks - use SkImageInfo instead of SkBitmap::Config - add support for colortables to SkImage - add drawImage to SkCanvas - return SkImage from readPixels This CL works towards the first task R=robertphillips@google.com Review URL: https://codereview.chromium.org/54363008 git-svn-id: http://skia.googlecode.com/svn/trunk@12077 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/image/SkImage_Raster.cpp
|
383a697692cf46951fd451f6f4c3d03634a6a1cb |
|
21-Oct-2013 |
reed@google.com <reed@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> |
store SkAlphaType inside SkBitmap, on road to support unpremul BUG= R=bsalomon@google.com, scroggo@google.com Review URL: https://codereview.chromium.org/25275004 git-svn-id: http://skia.googlecode.com/svn/trunk@11877 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/image/SkImage_Raster.cpp
|
d28ba8010c6058bf073f7e815d5b2d7fdf698601 |
|
20-Sep-2013 |
reed@google.com <reed@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> |
promote SkImage::AlphaType to SkAlphaType BUG= R=bsalomon@google.com Review URL: https://codereview.chromium.org/24130009 git-svn-id: http://skia.googlecode.com/svn/trunk@11421 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/image/SkImage_Raster.cpp
|
dfec28d4a9e05d1d525f377b380b4df3c5e07c7b |
|
23-Jul-2013 |
commit-bot@chromium.org <commit-bot@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81> |
Add SkImage->draw() call with src and dst rects. Committed: http://code.google.com/p/skia/source/detail?r=10237 R=junov@chromium.org, senorblanco@chromium.org, reed@google.com Author: arbesfeld@chromium.org Review URL: https://chromiumcodereview.appspot.com/19729007 git-svn-id: http://skia.googlecode.com/svn/trunk@10274 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/image/SkImage_Raster.cpp
|
1d241f13d5474770ff517bd0c7999d9014e12ced |
|
22-Jul-2013 |
fmalita@google.com <fmalita@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> |
Revert "Add SkImage->draw() call with src and dst rects." This reverts commit 9c730e27e0bb550dc228ad496cadeed8787b2244. git-svn-id: http://skia.googlecode.com/svn/trunk@10238 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/image/SkImage_Raster.cpp
|
1d91600a0272025364e2d3447b4c085f470a773b |
|
22-Jul-2013 |
commit-bot@chromium.org <commit-bot@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81> |
Add SkImage->draw() call with src and dst rects. R=junov@chromium.org, senorblanco@chromium.org, reed@google.com Author: arbesfeld@chromium.org Review URL: https://chromiumcodereview.appspot.com/19729007 git-svn-id: http://skia.googlecode.com/svn/trunk@10237 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/image/SkImage_Raster.cpp
|
4b0757b7489ee5d6bd7149bbcdb0b2af6b7a21cb |
|
20-May-2013 |
reed@google.com <reed@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> |
add encodeData() to SkImageEncoder, and add encoding to SkImage BUG= R=scroggo@google.com Review URL: https://codereview.chromium.org/15002004 git-svn-id: http://skia.googlecode.com/svn/trunk@9193 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/image/SkImage_Raster.cpp
|
af0583528c7dd5344abfe14347377011abe374e2 |
|
03-Apr-2013 |
junov@chromium.org <junov@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81> |
Fixed bug with SkImage leaving canvas backing store in an immutable state after destroy. Added unit test that verifies that surface backing is writable after creating and destroying an image. Review URL: https://codereview.chromium.org/13226002 git-svn-id: http://skia.googlecode.com/svn/trunk@8512 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/image/SkImage_Raster.cpp
|
e16efc1882ab34a0bb3ae361a2d37f840044cf87 |
|
26-Jan-2013 |
skia.committer@gmail.com <skia.committer@gmail.com@2bbb7eff-a529-9590-31e7-b0007b416f81> |
Sanitizing source files in Skia_Periodic_House_Keeping git-svn-id: http://skia.googlecode.com/svn/trunk@7406 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/image/SkImage_Raster.cpp
|
b947625800a26194fcf63d7b57dadb1a63677f6a |
|
15-Nov-2012 |
mike@reedtribe.org <mike@reedtribe.org@2bbb7eff-a529-9590-31e7-b0007b416f81> |
remove unused (and undefined) SkColorSpace parameter. git-svn-id: http://skia.googlecode.com/svn/trunk@6427 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/image/SkImage_Raster.cpp
|
d9e0181405c9853ffd20502555200205a5ab09b1 |
|
29-Aug-2012 |
bsalomon@google.com <bsalomon@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> |
Linux warning fixes Review URL: https://codereview.appspot.com/6500045/ git-svn-id: http://skia.googlecode.com/svn/trunk@5337 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/image/SkImage_Raster.cpp
|
97af1a64ae6bdddd346d8babfd9f188279dd6644 |
|
28-Aug-2012 |
reed@google.com <reed@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> |
Add caching of the snapshot image form a surface Notify the surface when the canvas draws into it, so it can invalidate the cached image, and (if needed) perform a copy-on-write on the surface if it was being shared with the image. Review URL: https://codereview.appspot.com/6441115 git-svn-id: http://skia.googlecode.com/svn/trunk@5306 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/image/SkImage_Raster.cpp
|
fbfcd5602128ec010c82cb733c9cdc0a3254f9f3 |
|
23-Aug-2012 |
rmistry@google.com <rmistry@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> |
Result of running tools/sanitize_source_files.py (which was added in https://codereview.appspot.com/6465078/) This CL is part I of IV (I broke down the 1280 files into 4 CLs). Review URL: https://codereview.appspot.com/6485054 git-svn-id: http://skia.googlecode.com/svn/trunk@5262 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/image/SkImage_Raster.cpp
|
58b21ec7f06cfa8b7d7403a6108c0216345b3510 |
|
30-Jul-2012 |
reed@google.com <reed@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> |
expand internal subclasses into separate files git-svn-id: http://skia.googlecode.com/svn/trunk@4836 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/image/SkImage_Raster.cpp
|