aboutsummaryrefslogtreecommitdiffhomepage
Commit message (Collapse)AuthorAge
* Beginning to refactor nvpr codeGravatar joshualitt2014-10-31
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/687563008
* Some distance field text optimizations.Gravatar jvanverth2014-10-31
| | | | | | | | | | - Add clipRect check - Remove creation of scalerContext to check for color fonts in canDraw() (no longer needed) BUG=skia:2933 Review URL: https://codereview.chromium.org/696503004
* Revert of GrContext no longer ever draws on Gpu (patchset #7 id:120001 of ↵Gravatar joshualitt2014-10-31
| | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/683733005/) Reason for revert: breaks some android bots Original issue's description: > GrContext no longer ever draws on Gpu > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/7a75e5230872732e71ea99058d9afe5d4b6758b0 TBR=bsalomon@google.com,joshualitt@chromium.org NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/694983002
* GrContext no longer ever draws on GpuGravatar joshualitt2014-10-31
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/683733005
* Revert of Turn SkTaskGroups back on. (patchset #1 id:1 of ↵Gravatar mtklein2014-10-31
| | | | | | | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/687263007/) Reason for revert: precautionary revert. may have caused WinXP crashes on Chrome tree: https://code.google.com/p/chromium/issues/detail?id=429207 Original issue's description: > Turn SkTaskGroups back on. > > Revert "Disable SkTaskGroup in SkMultiPictureDraw temporarily." > Revert "Revert harder, removing SkTaskGroup.cpp from core temporarily." > > NOTREECHECKS=true > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/2100c5ed7a5e5470a04e7af7309d8bd3fc4249f7 TBR=reed@google.com,mtklein@chromium.org NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/693993002
* small fix for cache clearGravatar joshualitt2014-10-31
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/695873002
* SK_USE_SCALED_FONTMETRICS for correct scalingGravatar reed2014-10-31
| | | | | | | related blink change: https://codereview.chromium.org/691133002 Review URL: https://codereview.chromium.org/692083002
* Rebaseline GMs for https://codereview.chromium.org/687283002/Gravatar jvanverth2014-10-31
| | | | | | BUG=skia:2935 Review URL: https://codereview.chromium.org/684833003
* Objects should be unrefed instead of deletedGravatar sugoi2014-10-31
| | | | | | | | When an error occurs reading a flattenable object, it should be unrefed instead of deleted because, in the case of SkXferMode, for example, we'd actually be deleting a cached object kept in a static array. BUG=428789 Review URL: https://codereview.chromium.org/695633003
* Quick and dirty test to call the C interface from GO.Gravatar stephana2014-10-31
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/670303005
* Cleanup: Go with SkDebugf instead of GrPrintf.Gravatar tfarina2014-10-31
| | | | | | | | | | | Since GrPrintf is just defined to SkDebugf, we can go with the later directly. BUG=None TEST=None R=bsalomon@google.com Review URL: https://codereview.chromium.org/695663003
* Update SKP versionGravatar skia.buildbots2014-10-31
| | | | | | | | Automatic commit by the RecreateSKPs bot. TBR= Review URL: https://codereview.chromium.org/689833006
* Cleanup: Remove GR_ARRAY_COUNT macro.Gravatar tfarina2014-10-31
| | | | | | | | | | | | It is defined to SK_ARRAY_COUNT, and there are no uses of it. Lets just use the SK_ variant. BUG=None TEST=None R=bsalomon@google.com Review URL: https://codereview.chromium.org/691133003
* Add support for clang to the android build scriptsGravatar djsollen2014-10-31
| | | | Review URL: https://codereview.chromium.org/692953002
* Remove adb from Skia repositoryGravatar djsollen2014-10-31
| | | | Review URL: https://codereview.chromium.org/696463003
* Use Color Processor for color bitmap text draws on gpu.Gravatar egdaniel2014-10-31
| | | | | | | | | | | | | | | | | | | This change is here since previously color bitmap text was rendered using a geometry processor in the coverage stage. The problem with this is that we cannot correctly do xfer modes with this method. So I now make color bitmap text draw using a color stage in the same was as a draw bitmap call. One issue that arrises from this fix is that we end up adding this final color processor after any previous color processors. Thus if we have a custom blend implemented as a color processor it will be before this text one and we won't blend correctly. This issue will get fixed once an xfer processor is fully implemented. I have hacked a test locally to show that if we can add the text color processor to the begining of the color stages we do blend correctly in all cases (so the xfer processor will be a fix). BUG=skia: Review URL: https://codereview.chromium.org/689923004
* begin clean up for scalar is double optionGravatar reed2014-10-31
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/694693002
* Turn SkTaskGroups back on.Gravatar mtklein2014-10-31
| | | | | | | | | | | Revert "Disable SkTaskGroup in SkMultiPictureDraw temporarily." Revert "Revert harder, removing SkTaskGroup.cpp from core temporarily." NOTREECHECKS=true BUG=skia: Review URL: https://codereview.chromium.org/687263007
* Upgrade GPUOptimizeGravatar robertphillips2014-10-30
| | | | | | | | | | | | This will be a bit hairy to review. The FillBounds and CollectLayers code has diverged significantly resulting in the rendering path seeing different bounds than the hoisting path. This CL merges the FillBounds changes into CollectLayers. A follow on CL will, hopefully, find a way to layer CollectLayers on top of FillBounds. The only code in CollectLayers that is different from FillBounds is bracketed by "LAYER HOISTING" comments. NOTREECHECKS=true Review URL: https://codereview.chromium.org/685263004
* Trying to be consistent when members are allowed to be NULLGravatar sugoi2014-10-30
| | | | | | | | Got a few crashes running the fuzzer locally, all related to handling NULL members/parameters in an inconsistent way. BUG=skia: Review URL: https://codereview.chromium.org/675013003
* Update the Android NDK to r10cGravatar djsollen2014-10-30
| | | | Review URL: https://codereview.chromium.org/689943002
* Blind GM rebaseline for Galaxy S4Gravatar borenet2014-10-30
| | | | | | BUG=skia:3032 Review URL: https://codereview.chromium.org/694573002
* Cleanup conversion that the new NDK compiler tries to over optimize.Gravatar djsollen2014-10-30
| | | | | | | This allows us to upgrade to NDK r10c as the dftext gm no longer fails when compiled with -O2. Review URL: https://codereview.chromium.org/690053002
* Add return value on GrRecordReplaceDrawGravatar robertphillips2014-10-30
| | | | | | This assists debugging layer hoisting errors (e.g., when a layer is hoisted but not actually used in subsequent rendering). Review URL: https://codereview.chromium.org/694533004
* Fix member variable namingGravatar robertphillips2014-10-30
| | | | | | | | This renaming was missed in (MultiPictureDraw is taskgroup aware - https://codereview.chromium.org/684923002) TBR=reed@google.com Review URL: https://codereview.chromium.org/654293005
* Combine nameFontStyleProc and FindByNameStyle.Gravatar bungeman2014-10-30
| | | | | | These two functions do the same thing, combine them. Review URL: https://codereview.chromium.org/688133003
* Add skimage expectations for GalaxyS4.Gravatar scroggo2014-10-30
| | | | | | Fixes build. Review URL: https://codereview.chromium.org/692913003
* Make RTree handle the case where the playback canvas has empty clip boundsGravatar junov2014-10-30
| | | | | | BUG=skia:3074 Review URL: https://codereview.chromium.org/678303004
* Fix bug in aggressive layer cache purgingGravatar robertphillips2014-10-30
| | | | | | A picture may possess many layers that get placed in one plot of the atlas. In this case we can only remove the plot from the plotUsage tracking structure when all the layers belonging to the picture in that plot have been removed. Review URL: https://codereview.chromium.org/654463004
* Cleanup android to ensure it can compile with clangGravatar djsollen2014-10-30
| | | | Review URL: https://codereview.chromium.org/694533002
* allow SkUserConfig.h to control NOGPUGravatar reed2014-10-30
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/685243002
* Update SKP versionGravatar skia.buildbots2014-10-30
| | | | | | | | Automatic commit by the RecreateSKPs bot. TBR= Review URL: https://codereview.chromium.org/688793003
* tell our grcontext to abandon its backendGravatar reed2014-10-30
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/692803003
* Add SkDocument::getStream() method.Gravatar halcanary2014-10-30
| | | | | | | | | | Motivation: SkDocument_PDF can be refactored to stream all PDF objects as SkCanvas calls are made, to save memory! BUG=skia:3030 BUG=skia:2683 Review URL: https://codereview.chromium.org/691783002
* Revert of scale xmin and xmax for FontMetrics (patchset #1 id:1 of ↵Gravatar robertphillips2014-10-30
| | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/685153004/) Reason for revert: Experimental revert to see if this is blocking the DEPS roll Original issue's description: > scale xmin and xmax for FontMetrics > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/79c276e4fd93d9c9fd57a45bc6929c48e4415b2f TBR=mtklein@google.com,bungeman@google.com,reed@google.com,reed@chromium.org NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/691033002
* Revert harder, removing SkTaskGroup.cpp from core temporarily.Gravatar mtklein2014-10-30
| | | | | | | | | Not all linkers are seeing the SkThread code is not needed. BUG=skia: NOTREECHECKS=true Review URL: https://codereview.chromium.org/673903007
* Disable SkTaskGroup in SkMultiPictureDraw temporarily.Gravatar mtklein2014-10-30
| | | | | | | | Think this should be enough to get rolling again. NOTREECHECKS=true Review URL: https://codereview.chromium.org/687273006
* Use the most recent version of ADB from the SDK if necessary.Gravatar djsollen2014-10-30
| | | | | | | This removes ADB from the Skia checkout and also does some quick checks to make sure the version of ADB used meets a minimum threshold Review URL: https://codereview.chromium.org/692513003
* Support multiple scales for dfpaths.Gravatar jvanverth2014-10-30
| | | | | | | | | Adds miplevel as part of dfpath key, and scale factor so we know how much to adjust to fit desired scale. BUG=skia:2935 Review URL: https://codereview.chromium.org/687283002
* Increase wait time for spreadsheetdashed pagesetGravatar rmistry2014-10-30
| | | | | | | | BUG=skia:3049 TBR= NOTRY=true Review URL: https://codereview.chromium.org/692843002
* Whitespace change to see how the Android bots behaveGravatar borenet2014-10-30
| | | | | | | | | NOTRY=true TBR=rmistry NOTREECHECKS=true BUG=skia: Review URL: https://codereview.chromium.org/693483003
* Increase timeout of blogger pageset and re-enable weather pagesetGravatar Ravi Mistry2014-10-30
| | | | BUG=skia:3049
* break transitions into separate fileGravatar reed2014-10-29
| | | | | | | | BUG=skia: TBR= NOTRY=True Review URL: https://codereview.chromium.org/692543004
* cleanup reentrance logic in CMMGravatar joshualitt2014-10-29
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/684223002
* Update SKP versionGravatar skia.buildbots2014-10-29
| | | | | | | | Automatic commit by the RecreateSKPs bot. TBR= Review URL: https://codereview.chromium.org/665233005
* Revert of Use SkTypeface::getBounds() in bounding-box calculations. ↵Gravatar mtklein2014-10-29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | (patchset #5 id:80001 of https://codereview.chromium.org/680363003/) Reason for revert: Mac mini asserting Original issue's description: > Use SkTypeface::getBounds() in bounding-box calculations. > > This should produce tighter conservative bounding boxes for text than the > approximation code it replaces. > > Recording performance is neutral on my desktop. Playback performance > improves by up to 15% on text heavy pages, e.g. > > desk_pokemonwiki.skp_1 3.24ms -> 2.83ms 0.87x > desk_baidu.skp_1 1.91ms -> 1.58ms 0.83x > > Committed: https://skia.googlesource.com/skia/+/bf8dc343df4fbdcb8af546eb68b640e011a33489 > > CQ_EXTRA_TRYBOTS=client.skia:Test-Win7-ShuttleA-HD2000-x86-Debug-Trybot > > Committed: https://skia.googlesource.com/skia/+/c51add674dfb89b988a7fbc05f41838c203f9dcd TBR=reed@google.com,mtklein@chromium.org NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/690833002
* use SkTaskGroup::batchGravatar reed2014-10-29
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/688753002
* Use SkTypeface::getBounds() in bounding-box calculations.Gravatar mtklein2014-10-29
| | | | | | | | | | | | | | | | | This should produce tighter conservative bounding boxes for text than the approximation code it replaces. Recording performance is neutral on my desktop. Playback performance improves by up to 15% on text heavy pages, e.g. desk_pokemonwiki.skp_1 3.24ms -> 2.83ms 0.87x desk_baidu.skp_1 1.91ms -> 1.58ms 0.83x Committed: https://skia.googlesource.com/skia/+/bf8dc343df4fbdcb8af546eb68b640e011a33489 CQ_EXTRA_TRYBOTS=client.skia:Test-Win7-ShuttleA-HD2000-x86-Debug-Trybot Review URL: https://codereview.chromium.org/680363003
* Increase timeouts of wowwiki and youtubeGravatar Ravi Mistry2014-10-29
|
* SkTaskGroup::batch(fn, args, N)Gravatar mtklein2014-10-29
| | | | | | | | Porting QuiltTask isn't important in itself; this is mostly an API feeler. BUG=skia: Review URL: https://codereview.chromium.org/689673003