aboutsummaryrefslogtreecommitdiffhomepage
Commit message (Collapse)AuthorAge
* When getGLInstance is called on a frag proc, the resulting ↵Gravatar wangyix2015-08-18
| | | | | | | | GrGLFragmentProcessor will be the root of a tree of GrGLFragmentProcessors that mirrors the GrFragmentProcessor's tree. This allows setData() to be called recursively (removing the responsibility from compose shader) and allows gl instances direct access to their children gl instances so they can emit their code. BUG=skia:4182 Review URL: https://codereview.chromium.org/1287023009
* Add subsets to SkImageGenerator and SkImageCacheratorGravatar reed2015-08-18
| | | | | | | | ... to support subsets in SkImage! BUG=skia: Review URL: https://codereview.chromium.org/1301633002
* Update skia to test with libpng to 1.6.10Gravatar msarett2015-08-18
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1301523005
* Add internal links section to skia.orgGravatar hcm2015-08-18
| | | | | | | | BUG=skia:4142 NOTRY=true DOCS_PREVIEW= https://skia.org/?cl=1284043004 Review URL: https://codereview.chromium.org/1284043004
* ClearStencilClip in GrBatchGravatar bsalomon2015-08-18
| | | | Review URL: https://codereview.chromium.org/1288963004
* Move GrTBatchTesselator to its own fileGravatar joshualitt2015-08-18
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1298983002
* Update SkOpts namespaces.Gravatar mtklein2015-08-18
| | | | | | | | portable -> default, and everyone gets an sk_ prefix. BUG=skia:4117 Review URL: https://codereview.chromium.org/1299013003
* Add missing overrides to clear and discard batch dumpInfo()Gravatar Brian Salomon2015-08-18
| | | | | | TBR=joshualitt@google.com Review URL: https://codereview.chromium.org/1303533003
* Patches on top of Radu's latest.Gravatar mtklein2015-08-18
| | | | | | | | patch from issue 1273033005 at patchset 120001 (http://crrev.com/1273033005#ps120001) BUG=skia: Review URL: https://codereview.chromium.org/1288323004
* Use portable code for family names with DirectWrite.Gravatar bungeman2015-08-18
| | | | | | | | | | | | IDWriteFamily::GetFamilyNames appears to be 'helpful' and removes parts of family names that look like style names. Since the iterator is supposed to return the actual names and not just the name the platform thinks the name is (as getFamilyName does), try returning the raw names when possible. BUG=skia:4217 Review URL: https://codereview.chromium.org/1302573002
* Put clear and discard into GrBatch.Gravatar bsalomon2015-08-18
| | | | Review URL: https://codereview.chromium.org/1293563003
* Update BUILD.public a bit.Gravatar mtklein2015-08-18
| | | | | | | | | | | | | - SSE2 files are unfortunately now mixed-case, _SSE2 or _sse2. - Adds lists for SSSE3 and SSE4 files. - Remove SkDocument_PDF_None.cpp - Remove a few more references to animator. - Exclude private headers from HDRS. - Formatting and notes. BUG=skia: Review URL: https://codereview.chromium.org/1298003007
* Clean up BUILD_simulator.pyGravatar halcanary2015-08-18
| | | | Review URL: https://codereview.chromium.org/1298813002
* Deduplicate typefaces across sub-picturesGravatar mtklein2015-08-18
| | | | | | | | | | | | | | | | | | | | | | | | | | | Old flow to serialize a picture: 1) serialize picture ops 2) serialize all sub pictures recursively 3) flatten the rest of this picture into a buffer, deduping flattenable factories and typefaces as we go 4) serialize the factories and typefaces 5) serialize the bytes from 3) This allows the data in step 5) to refer to the deduplicated factories and typefaces from step 4). But, each sub picture in step 2) is completely siloed, so they can't dedup with the parent picture or each other. New flow: 1) serialize picture ops 2) flatten the rest of this picture into a buffer, deduping flattenable factories and typefaces as we go 3) dummy-serialize sub pictures into /dev/null, with the effect of adding any new typefaces to our dedup set 4) serialize the factories and typefaces 5) serialize the bytes from 2) 6) serialize all sub pictures recursively, with perfect deduplication because of step 3). Now all typefaces in the top-level picture and all sub pictures recursively should end up deduplicated in the top-level typeface set. Decoding changes are similar: we just thread through the top-level typefaces to the sub pictures. What's convenient / surprising is that this new code correctly reads old pictures if we just have each picture prefer its local typeface set over the top-level one: old pictures always just use their own typefaces, and new pictures always use the top-level ones. BUG=skia:4092 Review URL: https://codereview.chromium.org/1233953004
* Prefer native scaling to samplingGravatar msarett2015-08-18
| | | | | | | | | | | | | | | | | | | | | In the cases we have come across so far, native scaling has better performance and correctness than sampling. If native scaling is supported we want to use it. Jpegs native scale rounds up. Ex: An 11x11 image with sampleSize=8 scales to 2x2. SkScaledCodec rounds down. Ex: An 11x11 image with sampleSize=8 scales to 1x1. Before the CL, we would choose to use SkScaledCodec because it scales closer to the "ideal" scale. I think we want to go with the native option as long as its within 1 of the ideal value. BUG=skia: Review URL: https://codereview.chromium.org/1284243004
* Regenerate LCD text blobs if GrPaint's color changes, not SkPaint.Gravatar jvanverth2015-08-18
| | | | | | | | | The GrPaint's color takes into account shaders and color filters, so is a more accurate picture of the color state. BUG=chromium:511787 Review URL: https://codereview.chromium.org/1297053004
* Made isEqual in GrFragmentProcessor recursiveGravatar wangyix2015-08-18
| | | | | | | | | | Added comment about how computeInvariantOutput() is non-recursive in GrFragmentProcessor Made isEqual() recursive in GrFragmentProcessor BUG=skia:4182 Review URL: https://codereview.chromium.org/1287343005
* GrAAFillRectBatch holds onto GrPipelineOptimizationsGravatar joshualitt2015-08-18
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1301603004
* Added class AutoFragmentChildProcAdvance to be constructed before a child ↵Gravatar wangyix2015-08-18
| | | | | | | | | | | | | | | | | | emitCode and destructed after Fixed wrong indent Changed auto child advance back to backwards linear search for getting subset of coords and samplers array of a child Used offset from parent instead of backwards linear search to find a child proc's coords and transforms in Auto...Advance append mangleString to variable name in nameVariable() BUILDS! Added AutoFragmentChildProcAdvance class; fixed a few errors from previous commits BUG=skia:4182 Review URL: https://codereview.chromium.org/1286293002
* fix pathops fuzz failuresGravatar caryclark2015-08-18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | If a curve has the identical start and control points, the initial or final tangent can't be trivally determined. The perpendicular to the tangent is used to measure coincidence. Add logic for cubics, quadratics, and conics, to use the secondary control points or the end points if the initial control point alone can't determine the tangent. Add debugging (currently untriggered by exhaustive testing) to detect zero-length tangents which are not at the curve endpoints. Increase the number of temporary intersecions gathered from 10 to 12 but reduce the max passed in by cubic intersection from 27 to 12. Also, add checks if the max passed exceeds the storage allocated. When cleaning up parallel lines, choose the intersection which is on the end of both segments over the intersection which is on the end of a single segment. TBR=reed@google.com BUG=425140,516266 Review URL: https://codereview.chromium.org/1288863004
* 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