aboutsummaryrefslogtreecommitdiffhomepage
Commit message (Collapse)AuthorAge
...
* Reland of Fix a few bugs in the google3 sync scripts. (patchset #1 id:1 of ↵Gravatar benjaminwagner2015-08-18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/1295213002/ ) Reason for revert: Retry. Original issue's description: > Revert of Fix a few bugs in the google3 sync scripts. (patchset #1 id:1 of https://codereview.chromium.org/1291343006/ ) > > Reason for revert: > MacMini buildbots were unable to apply the patch. > fatal: Could not parse object '18af0a0080cc2b2b3464292d35a0886cdade551f'. > http://build.chromium.org/p/client.skia/builders/Test-Mac10.9-Clang-MacMini6.2-CPU-AVX-x86_64-Release/builds/1580 > http://build.chromium.org/p/client.skia/builders/Test-Mac10.9-Clang-MacMini6.2-CPU-AVX-x86_64-Debug/builds/1474 > http://build.chromium.org/p/client.skia/builders/Test-Mac10.9-Clang-MacMini6.2-GPU-HD4000-x86_64-Release/builds/1566 > http://build.chromium.org/p/client.skia/builders/Perf-Mac10.9-Clang-MacMini6.2-GPU-HD4000-x86_64-Release/builds/1480 > http://build.chromium.org/p/client.skia/builders/Perf-Mac10.9-Clang-MacMini6.2-CPU-AVX-x86_64-Release/builds/1456 > http://build.chromium.org/p/client.skia/builders/Test-Mac10.9-Clang-MacMini6.2-GPU-HD4000-x86_64-Debug/builds/1364 > > > Original issue's description: > > Fix a few bugs in the google3 sync scripts. > > > > BUG=skia: > > > > Committed: https://skia.googlesource.com/skia/+/18af0a0080cc2b2b3464292d35a0886cdade551f > > TBR=mtklein@google.com > NOPRESUBMIT=true > NOTREECHECKS=true > NOTRY=true > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/e349d6b92574c1b0acdb4ba1ed8ff905ea830131 TBR=mtklein@google.com NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/1284093003
* Privatize GrBatch subclass overridesGravatar bsalomon2015-08-18
| | | | Review URL: https://codereview.chromium.org/1301663002
* Remove a couple zombie pathops files.Gravatar mtklein2015-08-18
| | | | | | | | These files don't compile and are not referenced by any GYP file. BUG=skia: Review URL: https://codereview.chromium.org/1299773002
* Remove SkOpts_sse2.cpp.Gravatar mtklein2015-08-18
| | | | | | | | | | | | | | | | | | It's sort of pointless: all our clients that will have SSE2 at runtime have it unconditionally at compile time, so the functions in namespace portable will pick up the SSE2 code. The procs in SkOpts_sse2.o were just duplicate code. A couple of the procs we had in _sse2.cpp can benefit slightly when compiled with SSSE3. I've moved those to _ssse3.cpp. This should lead to small speedups on platforms like Linux and Windows that have a baseline of SSE2. Similarly, I've removed the call to Init_neon() when NEON is available globally... it's a no-op. Renaming namespace portable to something clearer is TBD. BUG=skia:4117 Review URL: https://codereview.chromium.org/1294213002
* Revert[8] "move some public headers into private"Gravatar reed2015-08-17
| | | | | | | | | This reverts commit fb28cd2b13d76c324570341e1155aaadc1b532b6. BUG=skia: TBR= Review URL: https://codereview.chromium.org/1298833002
* Move normal map creation methods to sk_toolsGravatar robertphillips2015-08-17
| | | | | | Split off of https://codereview.chromium.org/1261433009/ (Add SkCanvas::drawLitAtlas call) Review URL: https://codereview.chromium.org/1298763002
* Add a simple script to simulate BUILD file glob() expansion.Gravatar mtklein2015-08-17
| | | | | | | | | | This lets us test changes to BUILD.public. This is not yet automated in any way. My hope is to trigger it quietly via the presubmit for any CL that adds or removes a file, or changes BUILD.public. BUG=skia: Review URL: https://codereview.chromium.org/1290833003
* Revert of Fix a few bugs in the google3 sync scripts. (patchset #1 id:1 of ↵Gravatar benjaminwagner2015-08-17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/1291343006/ ) Reason for revert: MacMini buildbots were unable to apply the patch. fatal: Could not parse object '18af0a0080cc2b2b3464292d35a0886cdade551f'. http://build.chromium.org/p/client.skia/builders/Test-Mac10.9-Clang-MacMini6.2-CPU-AVX-x86_64-Release/builds/1580 http://build.chromium.org/p/client.skia/builders/Test-Mac10.9-Clang-MacMini6.2-CPU-AVX-x86_64-Debug/builds/1474 http://build.chromium.org/p/client.skia/builders/Test-Mac10.9-Clang-MacMini6.2-GPU-HD4000-x86_64-Release/builds/1566 http://build.chromium.org/p/client.skia/builders/Perf-Mac10.9-Clang-MacMini6.2-GPU-HD4000-x86_64-Release/builds/1480 http://build.chromium.org/p/client.skia/builders/Perf-Mac10.9-Clang-MacMini6.2-CPU-AVX-x86_64-Release/builds/1456 http://build.chromium.org/p/client.skia/builders/Test-Mac10.9-Clang-MacMini6.2-GPU-HD4000-x86_64-Debug/builds/1364 Original issue's description: > Fix a few bugs in the google3 sync scripts. > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/18af0a0080cc2b2b3464292d35a0886cdade551f TBR=mtklein@google.com NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/1295213002
* Fix a few bugs in the google3 sync scripts.Gravatar benjaminwagner2015-08-17
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1291343006
* Add files added to Skia HEAD in google3 to the Git repo.Gravatar benjaminwagner2015-08-17
| | | | | | | | LICENSE is modified in google3 to mention some code copied from other projects. BUG=skia: Review URL: https://codereview.chromium.org/1291603003
* Make GrVertexBatch objects hold their own draws during GrDrawTarget flushGravatar bsalomon2015-08-17
| | | | | | NO_MERGE_BUILDS Review URL: https://codereview.chromium.org/1286043004
* Stop ping ponging AA in SkGpuDeviceGravatar joshualitt2015-08-17
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1299703003
* Revert of Depend on ETC1 via DEPS instead of a direct third_party checkin. ↵Gravatar mtklein2015-08-17
| | | | | | | | | | | | | | | | | | | | | | | | | | (patchset #1 id:1 of https://codereview.chromium.org/1296253003/ ) Reason for revert: This will make the google3 roll awkward. Reverting. Original issue's description: > Depend on ETC1 via DEPS instead of a direct third_party checkin. > > Yes, this does mean we'll check out ~40M of code for 2 files... > > I think these trybots are moot... they gclient sync before patching. > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/ca01d179347b5e848ded97c8228b79ffc6b5e7bf TBR=robertphillips@google.com,djsollen@google.com,mtklein@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/1290913008
* drawBitmapImage can batch across AA rectsGravatar joshualitt2015-08-17
| | | | | | | | BUG=464835 Committed: https://skia.googlesource.com/skia/+/0d4bcea0d570041434ac3de2df2bd9063138fdb5 Review URL: https://codereview.chromium.org/1293543002
* fix typo in android_framework_lib.gypGravatar Derek Sollenberger2015-08-17
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1293183002 .
* Add GPU to the android include path.Gravatar djsollen2015-08-17
| | | | Review URL: https://codereview.chromium.org/1301623002
* Depend on ETC1 via DEPS instead of a direct third_party checkin.Gravatar mtklein2015-08-17
| | | | | | | | | | Yes, this does mean we'll check out ~40M of code for 2 files... I think these trybots are moot... they gclient sync before patching. BUG=skia: Review URL: https://codereview.chromium.org/1296253003
* remove unused codeGravatar caryclark2015-08-17
| | | | | | R=mtklein@google.com Review URL: https://codereview.chromium.org/1297013002
* SkPaintFilterCanvas should inherit the target canvas stateGravatar fmalita2015-08-17
| | | | | | | | | | | | | | Currently, SkPaintFilterCanvas does not provide any help in cloning target canvas state. While that could be handled in subclasses, it is easy to miss (see linked bug). This CL adds a new constructor variant which ensures that the initial matrix and clip bounds are inherited from the target canvas. BUG=516790 R=reed@google.com,robertphillips@google.com Review URL: https://codereview.chromium.org/1294013002
* Fix webp compile warnings on windowsGravatar msarett2015-08-17
| | | | | | | | | | | | | | This fix was landed with: https://codereview.chromium.org/1280073002/ The above CL was reverted due to an unrelated bug in libwebp. The above CL contains multiple components, and I think that reverting this part of the change was unintentional. BUG=skia: Review URL: https://codereview.chromium.org/1286903003
* Revert of drawBitmapImage can batch across AA rects (patchset #5 id:80001 of ↵Gravatar joshualitt2015-08-17
| | | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/1293543002/ ) Reason for revert: a follow up patch exposes a bug Original issue's description: > drawBitmapImage can batch across AA rects > > BUG=464835 > > Committed: https://skia.googlesource.com/skia/+/0d4bcea0d570041434ac3de2df2bd9063138fdb5 TBR=bsalomon@google.com,robertphillips@google.com,joshualitt@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=464835 Review URL: https://codereview.chromium.org/1291663005
* drawBitmapImage can batch across AA rectsGravatar joshualitt2015-08-17
| | | | | | BUG=464835 Review URL: https://codereview.chromium.org/1293543002
* Update SKP versionGravatar rmistry2015-08-16
| | | | | | | | | Automatic commit by the RecreateSKPs bot. TBR= NO_MERGE_BUILDS Review URL: https://codereview.chromium.org/1296513004
* support both box and circle lcdGravatar reed2015-08-15
| | | | | | | BUG=skia: TBR= Review URL: https://codereview.chromium.org/1292943003
* show LCD on max fat-zoomGravatar reed2015-08-15
| | | | | | | BUG=skia: TBR= Review URL: https://codereview.chromium.org/1295483003
* Update SKP versionGravatar rmistry2015-08-15
| | | | | | | | | Automatic commit by the RecreateSKPs bot. TBR= NO_MERGE_BUILDS Review URL: https://codereview.chromium.org/1291273003
* Revert[7] "move some public headers into private"Gravatar reed2015-08-15
| | | | | | | | | This reverts commit e02716908fe82c7c4ae9b415793277620a22bcd6. BUG=skia: TBR= Review URL: https://codereview.chromium.org/1300523002
* increase zoom in fatbitsGravatar reed2015-08-15
| | | | | | | BUG=skia: TBR= Review URL: https://codereview.chromium.org/1300513002
* Revert[6] "move some public headers into private"Gravatar reed2015-08-15
| | | | | | | | | This reverts commit cc1229c9d5e57b45305ba8f99ae33e92db0032bc. BUG=skia: TBR= Review URL: https://codereview.chromium.org/1295003002
* Revert of stop dropping AA when rect stays rect (patchset #8 id:140001 of ↵Gravatar joshualitt2015-08-14
| | | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/1295523002/ ) Reason for revert: breaks layout tests Original issue's description: > stop dropping AA when rect stays rect > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/1bef9f59c566cc54c2259cc4d0171c115157cd1c TBR=bsalomon@google.com,robertphillips@google.com,joshualitt@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/1295813005
* SkPDF: Move SkFlate into src/pdf (part 3/3)Gravatar halcanary2015-08-14
| | | | | | | | This must land after http://crrev.com/1288053002 Motivation: Skia only uses SkFlate in the SkPDF module. Review URL: https://codereview.chromium.org/1288063002
* A bit of misc cleanup.Gravatar mtklein2015-08-14
| | | | | | | | | | | - Update comments about skia_shared_lib - Remove dead code (system_preferences) - Remove non-building code (techtalk1) - Turn gm/annotated_text back on. BUG=skia: Review URL: https://codereview.chromium.org/1286133004
* fix valgrind issueGravatar joshualitt2015-08-14
| | | | | | | TBR=bsalomon@google.com BUG=skia: Review URL: https://codereview.chromium.org/1289203004
* fix viewmatrix mapping in AAFillRectGravatar joshualitt2015-08-14
| | | | | | | TBR=bsalomon@google.com BUG=skia: Review URL: https://codereview.chromium.org/1293823003
* Add include paths in gputest.gyp for use by additional gpu tests.Gravatar egdaniel2015-08-14
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1290553005
* Build all GMs on Android.Gravatar mtklein2015-08-14
| | | | | | | | | | Many of these were excluded from the build for bots that no longer exist. Will fix up anything that starts failing using dm_flags.py. BUG=skia:2326,skia:2313, skia:1978 Review URL: https://codereview.chromium.org/1290943004
* Support more swizzles to 565 in SkCodecGravatar scroggo2015-08-14
| | | | | | | | | | | | | | | | | | | | | | | Add more swizzling functions for swizzling to 565. Much of this code was revived from crrev.com/1055743003 (for BMP). Also added swizzling functions for WBMP. Consolidate the static function conversion_possible. In SkCodec::getPixels, check that the alphatype corresponds to the colorType. This prevents requesting 565 + non-opaque. In SkIcoCodec, report that the image is unpremul (instead of whatever the largest embedded codec thinks), but modify the requested info to have the alpha type expected/required by the embedded codec. Add tests for decoding to 565. BUG=skia:3257 BUG=skia:3683 Review URL: https://codereview.chromium.org/1277213002
* SkScaledCodec classGravatar emmaleer2015-08-14
| | | | | | | | | | | | | | | | | | | | | | | | This class does scaling by using a scanlineDecoder. getScanlines and skipScanlines are used for y sampling, the swizzler is used for x sampling this class is currently only working for png and jpeg images I will update other Codec types to work soon For SkJpegCodec to implement width wise swizzling it now uses a swizzler. I ran performance tests on this change. Here are the performance test results: https://docs.google.com/a/google.com/spreadsheets/d/1D7-Q_GXD_dI68LZO005NNvb8Wq2Ee0wEBEPG72671yw/edit?usp=sharing BUG=skia: Committed: https://skia.googlesource.com/skia/+/0944100ac89f797714eeae0cf2875e2335ff52ee Committed: https://skia.googlesource.com/skia/+/d518ea7927f9f4e0ed5b4134d1b4f48243855a47 Committed: https://skia.googlesource.com/skia/+/b157917507d4f7d2651f0aeb566d31603cc02240 Review URL: https://codereview.chromium.org/1260673002
* stop dropping AA when rect stays rectGravatar joshualitt2015-08-14
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1295523002
* Revert of SkScaledCodec class (patchset #35 id:680001 of ↵Gravatar egdaniel2015-08-14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/1260673002/ ) Reason for revert: breaking ubuntu bots Original issue's description: > SkScaledCodec class > > This class does scaling by using a scanlineDecoder. > getScanlines and skipScanlines are used for y sampling, > the swizzler is used for x sampling > > this class is currently only working for png and jpeg images > I will update other Codec types to work soon > > For SkJpegCodec to implement width wise swizzling it now > uses a swizzler. I ran performance tests on this change. > Here are the performance test results: > https://docs.google.com/a/google.com/spreadsheets/d/1D7-Q_GXD_dI68LZO005NNvb8Wq2Ee0wEBEPG72671yw/edit?usp=sharing > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/0944100ac89f797714eeae0cf2875e2335ff52ee > > Committed: https://skia.googlesource.com/skia/+/d518ea7927f9f4e0ed5b4134d1b4f48243855a47 > > Committed: https://skia.googlesource.com/skia/+/b157917507d4f7d2651f0aeb566d31603cc02240 TBR=scroggo@google.com,msarett@google.com,djsollen@google.com,mtklein@google.com,emmaleer@google.com NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/1285173003
* Add GM to test drawing atlas caseGravatar joshualitt2015-08-14
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1294513002
* Revert of Paranoid temporary check that Chrome's building with SSE2 on all ↵Gravatar mtklein2015-08-14
| | | | | | | | | | | | | | | | | | | | | | Windows builders. (patchset #1 id:1 of https://codereview.chromium.org/1283183004/ ) Reason for revert: Nothing broke. Experiment is over... SSE2 is present. Original issue's description: > Paranoid temporary check that Chrome's building with SSE2 on all Windows builders. > > BUG=chromium:475596 > > Committed: https://skia.googlesource.com/skia/+/e8445307ac33eddcf04457ae21040e47dc9c2a97 TBR=egdaniel@google.com,mtklein@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=chromium:475596 Review URL: https://codereview.chromium.org/1296593002
* SkScaledCodec classGravatar emmaleer2015-08-14
| | | | | | | | | | | | | | | | | | | | | | This class does scaling by using a scanlineDecoder. getScanlines and skipScanlines are used for y sampling, the swizzler is used for x sampling this class is currently only working for png and jpeg images I will update other Codec types to work soon For SkJpegCodec to implement width wise swizzling it now uses a swizzler. I ran performance tests on this change. Here are the performance test results: https://docs.google.com/a/google.com/spreadsheets/d/1D7-Q_GXD_dI68LZO005NNvb8Wq2Ee0wEBEPG72671yw/edit?usp=sharing BUG=skia: Committed: https://skia.googlesource.com/skia/+/0944100ac89f797714eeae0cf2875e2335ff52ee Committed: https://skia.googlesource.com/skia/+/d518ea7927f9f4e0ed5b4134d1b4f48243855a47 Review URL: https://codereview.chromium.org/1260673002
* Paranoid temporary check that Chrome's building with SSE2 on all Windows ↵Gravatar mtklein2015-08-13
| | | | | | | | builders. BUG=chromium:475596 Review URL: https://codereview.chromium.org/1283183004
* Split GrDrawBatch and GrVertexBatch into their own filesGravatar bsalomon2015-08-13
| | | | | | TBR=joshualitt@google.com Review URL: https://codereview.chromium.org/1296483002
* some catchup CL revisionsGravatar reed2015-08-13
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1295593002
* Every pixel ref gets its own mutex.Gravatar mtklein2015-08-13
| | | | | | | | Seems like a memory-saving flourish follow up would be to use SkSpinlock. BUG=Florin's email. Review URL: https://codereview.chromium.org/1289623004
* Revert of SkScaledCodec class (patchset #34 id:660001 of ↵Gravatar emmaleer2015-08-13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/1260673002/ ) Reason for revert: Seg-faulting: http://build.chromium.org/p/client.skia/builders/Test-Ubuntu-GCC-GCE-CPU-AVX2-x86-Release/builds/1608/steps/dm/logs/stdio Original issue's description: > SkScaledCodec class > > This class does scaling by using a scanlineDecoder. > getScanlines and skipScanlines are used for y sampling, > the swizzler is used for x sampling > > this class is currently only working for png and jpeg images > I will update other Codec types to work soon > > For SkJpegCodec to implement width wise swizzling it now > uses a swizzler. I ran performance tests on this change. > Here are the performance test results: > https://docs.google.com/a/google.com/spreadsheets/d/1D7-Q_GXD_dI68LZO005NNvb8Wq2Ee0wEBEPG72671yw/edit?usp=sharing > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/0944100ac89f797714eeae0cf2875e2335ff52ee > > Committed: https://skia.googlesource.com/skia/+/d518ea7927f9f4e0ed5b4134d1b4f48243855a47 TBR=scroggo@google.com,msarett@google.com,djsollen@google.com,mtklein@google.com NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/1294613002
* Belt and suspenders for colors in DrawAtlasBatch testGravatar jvanverth2015-08-13
| | | | Review URL: https://codereview.chromium.org/1290253002
* Introduce GrBatch subclasses GrDrawBatch and GrVertexBatch to prepare for ↵Gravatar bsalomon2015-08-13
| | | | | | non-drawing batches Review URL: https://codereview.chromium.org/1293583002