aboutsummaryrefslogtreecommitdiffhomepage
Commit message (Collapse)AuthorAge
* Fix usage of SK_BUILD_* defines.Gravatar tfarina2014-10-06
| | | | | | | | | | | | | | | Since we just 'define' them, but not attribute anything to them, like '1' for example, cpp expands it to nothing and that breaks the "#if" clauses. To fix that, uses "#if defined(...)" which will correctly check if your macro name was defined or not. BUG=skia:2850 TEST=make most R=robertphillips@google.com Review URL: https://codereview.chromium.org/628763005
* a few more tweaks to the webtry setup docGravatar humper2014-10-06
| | | | | | | BUG=skia: TBR=tfarina Review URL: https://codereview.chromium.org/626623002
* add environment variable to webtry local run documentationGravatar Greg Humphreys2014-10-06
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/631693003
* Automatically resize the webtry text editor to fit the contentGravatar Greg Humphreys2014-10-06
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/626033004
* fix webtry local run documentationGravatar Greg Humphreys2014-10-06
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/630093002
* experimental c apiGravatar reed2014-10-06
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/624973004
* Add GrAASmallPathRenderer.Gravatar jvanverth2014-10-06
| | | | | | | | | Uses cached signed distance fields to render scaled and rotated versions of small paths. BUG=skia:2935 Review URL: https://codereview.chromium.org/589103004
* Update SKP version to 118Gravatar borenet2014-10-05
| | | | | | | | Automatic commit by the RecreateSKPs bot. TBR= Review URL: https://codereview.chromium.org/630733002
* Update SKP version to 117Gravatar borenet2014-10-04
| | | | | | | | Automatic commit by the RecreateSKPs bot. TBR= Review URL: https://codereview.chromium.org/632503002
* Update SKP version to 116Gravatar borenet2014-10-03
| | | | | | | | Automatic commit by the RecreateSKPs bot. TBR= Review URL: https://codereview.chromium.org/623423002
* Restart webtry to pick up changes.Gravatar tfarina2014-10-03
| | | | | | | | | | Also add webtry.db to gitignore so it does not polute `git status`. BUG=None TEST=None R=humper@google.com Review URL: https://codereview.chromium.org/626703002
* Run goimports through 'go' code in bug_chomper.Gravatar tfarina2014-10-03
| | | | | | | | | | | | | $ cd ~/golib $ go get code.google.com/p/go.tools/cmd/goimports $ cd ~/skia/tools/bug_chomper/ $ ~/golib/bin/goimports -w src/ BUG=None TEST=None R=borenet@google.com Review URL: https://codereview.chromium.org/624813002
* Remove globals from tests/CanvasTest.cppGravatar piotaixr2014-10-03
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/617093004
* Speculative revert to diagnose crash in chrome. Revert "Add SkCachedData and ↵Gravatar reed2014-10-03
| | | | | | | | | | | use it for SkMipMap" This reverts commit 92561a0b99ad6c08ab7a11dd1872f028199392e9. crasher in question: https://code.google.com/p/chromium/issues/detail?id=420178 Review URL: https://codereview.chromium.org/617613003
* remove unused TextBufferDirection enumGravatar reed2014-10-03
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/621403002
* Fix int16_t for glyphs, const glyphs, and clarify glyph loop.Gravatar bungeman2014-10-03
| | | | | | | | | | | | | | | | | | Several places in the PDF code are using int16_t for glyphs. With newer NotoSans fonts, all possible glyph ids are being used, so this can lead to problems. The PDF glyphs from text code returns the text for the glyphs if the encoding is for glyphs. However, it returns this using an unsafe const cast which is hiding possible bugs and preventing correct use of const in other places. The way the glyph loop in SkPDFDevice::drawPosText is written uses a '--i' in the loop, which makes it appear this can loop forever. I don't believe it can, but it is an unecessary code folding. We should also at least assert the forward progress correctness here. Review URL: https://codereview.chromium.org/626613002
* Reduce overdraw in MultiPictureDraw Sierpinski GMGravatar robertphillips2014-10-03
| | | | | | | | | | | | | As written, all the layers in the MPD Sierpinski GM are full screen. This CL bounds each layer to reduce overdraw. This CL will cause differences in: multipicturedraw_sierpinski_simple multipicturedraw_sierpinski_tiled but they are/will be suppressed in: (Don't turn on dither for saveLayers which have no paint - https://codereview.chromium.org/619363002/) Review URL: https://codereview.chromium.org/582633003
* Don't turn on dither for saveLayers which have no paintGravatar robertphillips2014-10-03
| | | | | | | | Only the gpu, 565 and msaa configs of GMs in ignored-tests.txt need rebaselining. NOTRY=true Review URL: https://codereview.chromium.org/619363002
* Add isSingleComponent bool to getConstantColorComponentGravatar egdaniel2014-10-03
| | | | | | | | | | Initial step to allowing effects to use/output 1 or 4 color/coverage components. This cl doesn't change any current logic and all effects still assume they are working with 4 components. BUG=skia: Committed: https://skia.googlesource.com/skia/+/3b8af078281a5a20f951b9fd84f38d92b8f6217b Review URL: https://codereview.chromium.org/608253002
* abort op early if path isn't parseableGravatar caryclark2014-10-03
| | | | | | | TBR= BUG=419649 Review URL: https://codereview.chromium.org/623943002
* Fix ref cnt'ing issue in GrProgramElement.Gravatar bsalomon2014-10-03
| | | | | | | | Drop ref on GrGpuResources when GrProgramElement loses its last ref and already has a pending execution. BUG=skia:2889 Review URL: https://codereview.chromium.org/612293003
* Update SKP version to 115Gravatar borenet2014-10-02
| | | | | | | | Automatic commit by the RecreateSKPs bot. TBR= Review URL: https://codereview.chromium.org/623023002
* make SkMatrix::dump always availableGravatar reed2014-10-02
| | | | | | | BUG=skia: TBR= Review URL: https://codereview.chromium.org/624783002
* Add SkCachedData and use it for SkMipMapGravatar reed2014-10-02
| | | | Review URL: https://codereview.chromium.org/592843003
* Handle getMemoryBase returning NULL in WriteTask.Gravatar bungeman2014-10-02
| | | | | | | | | | DM::WriteTask::fData was changed from a data to a stream in "SkData to SkStreamAsset to avoid unneeded copying" https://skia.googlesource.com/skia/+/a4c6094177ebde18c706cbcfbd2013d0a088e0ee However, DM::WriteTask::draw was not updated to handle this, resulting in segfaults when trying to write out PDF files. Review URL: https://codereview.chromium.org/627473002
* Save/Restore around bench runs.Gravatar mtklein2014-10-02
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/619353002
* Use +- 2B for unbounded draw ops.Gravatar mtklein2014-10-02
| | | | | | | | This lets us skip a check inside SkTileGrid::insert(). BUG=skia: Review URL: https://codereview.chromium.org/620093006
* check for newSurface failureGravatar reed2014-10-02
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/622063003
* specialize setConcat for scale+translateGravatar reed2014-10-02
| | | | | | | | This reverts commit 2a382f519999109b2e3445e73ddb4f0eae926b51. BUG=skia: Review URL: https://codereview.chromium.org/626583002
* Revert of Add isSingleComponent bool to getConstantColorComponent (patchset ↵Gravatar robertphillips2014-10-02
| | | | | | | | | | | | | | | | | | | | | | | #6 id:100001 of https://codereview.chromium.org/608253002/) Reason for revert: Changing some GMs Original issue's description: > Add isSingleComponent bool to getConstantColorComponent > > Initial step to allowing effects to use/output 1 or 4 color/coverage components. This cl doesn't change any current logic and all effects still assume they are working with 4 components. > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/3b8af078281a5a20f951b9fd84f38d92b8f6217b TBR=joshualitt@chromium.org,bsalomon@google.com,reed@google.com,egdaniel@google.com NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/617853003
* Revert of GrContext::copyTexture->GrContext::copySurface. Add a flush writes ↵Gravatar bsalomon2014-10-02
| | | | | | | | | | | | | | | | | | | | | | | | | pixel ops flag. (patchset #3 id:40001 of https://codereview.chromium.org/622663002/) Reason for revert: Breaking GMs on some bots Original issue's description: > GrContext::copyTexture->GrContext::copySurface. > > Add a flush writes pixel ops flag. > > Add an explicit flush writes for GrSurface. > > BUG=skia:2977 > > Committed: https://skia.googlesource.com/skia/+/cf99b00980b6c9c557e71abf1a7c9f9b21217262 TBR=robertphillips@google.com NOTREECHECKS=true NOTRY=true BUG=skia:2977 Review URL: https://codereview.chromium.org/621073002
* Caching the result of readPixelsSupportedGravatar piotaixr2014-10-02
| | | | | | | | | | | | | | | | | The call was calling GR_GL_GetIntegerv 2 times for each readPixels and thus was causing a loss of performance (resubmit of issue 344793008) Benchmark url: http://packages.gkny.fr/tst/index.html BUG=skia:2681 Committed: https://skia.googlesource.com/skia/+/753a2964afe5661ce9b2a8ca77ca9d0aabd3173c Committed: https://skia.googlesource.com/skia/+/8339371f1ec3c57a0741932fd96bff32c53d4e54 Review URL: https://codereview.chromium.org/364193004
* GrContext::copyTexture->GrContext::copySurface.Gravatar bsalomon2014-10-02
| | | | | | | | | | Add a flush writes pixel ops flag. Add an explicit flush writes for GrSurface. BUG=skia:2977 Review URL: https://codereview.chromium.org/622663002
* Update webtry setup docs to reflect new installation procedureGravatar humper2014-10-02
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/626443004
* fix sudo redirection in setup scriptGravatar humper2014-10-02
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/618013004
* Add isSingleComponent bool to getConstantColorComponentGravatar egdaniel2014-10-02
| | | | | | | | Initial step to allowing effects to use/output 1 or 4 color/coverage components. This cl doesn't change any current logic and all effects still assume they are working with 4 components. BUG=skia: Review URL: https://codereview.chromium.org/608253002
* Demote getCount, getDepth, and clear to RTree-only methods.Gravatar mtklein2014-10-02
| | | | | | | | We use them only to test RTree. BUG=skia: Review URL: https://codereview.chromium.org/622773003
* Archive more dead code.Gravatar mtklein2014-10-02
| | | | | | | | These classes were used by QuadTree, which has been archived (with prejudice). BUG=skia: Review URL: https://codereview.chromium.org/616963006
* TileGrid: earliest need not be a pointer anymoreGravatar mtklein2014-10-02
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/625573003
* Add a MultiPictureDraw GM to exercise layer clippingGravatar robertphillips2014-10-02
| | | | | | | | This adds a "biglayer" variant that creates a picture with a single big layer. The tiling composition will test whether the layer hoisting code correctly caches the layers with their clipping information. BUG=skia:2315 Review URL: https://codereview.chromium.org/617723004
* BBHs: void* data -> unsigned dataGravatar mtklein2014-10-02
| | | | | | | | | | | | | | Now that the old backend's not using BBHs, we can specialize them for SkRecord's needs. The only thing we really want to store is op index, which should always be small enough to fit into an unsigned (unsigned also helps keep it straight from other ints floating around). This means we'll need half (32-bit) or a quarter (64-bit) the bytes in SkTileGrid, because we don't have to store an extra int for ordering. BUG=skia:2834 Review URL: https://codereview.chromium.org/617393004
* Remove duplicated code in android_get_gl_proc.Gravatar jungjik.lee2014-10-02
| | | | | | | | | | code for getting gles function pointer has been written twice. so this patch removes the last one. BUG=skia: Review URL: https://codereview.chromium.org/606413002
* Add blobshader GM baselines.Gravatar fmalita2014-10-02
| | | | | | | TBR=robertphillips@google.com NOTRY=true Review URL: https://codereview.chromium.org/616853003
* Revert of specialize setConcat for scale+translate (patchset #1 id:1 of ↵Gravatar robertphillips2014-10-02
| | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/620433002/) Reason for revert: Experimental to see if this revert "fixes" nanobench on the main console. Original issue's description: > specialize setConcat for scale+translate, helps drawText perf > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/e8db3ef2a2161084fa8eb1fd1d54c35bfff52d6c TBR=jvanverth@google.com,reed@google.com NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/620973003
* complete overhaul of webtry work partition between jail and hostGravatar humper2014-10-01
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/619943002
* fix prev change to tooBigForLCD, and add testGravatar reed2014-10-01
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/617353003
* remove suppression for gmGravatar reed2014-10-01
| | | | | | | TBR= NOTRY=True Review URL: https://codereview.chromium.org/617193003
* Don't adjust the bounds after a restore with the restore's paired ↵Gravatar mtklein2014-10-01
| | | | | | | | | | | | saveLayer's paint. It makes no sense for the paint from a saveLayer to effect anything outside its saveLayer/restore block. But as currently written, we'll adjust the clip bounds just after a restore by that paint. Turns out the test I wrote for the last CL (which caused this bug) actually had the bug baked into its expectations. I've updated them and added some notes to explain. BUG=418417 Review URL: https://codereview.chromium.org/623563002
* add baseline for clip_strokerectGravatar reed2014-10-01
| | | | | | | | BUG=skia: TBR= NOTRY=True Review URL: https://codereview.chromium.org/618373002
* baseline new GMsGravatar reed2014-10-01
| | | | | | | TBR= NOTRY=True Review URL: https://codereview.chromium.org/623443004
generated by cgit v1.2.3 (git 2.39.1) at 2024-07-05 00:25:14 +0000