aboutsummaryrefslogtreecommitdiffhomepage
Commit message (Collapse)AuthorAge
* Update SKP versionGravatar skia.buildbots2015-02-05
| | | | | | | | Automatic commit by the RecreateSKPs bot. TBR= Review URL: https://codereview.chromium.org/901403002
* migrate more samples over to SkAnimTImerGravatar reed2015-02-05
| | | | | | | BUG=skia: TBR= Review URL: https://codereview.chromium.org/901933004
* Convex batchGravatar joshualitt2015-02-05
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/880643002
* Revert of Move DstCopy on gpu into the GrXferProcessor. (patchset #11 ↵Gravatar bsalomon2015-02-05
| | | | | | | | | | | | | | | | | | | | | | | | id:200001 of https://codereview.chromium.org/885923002/) Reason for revert: Testing to see if reverting fixes 10.9 bots. Original issue's description: > Move DstCopy on gpu into the GrXferProcessor. > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/74a11753604768bf461b80cabb66060e8564d82c > > Committed: https://skia.googlesource.com/skia/+/5e1378d0e075a323144ba14e0a4cbcca35eccc69 TBR=joshualitt@google.com,egdaniel@google.com NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/901663007
* whitespaceGravatar joshualitt2015-02-05
| | | | | | | | NOTRY=True TBR= BUG=skia: Review URL: https://codereview.chromium.org/881413006
* Move DstCopy on gpu into the GrXferProcessor.Gravatar egdaniel2015-02-05
| | | | | | | | BUG=skia: Committed: https://skia.googlesource.com/skia/+/74a11753604768bf461b80cabb66060e8564d82c Review URL: https://codereview.chromium.org/885923002
* Document my current understanding of C++11 in Skia.Gravatar mtklein2015-02-05
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/901053007
* GM for http://skbug.com/3398Gravatar halcanary2015-02-05
| | | | | | BUG=skia:3398 Review URL: https://codereview.chromium.org/896093005
* Try again to use a C++11 feature in DM as a canary.Gravatar mtklein2015-02-05
| | | | | | | | CQ_EXTRA_TRYBOTS=client.skia.compile:Build-Ubuntu13.10-GCC4.8-NaCl-Release-Trybot,Build-Mac10.7-Clang-Arm7-Debug-iOS-Trybot BUG=skia: Review URL: https://codereview.chromium.org/898863003
* Build in C++11 mode on Unix-like bots.Gravatar mtklein2015-02-05
| | | | | | | | | | | | | | | | Mac and Windows bots are already building in C++11 mode. This turns on the rest, mostly to see what work remains. This will probably break a few bots. It'd be nice if we could let those all come in as red before reverting this so I can see the full list to fix. BUG=skia: Committed: https://skia.googlesource.com/skia/+/779e49602a9c8f4d2799504822e01bcafbcaa534 CQ_EXTRA_TRYBOTS=client.skia.compile:Build-Ubuntu13.10-GCC4.8-NaCl-Release-Trybot,Build-Mac10.7-Clang-Arm7-Debug-iOS-Trybot Review URL: https://codereview.chromium.org/868233008
* Suppress another FontConfig 'leak'.Gravatar bungeman2015-02-05
| | | | | | | | | | | | FontConfig uses offsets for elements instead of pointers, so any value held by an element goes missing from memory checkers. Previous suppressions took care of rule copies and adds to elements, but value lists may be copied as well. When they are, values are copied, including the content of strings and matricies. Since value lists are effectively like Fc*Add functions (even calling the same underlying helpers), treat them as such and suppress any 'leaks' they may cause. Review URL: https://codereview.chromium.org/894863003
* BUG=skia:Gravatar joshualitt2015-02-05
| | | | Review URL: https://codereview.chromium.org/894693003
* Update SKP versionGravatar skia.buildbots2015-02-04
| | | | | | | | Automatic commit by the RecreateSKPs bot. TBR= Review URL: https://codereview.chromium.org/901153002
* Fix GIF transparency.Gravatar jei.mayol2015-02-04
| | | | | | | | | | | | | | | | | | | The GIF decoder, in the onDecode() function, tries to obtain the transparency info from the temporary SavedImage object, temp_save, but the decoder had been changed to save extension block data to the GifFileType object instead. As a result, find_transpIndex() can't find the correct transaprency index and returns -1. This had been reported in Android 4.4. AOSP pulled skia changes related to giflib 5.x changes after Android 4.3, which is why the problem is not reproducible in versions before KitKat. AOSP Issue 62016: https://code.google.com/p/android/issues/detail?id=62016 BUG=skia: Review URL: https://codereview.chromium.org/429053004
* force a buildGravatar mtklein2015-02-04
| | | | | | | | BUG=skia: NOTRY=true TBR= Review URL: https://codereview.chromium.org/899503002
* add length parameter to addText, to match attr valuesGravatar reed2015-02-04
| | | | | | BUG=skia:3392 Review URL: https://codereview.chromium.org/896363002
* [SkSVGDevice] drawTextOnPath() supportGravatar fmalita2015-02-04
| | | | | | R=reed@google.com,mtklein@google.com Review URL: https://codereview.chromium.org/874853003
* [SkSVGDevice] Add support for more stroke paramsGravatar fmalita2015-02-04
| | | | | | | | stroke-linecap, stroke-linejoin, stroke-miterlimit, non-scaling-stroke. R=mtklein@google.com Review URL: https://codereview.chromium.org/897953004
* More cleanup around GrContext, textures, and SkGr.cppGravatar bsalomon2015-02-04
| | | | Review URL: https://codereview.chromium.org/880983008
* No more caching volatile bitmapsGravatar sugoi2015-02-04
| | | | | | | | | | VideoFrames are always marked as volatile and load_yuv_texture() was caching everything, even volatile bitmaps. To solve this issue, we no longer cache volatile bitmaps' YUV planes in load_yuv_texture(). There is another issue which cause this to happen, related to how VideoImageGenerator::onGetYUV8Planes() is implemented, which is logged here: crbug.com/455235 BUG=450706, 450699 Review URL: https://codereview.chromium.org/900943002
* Fix casting bug in GrArithmeticXPGravatar egdaniel2015-02-04
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/858373006
* Make npot resizing work properly for bmps that are explicitly texture backed.Gravatar bsalomon2015-02-04
| | | | Review URL: https://codereview.chromium.org/890223003
* return reference to cache instead of copying the maskGravatar reed2015-02-04
| | | | | | BUG=437128 Review URL: https://codereview.chromium.org/889303005
* [SkSVGDevice] Initial clipping supportGravatar fmalita2015-02-04
| | | | | | | | | | Implement SVG clips based on clip stack flattening - which is now exposed in SkClipStack::asPath() and shared with SkCanvas's simplify-clip code. R=reed@google.com,mtklein@google.com Review URL: https://codereview.chromium.org/876923003
* Revert of Build in C++11 mode on Unix-like bots. (patchset #4 id:60001 of ↵Gravatar mtklein2015-02-04
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/868233008/) Reason for revert: Perf-Ubuntu12, and Test-Ubuntu12, Build-Nacl all too old. Android and Chrome OS builders look ok. Android testers look ok. Chrome OS testers haven't run yet. Original issue's description: > Build in C++11 mode on Unix-like bots. > > Mac and Windows bots are already building in C++11 mode. > This turns on the rest, mostly to see what work remains. > > This will probably break a few bots. It'd be nice if we could let those > all come in as red before reverting this so I can see the full list to fix. > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/779e49602a9c8f4d2799504822e01bcafbcaa534 TBR=stephana@google.com,mtklein@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/879803003
* Build in C++11 mode on Unix-like bots.Gravatar mtklein2015-02-04
| | | | | | | | | | | | Mac and Windows bots are already building in C++11 mode. This turns on the rest, mostly to see what work remains. This will probably break a few bots. It'd be nice if we could let those all come in as red before reverting this so I can see the full list to fix. BUG=skia: Review URL: https://codereview.chromium.org/868233008
* Revert "Move DstCopy on gpu into the GrXferProcessor."Gravatar egdaniel2015-02-04
| | | | | | | | | | | | | | This reverts commit 74a11753604768bf461b80cabb66060e8564d82c. TBR=joshualitt@google.com,bsalomon@google.com NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia: Committed: https://skia.googlesource.com/skia/+/3e9dfdb3784c0cbfecf7589a74aa9aff7ef40abd Review URL: https://codereview.chromium.org/896163003
* Revert of Revert "Move DstCopy on gpu into the GrXferProcessor." (patchset ↵Gravatar egdaniel2015-02-04
| | | | | | | | | | | | | | | | | | | | | | | | | | | | #1 id:1 of https://codereview.chromium.org/896163003/) Reason for revert: failed on my manual revert Original issue's description: > Revert "Move DstCopy on gpu into the GrXferProcessor." > > This reverts commit 74a11753604768bf461b80cabb66060e8564d82c. > > TBR=joshualitt@google.com,bsalomon@google.com > NOPRESUBMIT=true > NOTREECHECKS=true > NOTRY=true > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/3e9dfdb3784c0cbfecf7589a74aa9aff7ef40abd TBR= NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/900913002
* Revert "Move DstCopy on gpu into the GrXferProcessor."Gravatar egdaniel2015-02-04
| | | | | | | | | | | | This reverts commit 74a11753604768bf461b80cabb66060e8564d82c. TBR=joshualitt@google.com,bsalomon@google.com NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/896163003
* Fix the URL for previewing Markdown changes.Gravatar tfarina2015-02-04
| | | | | | | | BUG=None R=jcgregorio@google.com NOTRY=true Review URL: https://codereview.chromium.org/883003004
* Update SKP versionGravatar skia.buildbots2015-02-03
| | | | | | | | Automatic commit by the RecreateSKPs bot. TBR= Review URL: https://codereview.chromium.org/876313006
* Remove unused and unimplemented abstract functions from GrRenderTargetGravatar kkinnunen2015-02-03
| | | | | | | | | Remove unused and unimplemented abstract functions from GrRenderTarget. The functions might cause confusion later. BUG=skia:3388 Review URL: https://codereview.chromium.org/865603007
* [SkSVGDevice] Initial text supportGravatar fmalita2015-02-03
| | | | | | R=reed@google.com,mtklein@google.com Review URL: https://codereview.chromium.org/899683002
* fixing GrBatch leakGravatar joshualitt2015-02-03
| | | | | | | TBR= BUG=skia: Review URL: https://codereview.chromium.org/886393008
* Hairline batchGravatar joshualitt2015-02-03
| | | | | | | | | | BUG=skia: Committed: https://skia.googlesource.com/skia/+/6eff8701f027016fbb3147412ec2292dcec2b7f5 Committed: https://skia.googlesource.com/skia/+/658d55cd6121c67488aaf5d0832c9712737f26a5 Review URL: https://codereview.chromium.org/876673002
* Dash using vertex attributes for interval and stroke widthGravatar joshualitt2015-02-03
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/892203003
* Cleanup the android scripts.Gravatar djsollen2015-02-03
| | | | | | | Rename a few files to make their function clearer. Update other files to remove dead code or improve function. Review URL: https://codereview.chromium.org/865943007
* Move DstCopy on gpu into the GrXferProcessor.Gravatar egdaniel2015-02-03
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/885923002
* add textual overviewGravatar reed2015-02-03
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/898573002
* Port SkRefCnt.h to new SkAtomics.hGravatar mtklein2015-02-03
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This adds sk_memory_barrier(), implemented using sk_atomic_fetch_add() on an uninitialized variable. If that becomes a problem we can drop this to the porting layer, using std::atomic_thread_fence() / __atomic_thread_fence() / __sync_synchronize(). The big win is that ref() doesn't generate a memory barrier any more on ARM. This is an instance of SkSafeRef() in SkPaint(const SkPaint&) after this CL: 4d0: 684a ldr r2, [r1, #4] 4d2: 6018 str r0, [r3, #0] 4d4: b13a cbz r2, 4e6 <_ZN7SkPaintC1ERKS_+0x2e> 4d6: 1d10 adds r0, r2, #4 4d8: e850 4f00 ldrex r4, [r0] 4dc: 3401 adds r4, #1 4de: e840 4500 strex r5, r4, [r0] 4e2: 2d00 cmp r5, #0 4e4: d1f8 bne.n 4d8 <_ZN7SkPaintC1ERKS_+0x20> Here's the before, pretty much the same with two memory barriers surrounding the ref(): 4d8: 684a ldr r2, [r1, #4] 4da: 6018 str r0, [r3, #0] 4dc: b15a cbz r2, 4f6 <_ZN7SkPaintC1ERKS_+0x3e> 4de: 1d10 adds r0, r2, #4 4e0: f3bf 8f5f dmb sy 4e4: e850 4f00 ldrex r4, [r0] 4e8: 3401 adds r4, #1 4ea: e840 4500 strex r5, r4, [r0] 4ee: 2d00 cmp r5, #0 4f0: d1f8 bne.n 4e4 <_ZN7SkPaintC1ERKS_+0x2c> 4f2: f3bf 8f5f dmb sy The miscellaneous files in here are just fixups to explicitly include SkMutex.h, instead of leeching it off SkRefCnt.h. No public API changes. TBR=reed@google.com Build trybots seem hosed. NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/896803002
* Build in C++11 mode on Macs.Gravatar mtklein2015-02-03
| | | | | | | | | Build trybots are not triggering. NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/894773002
* Remove link to Push application.Gravatar tfarina2015-02-03
| | | | | | | | BUG=None R=jcgregorio@google.com NOTRY=true Review URL: https://codereview.chromium.org/884203002
* remove remaining parts of SkExampleGravatar caryclark2015-02-03
| | | | | | R=reed@google.com Review URL: https://codereview.chromium.org/895103002
* add new Skia team member to committers listGravatar hcm2015-02-03
| | | | | | | | | BUG=skia: NOTRY=true TBR=rmistry@google.com Review URL: https://codereview.chromium.org/885493008
* Save and temp dst and src FBO on GrGLGpu and use the temp FBOs for ↵Gravatar egdaniel2015-02-03
| | | | | | | | copySurface if needed. BUG=skia: Review URL: https://codereview.chromium.org/893703003
* Revert of Hairline batch (patchset #17 id:360001 of ↵Gravatar joshualitt2015-02-03
| | | | | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/876673002/) Reason for revert: still a performance regression Original issue's description: > Hairline batch > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/6eff8701f027016fbb3147412ec2292dcec2b7f5 > > Committed: https://skia.googlesource.com/skia/+/658d55cd6121c67488aaf5d0832c9712737f26a5 TBR=bsalomon@google.com,joshualitt@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/882883003
* Land Ben's 32-bit symlink script.Gravatar Mike Klein2015-02-03
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/890613005
* Fix references to https://sites.google.com/site/skiadocs/.Gravatar Thiago Farina2015-02-03
| | | | | | | | | | Point them to their equivalent in https://skia.org. BUG=None R=hcm@google.com, jcgregorio@google.com NOTRY=true Review URL: https://codereview.chromium.org/856353004
* Hairline batchGravatar joshualitt2015-02-03
| | | | | | | | BUG=skia: Committed: https://skia.googlesource.com/skia/+/6eff8701f027016fbb3147412ec2292dcec2b7f5 Review URL: https://codereview.chromium.org/876673002
* [SkSVGDevice] Initial shader/gradient supportGravatar fmalita2015-02-03
| | | | | | | | | | * linear gradient support (based on shawcroft@google.com's CL) * generic paint resources reorg * opacity support R=reed@google.com,mtklein@google.com,halcanary@google.com Review URL: https://codereview.chromium.org/892973002