aboutsummaryrefslogtreecommitdiffhomepage
path: root/src
Commit message (Collapse)AuthorAge
* Add GrResourceCache2.Gravatar bsalomon2014-08-19
| | | | | | | | | | Currently it just replaces GrGpu as the owner of the linked list of resources. R=robertphillips@google.com, mtklein@google.com Author: bsalomon@google.com Review URL: https://codereview.chromium.org/481443002
* remove unused SkIntToFloatCast_NoOverflowCheckGravatar caryclark2014-08-19
| | | | | | | | | R=reed@google.com, reed BUG=skia:2849 Author: caryclark@google.com Review URL: https://codereview.chromium.org/483273003
* Record concat as setMatrix.Gravatar mtklein2014-08-19
| | | | | | | | | | | | | Mainly this cuts out a type to have to think about in SkRecord world. It also means all the CTMs are recorded directly in the SkRecord, so we can track the current CTM by pointer rather than by copying. BUG=skia: R=reed@google.com, mtklein@google.com Author: mtklein@chromium.org Review URL: https://codereview.chromium.org/484673003
* Quickfix for incorrect PICTURE_MATRIX_PAINT & PATCH optimization.Gravatar fmalita2014-08-19
| | | | | | | | | | DRAW_PICTURE_MATRIX_PAINT & DRAW_PATCH are drawing ops. R=reed@google.com, robertphillips@google.com, mtklein@google.com Author: fmalita@chromium.org Review URL: https://codereview.chromium.org/480463004
* Move SkReadBuffer.h and SkReader32.h out of include.Gravatar halcanary2014-08-19
| | | | | | | | | | Committed: https://skia.googlesource.com/skia/+/2a51d7c74cec217195f861677de8998b382b39e4 R=mtklein@google.com, reed@google.com Author: halcanary@google.com Review URL: https://codereview.chromium.org/481053002
* Revert "Move the code over using the same template type approach previously ↵Gravatar Mike Klein2014-08-18
| | | | | | | | | | used for willPlayBackBitmaps in http://skbug.com/2702." This reverts commit 60c2a79cfa8ceebcbafc243407564dc71f5e3b4f. BUG=skia: Review URL: https://codereview.chromium.org/481173003
* Move the code over using the same template type approach previously used for ↵Gravatar tomhudson2014-08-18
| | | | | | | | | | | | | willPlayBackBitmaps in http://skbug.com/2702. Also unifies that flag and this one into a struct so they and others can be computed together. The struct is stored const to enforce lifetime expectations. Adds a few new cases to the unit test. BUG=skia:2700 R=mtklein@google.com, reed@google.com, robertphillips@google.com, tomhudson@google.com Author: tomhudson@chromium.org Review URL: https://codereview.chromium.org/364823009
* Revert "Move SkReadBuffer.h and SkReader32.h out of include."Gravatar halcanary2014-08-18
| | | | | | | | | | | | | | This reverts commit 2a51d7c74cec217195f861677de8998b382b39e4. Breaking Blink NOTRY=true R=bungeman@google.com TBR=bungeman@google.com Author: halcanary@google.com Review URL: https://codereview.chromium.org/472303006
* Counterproposal for skirting the BBH when the query fully contains the picture.Gravatar mtklein2014-08-18
| | | | | | | | | BUG=skia: R=reed@google.com, robertphillips@google.com, mtklein@google.com Author: mtklein@chromium.org Review URL: https://codereview.chromium.org/485703002
* Move SkReadBuffer.h and SkReader32.h out of include.Gravatar halcanary2014-08-18
| | | | | | | | R=mtklein@google.com, reed@google.com Author: halcanary@google.com Review URL: https://codereview.chromium.org/481053002
* Expose API for whether an SkPicture contains textGravatar ajuma2014-08-18
| | | | | | | | | BUG=chromium:399728 R=reed@google.com, nduca@chromium.org Author: ajuma@chromium.org Review URL: https://codereview.chromium.org/478673002
* Fix srcAlpaWillBeOne() for coverage drawingGravatar bsalomon2014-08-18
| | | | | | | | R=egdaniel@google.com Author: bsalomon@google.com Review URL: https://codereview.chromium.org/480113002
* Start actually bounding some draw ops.Gravatar mtklein2014-08-18
| | | | | | | | | | | This covers most of the common draws. BUG=skia: R=robertphillips@google.com, mtklein@google.com Author: mtklein@chromium.org Review URL: https://codereview.chromium.org/469213007
* Fix copyright on SkTLS.h.Gravatar bungeman2014-08-18
| | | | | | | | R=reed@google.com Author: bungeman@google.com Review URL: https://codereview.chromium.org/481783002
* Rename GPUAccelData to GrAccelDataGravatar robertphillips2014-08-18
| | | | | | | | | | | This is calved off of (Add new API to allow layer hoisting/atlasing across picture piles - https://codereview.chromium.org/474623002/) R=bsalomon@google.com Author: robertphillips@google.com Review URL: https://codereview.chromium.org/482773002
* Refactor GrLayerCache for new APIGravatar robertphillips2014-08-18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The only substantive change in this CL is skipping atlasing for any layers that are involved in nesting. Prior versions have allowed nesting layers to be atlased. -------------------------------------------------------- All times are on Windows with a repeat count of 200. Format is: <time in ms> (<# of glBindframebuffer calls>) How to interpret this: For the boogie page: both columns should be the same (since boogie has no nested layers) without the new API the tiled case doesn't show any real benefit from hoisting For the carsvg page: The nesting change does increase the number of FBO switches but doesn't kill performance Because of the location & size of the layers the tile case does show some improvement (even without the new API) boogie ------- w/o nested change w/ nested change simple 5.62 (811) N/A tiled 7.72 (811) N/A simple w/ hoisting 5.23 (409) 5.77 (409) (but no caching) tiled w/ hoisting 7.57 (809) 7.49 (809) (but no caching) carsvg ------ w/o nested change w/ nested change simple 60.37 (141990) N/A tiled 115.13 (256929) N/A simple w/ hoisting 41.57 (64570) 42.82 (72279) (but no caching) tiled w/ hoisting 84.24 (154352) 84.71 (165630) (but no caching) R=bsalomon@google.com Author: robertphillips@google.com Review URL: https://codereview.chromium.org/476833004
* Small tweaks and a bug fix.Gravatar mtklein2014-08-18
| | | | | | | | | | | | | | | | | Bug fixed: I was calling paint->canComputeFastBounds() where I should have been calling fSaveStack[i].paint->canComputeFastBounds(). This suggested I factor out the paint adjusting code, now called AdjustForPaint(). This removes the getImageFilter() check for SaveLayers, which seems like an optimization we can add back later if it proves important. We're going to want to intersect the bounds with the current clip bounds eventually, so might as well land that here too. Plus, more const. BUG=skia: R=robertphillips@google.com, mtklein@google.com Author: mtklein@chromium.org Review URL: https://codereview.chromium.org/481543002
* SkImage::NewFromGenerator(SkImageGenerator*), and a unit test.Gravatar halcanary2014-08-18
| | | | | | | | R=reed@google.com Author: halcanary@google.com Review URL: https://codereview.chromium.org/465823003
* Stop sorting the results of SkBBH::search().Gravatar mtklein2014-08-18
| | | | | | | | | | | | | We only seem to need to sort with SkQuadTree, which is not actively used by client code. So it's really just wasted work. SkQuadTree is interesting, but I'd rather disable it than slow down the production code path. BUG=skia:2834 R=robertphillips@google.com, mtklein@google.com Author: mtklein@chromium.org Review URL: https://codereview.chromium.org/471063004
* eliminate code related to SkBitmap::ConfigGravatar reed2014-08-18
| | | | | | | | | BUG=skia: R=reed@google.com, mike@reedtribe.org Author: reed@chromium.org Review URL: https://codereview.chromium.org/483593002
* Add alias mapping for Noto Sans CJK for ja/hansGravatar jshin2014-08-18
| | | | | | | | | | | | | | | | | Chrome/Chromium OS are getting a brand new CJK fonts (Noto Sans CJK). We want them to be used in place of common Japanese and Simplified Chinese sans-serif fonts. BUG=chromium:399080 TEST=With CrOS CL (https://chromium-review.googlesource.com/#/c/212624/), web pages using 'Simhei' and 'MSP Gothic' are rendered with Noto Sans CJK. R=reed@chromium.org, bungeman@google.com Author: jshin@chromium.org Review URL: https://codereview.chromium.org/476203003
* Correct order for alignment in name iterator record.Gravatar bungeman2014-08-18
| | | | | | | | | | | | | | | We must be quite careful about the order of fields in a 'pragma pack(1)'. On Android the order 'uint16_t, void*' will misalign the pointer, potentially leading to a crash. In the future we should move anything which is not strictly part of the table definition outside of the packed region. R=mtklein@google.com Author: bungeman@google.com Review URL: https://codereview.chromium.org/474313002
* simplify SkCanvas::init()Gravatar reed2014-08-18
| | | | | | | | | BUG=skia: R=robertphillips@google.com Author: reed@google.com Review URL: https://codereview.chromium.org/472123002
* Fix typoGravatar krajcevski2014-08-18
| | | | | | | | R=robertphillips@google.com Author: krajcevski@google.com Review URL: https://codereview.chromium.org/478533002
* Remove fOrigFontID from SkScalerContextRec.Gravatar bungeman2014-08-18
| | | | | | | | | | With the recent Android changes and removal of chaining, this is no longer used. R=djsollen@google.com, reed@google.com Author: bungeman@google.com Review URL: https://codereview.chromium.org/476713003
* Remove unused SkGlyphCache::VisitAllCaches.Gravatar bungeman2014-08-18
| | | | | | | | R=reed@google.com Author: bungeman@google.com Review URL: https://codereview.chromium.org/472153002
* Added bench for grid of patches.Gravatar dandov2014-08-15
| | | | | | | | | | | | It is on top of my previous cl to fix the mem leaks of the regular patch bench. NOTREECHECKS=true BUG=skia: R=egdaniel@google.com, bsalomon@google.com Author: dandov@google.com Review URL: https://codereview.chromium.org/470543004
* SkRecordDraw: incorporate clip into BBHGravatar mtklein2014-08-15
| | | | | | | | | | | NOTREECHECKS=true BUG=skia: R=robertphillips@google.com, senorblanco@chromium.org, mtklein@google.com Author: mtklein@chromium.org Review URL: https://codereview.chromium.org/474983002
* Check all scratch texture allocations for image filters.Gravatar senorblanco2014-08-15
| | | | | | | | | | BUG=403677 R=bsalomon@google.com NOTREECHECKS=true Author: senorblanco@chromium.org Review URL: https://codereview.chromium.org/473283002
* Fixed memory leak for nanobench and crashing in SampleAppGravatar dandov2014-08-15
| | | | | | | | | | | NOTREECHECKS=True BUG=skia:2830 R=mtklein@google.com, egdaniel@google.com Author: dandov@google.com Review URL: https://codereview.chromium.org/469333002
* int SkPicture::approximateOpCount()Gravatar mtklein2014-08-14
| | | | | | | | | | | NOTREECHECKS=true BUG=skia: R=reed@google.com, mtklein@google.com Author: mtklein@chromium.org Review URL: https://codereview.chromium.org/470333002
* Make null gpu context threadsafe(r) and make sure buffer objects are destroyed.Gravatar bsalomon2014-08-14
| | | | | | | | | NOTREECHECKS=true R=robertphillips@google.com Author: bsalomon@google.com Review URL: https://codereview.chromium.org/470993002
* Remove SkPaintOptionsAndroidGravatar djsollen2014-08-14
| | | | | | | | | | Committed: https://skia.googlesource.com/skia/+/f32331ffdb5de0440bb337aa7cbdd6f33e9ff23b R=reed@google.com, mtklein@google.com, tomhudson@google.com Author: djsollen@google.com Review URL: https://codereview.chromium.org/447873003
* Revert of Remove SkPaintOptionsAndroid (patchset #5 of ↵Gravatar bsalomon2014-08-13
| | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/447873003/) Reason for revert: Breaks the Chromium build: http://108.170.220.120:10117/builders/Canary-Chrome-Ubuntu13.10-Ninja-x86_64-DRT/builds/2469/steps/BuildContentShell_1/logs/stdio Original issue's description: > Remove SkPaintOptionsAndroid > > Committed: https://skia.googlesource.com/skia/+/f32331ffdb5de0440bb337aa7cbdd6f33e9ff23b R=reed@google.com, mtklein@google.com, tomhudson@google.com, djsollen@google.com TBR=djsollen@google.com, mtklein@google.com, reed@google.com, tomhudson@google.com NOTREECHECKS=true NOTRY=true Author: bsalomon@google.com Review URL: https://codereview.chromium.org/473543004
* Start tracking the CTM while filling the BBH in SkRecordDraw.Gravatar mtklein2014-08-13
| | | | | | | | | | | Depends on https://codereview.chromium.org/475473002/ BUG=skia: R=robertphillips@google.com, reed@google.com, mtklein@google.com Author: mtklein@chromium.org Review URL: https://codereview.chromium.org/468193003
* Remove SkPaintOptionsAndroidGravatar djsollen2014-08-13
| | | | | | | | R=reed@google.com, mtklein@google.com, tomhudson@google.com Author: djsollen@google.com Review URL: https://codereview.chromium.org/447873003
* Start filling BBH in SkRecordDraw.Gravatar mtklein2014-08-13
| | | | | | | | | | | | | | | | | | | | This starts with a skeleton of how to calculate bounds for ops which don't have their own bounds. For any given Save/Restore block, we'll find the union of the bounds of all the draws inside it (including other Save/Restore blocks), then say those are the bounds of all non-draws in the block, including the Save and Restore. To implement this, we keep a stack of active Save blocks. Any time we hit a non-drawing op ("control"), we'll add it to that Save block (implemented with a separate stack of indices and a count of control ops in the entry on the Save stack). Save and SaveLayer push onto the stack, and Restore pops the stack, at which point we can fill in the bounds for all the control ops in the block. BUG=skia: R=robertphillips@google.com, mtklein@google.com Author: mtklein@chromium.org Review URL: https://codereview.chromium.org/475473002
* Update path to Android font config fileGravatar tomhudson2014-08-13
| | | | | | | | | | | | | | | | | | | | | In version LMP, Android is expected to change the path to their font configuration file (and condense several files into one), as well as changing the format (qv http://crrev.com/446473003/). This patch tries the new path before falling back to the old path. The new fonts are incompatible with assumptions made by the old SkFontConfigInterfaceAndroid, so this patch MUST NOT land until https://crrev.com/462073002/ or the equivalent has turned on the new SkFontManager. BUG=chromium:400801 R=djsollen@google.com, bungeman@google.com, tomhudson@google.com Committed: https://skia.googlesource.com/skia/+/94fa4b99e2a53e997a90c7808cc5263f1bf40c9f Author: tomhudson@chromium.org Review URL: https://codereview.chromium.org/458543002
* Add support for new drawPicture entry point to debuggerGravatar robertphillips2014-08-13
| | | | | | | | R=fmalita@google.com, fmalita@chromium.org Author: robertphillips@google.com Review URL: https://codereview.chromium.org/464063003
* Delete the old font management implementation based on ↵Gravatar tomhudson2014-08-13
| | | | | | | | | | | SkFontConfigInterface, now that SkFontMgr_Android is live. BUG=chromium:400801 R=djsollen@google.com, bungeman@google.com Author: tomhudson@chromium.org Review URL: https://codereview.chromium.org/445143002
* Add override to force bitmap renderingGravatar krajcevski2014-08-13
| | | | | | | | R=robertphillips@google.com Author: krajcevski@google.com Review URL: https://codereview.chromium.org/462203002
* Mark fallback fonts in new Android font config parserGravatar tomhudson2014-08-13
| | | | | | | | | | | | | | | | | The new Android fonts.xml format doesn't explicitly tag fallback fonts, or separate them from system fonts by putting them in a separate file. We're assuming that any nameless font is a fallback font. Adds a unit test to track that we see as many fallback fonts as expected when parsing a file. BUG=chromium:400801 R=bungeman@google.com, djsollen@google.com, tomhudson@google.com Author: tomhudson@chromium.org Review URL: https://codereview.chromium.org/468893002
* Use combined color/coverage attribute when possible in aa rect renderer.Gravatar bsalomon2014-08-13
| | | | | | | | | | Also restore the is_irect test to detect AA rects that are integer aligned. R=robertphillips@google.com Author: bsalomon@google.com Review URL: https://codereview.chromium.org/466873002
* Add a parameter GR_ALWAYS_ALLOCATE_ON_HEAP to allow for only ever creating ↵Gravatar george2014-08-12
| | | | | | | | | | | | | temporary buffers on the heap instead of allowing stack allocation Unfortunately this can't be a GR_GL_* config like we'd discussed because there are SkAutoSMalloc buffers that are built outside of gl/ R=bsalomon@google.com, bsalomon BUG=skia: Author: george@mozilla.com Review URL: https://codereview.chromium.org/459263003
* Proposal for the mesh gradient interface. Implemented as a grid ofGravatar dandov2014-08-12
| | | | | | | | | | | | | | | | patches and uses 4 private arrays to store the values of the control points and colors. When it needs a patch at a certain position of the grid it just builds it using the corresponding values of the array and the grid coordinates provided. Details on implementation are documented in the corresponding classes' comments. Also added a gm for mesh gradients. BUG=skia: R=egdaniel@google.com, reed@google.com Author: dandov@google.com Review URL: https://codereview.chromium.org/451723003
* Turn off pedantic blit rectGravatar krajcevski2014-08-12
| | | | | | | | R=robertphillips@google.com Author: krajcevski@google.com Review URL: https://codereview.chromium.org/460253003
* Add missing DRAW_PICTURE_MATRIX_PAINT paint offset.Gravatar fmalita2014-08-12
| | | | | | | | | | | Also remove explicit gPaintOffsets sizing to make the existing compile time size check effective. R=reed@google.com, robertphillips@google.com Author: fmalita@chromium.org Review URL: https://codereview.chromium.org/467683003
* Update path to Android font config fileGravatar tomhudson2014-08-12
| | | | | | | | | | | | | | | | | | | In version LMP, Android is expected to change the path to their font configuration file (and condense several files into one), as well as changing the format (qv http://crrev.com/446473003/). This patch tries the new path before falling back to the old path. The new fonts are incompatible with assumptions made by the old SkFontConfigInterfaceAndroid, so this patch MUST NOT land until https://crrev.com/462073002/ or the equivalent has turned on the new SkFontManager. BUG=chromium:400801 R=djsollen@google.com, bungeman@google.com, tomhudson@google.com Author: tomhudson@chromium.org Review URL: https://codereview.chromium.org/458543002
* Replace LLU with ULL to make VS2012 work again.Gravatar halcanary2014-08-12
| | | | | | | | R=bungeman@google.com Author: halcanary@google.com Review URL: https://codereview.chromium.org/458663004
* Revert of SkImage::NewFromGenerator(SkImageGenerator*), and a unit test. ↵Gravatar halcanary2014-08-12
| | | | | | | | | | | | | | | | | | | | | (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
generated by cgit v1.2.3 (git 2.39.1) at 2024-07-03 22:38:29 +0000