aboutsummaryrefslogtreecommitdiffhomepage
Commit message (Collapse)AuthorAge
* make gm background colors 565 compatibleGravatar caryclark2015-06-15
| | | | | | | | | | | | | | | Change a batch of GM tests to convert their background color so that it is representable in 8888 and 565. Enable portable text in those same tests to minimize platform differences. In a couple of bitmap tests, use portable typefaces instead of choosing 'Times' which may or may not be available on the platform. R=borenet@google.com Review URL: https://codereview.chromium.org/1176243006
* SkGIFMovie: Fix corrupted background colorGravatar Takahiro.Aizawa2015-06-15
| | | | | | | | | | | | | | | | | | The 'paintingColor' in onGetBitmap() was used to save the entire gif's background color while filling the 1st frame of a gif image, since it is 'static' qualified, so when re-entering this function for subsequent frames, this saved value will be used directly without re-obtaining it. But if asynchronous playing multi gif files in single process without being controlled, this 'static' will make the 'paintingColor' corrupted, because the different SkGIFMovie instances hold the same reference of 'paintingColor'. Signed-off-by: Lu Tong <lu.x.tong@sonymobile.com>; BUG=skia: Review URL: https://codereview.chromium.org/1184083002
* SkPDF: Annotations are clipped by canvas clip stack.Gravatar halcanary2015-06-14
| | | | | | | | | | Also, remove some SkPDFDevice functions. Will fix this GM: http://crrev.com/1159273003 BUG=skia:3872 Review URL: https://codereview.chromium.org/1148263005
* Update SKP versionGravatar skia.buildbots2015-06-14
| | | | | | | | | Automatic commit by the RecreateSKPs bot. TBR= NO_MERGE_BUILDS Review URL: https://codereview.chromium.org/1183033002
* The correct case of 'sdl' directory in gyp.Gravatar bungeman2015-06-12
| | | | | The 'sdl' directory is lower case, but the gyp files refer to it with upper case.
* Import functionality for new GL backendGravatar cdalton2015-06-12
| | | | | | | | | | | | | | | | | | Brings in the following functionality: ARB_draw_instanced ARB_instanced_arrays NV_bindless_texture EXT_direct_state_access KHR_debug Also cleans up some of the NVPR extension loading. BUG=skia: Committed: https://skia.googlesource.com/skia/+/202c4c75fb806f751cf796850bf67fe9793e0643 Review URL: https://codereview.chromium.org/1185573003
* make drawlooper portableGravatar caryclark2015-06-12
| | | | | | TBR=djsollen@google.com Review URL: https://codereview.chromium.org/1187603002
* Visual bench on native androidGravatar joshualitt2015-06-12
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1164403002
* remove lcdGravatar caryclark2015-06-12
| | | | | | TBR=djsollen@google.com Review URL: https://codereview.chromium.org/1183753002
* Update DM to respect blacklists for testsGravatar djsollen2015-06-12
| | | | Review URL: https://codereview.chromium.org/1173363011
* Revert of Import functionality for new GL backend (patchset #2 id:20001 of ↵Gravatar djsollen2015-06-12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/1185573003/) Reason for revert: breaking multiple builds. For example... https://uberchromegw.corp.google.com/i/client.skia/builders/Test-Ubuntu-GCC-Golo-GPU-GT610-x86_64-Debug-ASAN/builds/667 Original issue's description: > Import functionality for new GL backend > > Brings in the following functionality: > > ARB_draw_instanced > ARB_instanced_arrays > NV_bindless_texture > EXT_direct_state_access > KHR_debug > > Also cleans up some of the NVPR extension loading. > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/202c4c75fb806f751cf796850bf67fe9793e0643 TBR=bsalomon@google.com,cdalton@nvidia.com NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/1184863002
* make emptypath portable; add 565 color utilityGravatar caryclark2015-06-12
| | | | | | R=djsollen@google.com Review URL: https://codereview.chromium.org/1182833004
* Stop running SKPs in DM on the bots.Gravatar mtklein2015-06-12
| | | | | | | | | We're not triaging them, and we get the same coverage (both Release and Debug) from nanobench. BUG=skia: Review URL: https://codereview.chromium.org/1178943006
* Print command to be executed in verbose modeGravatar djsollen2015-06-12
| | | | Review URL: https://codereview.chromium.org/1180163002
* Import functionality for new GL backendGravatar cdalton2015-06-12
| | | | | | | | | | | | | | | | Brings in the following functionality: ARB_draw_instanced ARB_instanced_arrays NV_bindless_texture EXT_direct_state_access KHR_debug Also cleans up some of the NVPR extension loading. BUG=skia: Review URL: https://codereview.chromium.org/1185573003
* Disable failing GPU tests on Nexus 6.Gravatar djsollen2015-06-12
| | | | | | BUG=skia:3942 Review URL: https://codereview.chromium.org/1185753002
* update complexclip to portableGravatar caryclark2015-06-12
| | | | | | TBR=djsollen@google.com Review URL: https://codereview.chromium.org/1179713007
* Fixing stripe testGravatar msarett2015-06-12
| | | | | | | | | | | | | | | | | I originally thought that there was no harm in reading or skipping zero lines after we have already reached the end of the image. However, once we reach the end of the image, onFinish() is automatically called. Performing a read or a skip after the call to onFinish() is invalid and will cause onFinish() to be called a second time (which is also invalid). Seems like the code requires good behavior and the test is wrong. BUG=skia: Review URL: https://codereview.chromium.org/1179213002
* Review URL: https://codereview.chromium.org/1166513002Gravatar cdalton2015-06-12
|
* Implement support for mixed sampled render targetsGravatar vbuzinov2015-06-12
| | | | | | | | | | | | | | | | | | Adds a new FBO type kStencil_MSFBOType that is selected whenever NV_framebuffer_mixed_samples extension is available. In this new FBO type a non-msaa color buffer is created with a multisampled stencil buffer attachment. Replaces numSamples() with separate numColorSamples and numStencilSamples methods in RenderTarget. In mixed samples mode non-MSAA codepaths are used to draw simple shapes, while NVPR-rendered paths and text are rendered with a multisampled stencil. BUG=skia:3177 Review URL: https://codereview.chromium.org/1001503002
* convert cubicpaths gm tests to portableGravatar caryclark2015-06-12
| | | | | | TBR=djsollen Review URL: https://codereview.chromium.org/1182113002
* Remove ambiguity of SkJpegUtility name.Gravatar mtklein2015-06-12
| | | | | | | | | We have two, one in images/, the other in codec/. As codec's the new hotness, I left it's name vanilla and suffixed the old one. BUG=skia: Review URL: https://codereview.chromium.org/1185733002
* Don't leak fragment processor in SkColorFilter::asFragmentProcessors ↵Gravatar bsalomon2015-06-12
| | | | | | | | implementations. TBR=reed@google.com Review URL: https://codereview.chromium.org/1182813002
* Move blend enums into GrBlend.hGravatar cdalton2015-06-12
| | | | | | | | | Moves blend enums from GrXferProcessor.h to GrBlend.h, makes GrBlend.h public. BUG=skia: Review URL: https://codereview.chromium.org/1180713008
* fix portable text in testGravatar caryclark2015-06-12
| | | | | | TBR=djsollen@google.com Review URL: https://codereview.chromium.org/1179173003
* SkiaLab doc: add another step for new Android devicesGravatar borenet2015-06-12
| | | | | | | | BUG=skia:3718 NOTRY=true DOCS_PREVIEW= https://skia.org/?cl=1185663003 Review URL: https://codereview.chromium.org/1185663003
* change Resize to take pixmap for srcGravatar reed2015-06-12
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1174293005
* remove unused private ResizerGravatar reed2015-06-12
| | | | | | | BUG=skia: TBR= Review URL: https://codereview.chromium.org/1177173003
* next batch of portable typeface changesGravatar caryclark2015-06-12
| | | | | | TBR=djsollen@google.com Review URL: https://codereview.chromium.org/1177303005
* DM testing for skipping and scalingGravatar msarett2015-06-11
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1175993005
* fix deserialization after FirstDirection change to pathsGravatar reed2015-06-11
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1185453003
* Update README.chromium.Gravatar bungeman2015-06-11
| | | | | | Capitalize Skia and use new site url. Review URL: https://codereview.chromium.org/1183503002
* Fix for dashing shader on Mali: use highp where necessary.Gravatar senorblanco2015-06-11
| | | | | | | | | | | The varyings used to compute rect coverage need to be high precision, since they're offset by the frag coord. R=egdaniel BUG=skia:3935 Review URL: https://codereview.chromium.org/1178123004
* Eliminate the check for ANDROID_LARGE_MEMORY in jpeg decodersGravatar msarett2015-06-11
| | | | | | BUG=skia:1282 Review URL: https://codereview.chromium.org/1103313002
* remove duplicate assign of fLastGlyphIDGravatar caryclark2015-06-11
| | | | | | | | Hal found a brain-dead bug on my part. Thanks Hal! TBR=halcanary@google.com Review URL: https://codereview.chromium.org/1176423003
* PDF expects fonts to map glyph ID 0 to the missing character.Gravatar caryclark2015-06-11
| | | | | | | | Add a mapping of glyph ID 0 to the toy font to allow for this. TBR=bungeman@google.com Review URL: https://codereview.chromium.org/1175243004
* retro add copyrightGravatar reed2015-06-11
| | | | | | | | BUG=skia: NOTRY=True TBR= Review URL: https://codereview.chromium.org/1183463002
* initialize font metrics for pdf (found by valgrind)Gravatar caryclark2015-06-11
| | | | | | TBR=halcanary@google.com Review URL: https://codereview.chromium.org/1175253004
* Init GrGLSLCaps fields from inside GrGLCapsGravatar cdalton2015-06-10
| | | | | | | | | Fixes circular dependencies by allowing the initialization to be done intertwined. BUG=skia: Review URL: https://codereview.chromium.org/1177053002
* SkPath::Direction serves two masters:Gravatar reed2015-06-10
| | | | | | | | | | | | | - input param to addFoo (e.g. addRect), where only CW or CCW are valid) - output param from computing functions, that sometimes return kUnknown This CL's intent is to split these into distinct enums/features: - Direction (public) loses kUnknown, and is only used for input - FirstDirection (private) is used for computing the first direction we see when analyzing a contour BUG=skia: Review URL: https://codereview.chromium.org/1176953002
* fix broken build by not passing the verbose flag throughGravatar Derek Sollenberger2015-06-10
| | | | Review URL: https://codereview.chromium.org/1176543004
* Remove verbose bash loggingGravatar djsollen2015-06-10
| | | | Review URL: https://codereview.chromium.org/1170283004
* Added GrAAFlatteningConvexPathRenderer.Gravatar ethannicholas2015-06-10
| | | | | | | | | This is an alternate version of GrAAConvexPathRenderer which handles curves by first flattening them to straight lines. BUG=skia: Review URL: https://codereview.chromium.org/1158803002
* Label some unused static vars as SK_UNUSEDGravatar fmalita2015-06-10
| | | | | | | | | | | | Makes my gcc 5.1.1 much happier. R=mtklein@google.com,scroggo@google.com [ mtklein mischief below here ] No public API changes. TBR=reed@google.com Review URL: https://codereview.chromium.org/1170863006
* turn on antialiased textGravatar caryclark2015-06-10
| | | | | | | | | Aliased text didn't work with msaa16, so make it all antialiased. TBR=reed@google.com Review URL: https://codereview.chromium.org/1169173006
* Today's gm include many differences caused by platform font implementations.Gravatar caryclark2015-06-10
| | | | | | | | | | | | | | | This experiment replaces the label used in the aaxfermodes gm with aliased text generated from paths common to all platforms. Since there is no way today to generate all dm output from trybots, this will be checked in to confirm that this strategy provides simpler output across devices. This does not introduce a new public interface; instead, dm uses a extern backdoor to install the SkTypeface::CreateFromName handler. Review URL: https://codereview.chromium.org/1163283002
* Update flags passed to Android GPU bots to exclude running skps and image ↵Gravatar djsollen2015-06-10
| | | | | | sources. Review URL: https://codereview.chromium.org/1176923003
* Remove overly-promiscuous SkNx syntax sugar.Gravatar mtklein2015-06-10
| | | | | | | | | | | | I haven't figured out a pithy way to have these apply to only classes originating from SkNx, so let's just remove them. There aren't too many use cases, and it's not really any less readable without them. Semantically, this is a no-op. BUG=skia: Review URL: https://codereview.chromium.org/1167153002
* Towards removing getTexture() on SkImageGravatar bsalomon2015-06-10
| | | | Review URL: https://codereview.chromium.org/1166993002
* Use SkMScalar as float by default to match Chromium.Gravatar mtklein2015-06-09
| | | | | | | | | | Tweak some test values to pass with floats. As expected, this regresses matrix44_setconcat_general by about 2x. BUG=skia: Review URL: https://codereview.chromium.org/1169813006