aboutsummaryrefslogtreecommitdiffhomepage
Commit message (Collapse)AuthorAge
* add color wheel to validate decoding colorsGravatar Hal Canary2014-10-24
| | | | | | | BUG=skia:3061 R=mtklein@google.com Review URL: https://codereview.chromium.org/681433002
* Revert of Added varying struct (patchset #9 id:160001 of ↵Gravatar joshualitt2014-10-24
| | | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/671023002/) Reason for revert: may have caused gm change on arm Original issue's description: > Added varying struct > > TBR= > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/852ae80b9c3c6fd53f993ac35133d80863993cbe TBR=bsalomon@google.com,joshualitt@chromium.org NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/675193002
* Add isScale() helper function to SkMatrix44.Gravatar tfarina2014-10-24
| | | | | | | | | | This will be used later in Chromium to cleanup gfx::Transform::IsScale2d(). BUG=408710,skia:997 TEST=None R=bsalomon@google.com,danakj@chromium.org Review URL: https://codereview.chromium.org/676583002
* rebaseline for new shaderprocGravatar mtklein2014-10-24
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/674123003
* Added varying structGravatar joshualitt2014-10-24
| | | | | | | TBR= BUG=skia: Review URL: https://codereview.chromium.org/671023002
* Rebaseline lots of images, associate some with bug 3063.Gravatar bsalomon2014-10-24
| | | | | | | TBR= NOTRY=true Review URL: https://codereview.chromium.org/661323003
* Set temporary paths volatile so we don't cache them.Gravatar jvanverth2014-10-24
| | | | | | | | | | Any path that is generated frame-to-frame should not be rendered by using the DistanceFieldPathRenderer, because generating the initial distance field, uploading it and rendering it takes longer than the SoftwarePathRenderer. BUG=skia:2935 Review URL: https://codereview.chromium.org/677463002
* Add test for new FrontBufferedStream behavior.Gravatar scroggo2014-10-24
| | | | | | | | | | | Test for https://skia.googlesource.com/skia/+/dd5a1e094c19fa10202c37c50a1f799e5af5dac0 Verify that FrontBufferedStream does not attempt to read beyond the end of its underlying stream. Make SkStreamToCGImageSource handle an empty stream better. Review URL: https://codereview.chromium.org/641813009
* Don't read random data in ICO check.Gravatar scroggo2014-10-24
| | | | | | Check to ensure SkStream::read succeeds before checking the result. Review URL: https://codereview.chromium.org/656673005
* Re-add in v8 to DEPS.Gravatar jcgregorio2014-10-24
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/673713004
* Update SKP versionGravatar skia-deps-roller2014-10-24
| | | | | | | | Automatic commit by the RecreateSKPs bot. TBR= Review URL: https://codereview.chromium.org/679693002
* Possible optimization to SkStringGravatar joshualitt2014-10-23
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/679433003
* Cleanup: Turn GrReducedClip into a class with a static function.Gravatar tfarina2014-10-23
| | | | | | | | | | | | | Clean up namespace usage. Similar to what was done in https://skia.googlesource.com/skia/+/a5414c4a8efc3119ee20fcee96c0bf68a04909c7 BUG=None TEST=None R=bsalomon@google.com Review URL: https://codereview.chromium.org/653393003
* Add flag to skimage for decoding into unpremul.Gravatar scroggo2014-10-23
| | | | Review URL: https://codereview.chromium.org/666073006
* squelch filterindiabox tooGravatar mtklein2014-10-23
| | | | | | | | TBR=bsalomon@google.com BUG=skia: Review URL: https://codereview.chromium.org/677463003
* Fix imagemagnifier GM quality and speed.Gravatar senorblanco2014-10-23
| | | | | | | | | | | | | | The imagemagnifier GM was applying the filter both to a saveLayer() restore() around the whole scene, as well as to each individual text draw. Applying the filter only in the saveLayer() makes it run ~20x faster on my Linux box, and the quality is improved as well (since the primitives are not double-filtered). BUG=skia: Review URL: https://codereview.chromium.org/637283009
* Revert "Revert of create shaderproc for nofilter-opaque-dx (patchset #7 ↵Gravatar mtklein2014-10-23
| | | | | | | | | | id:120001 of https://codereview.chromium.org/664783004/)" This reverts commit 430b795cc8a1cdbddd8fdc5511a3a523348937f7 and adds suppressions. BUG=skia: Review URL: https://codereview.chromium.org/673023002
* Don't use MIP filter mode on compressed texturesGravatar bsalomon2014-10-23
| | | | | | | | Add compressed texture image downsample GMs BUG=chromium:426331 Review URL: https://codereview.chromium.org/672863003
* Remove test expectations from ignored-tests.txt.Gravatar senorblanco2014-10-23
| | | | | | | | TBR=mtklein@google.com NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/659393006
* Adding an option to render only the shadow in SkDropShadowImageFilterGravatar sugoi2014-10-23
| | | | | | | | This is basically how blink uses the filter. Currently, I can't use it for "ShadowOnly" mode with the filter at all, but instead of copying the code and risking to have the codepaths diverge, I'm simply going to add the option here. BUG=skia: Review URL: https://codereview.chromium.org/646213004
* Rebaselines for tests affected by image filter PDF change.Gravatar senorblanco2014-10-23
| | | | | | | | | | | (aka https://codereview.chromium.org/644323006/) TBR=mtklein@google.com NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/671303003
* Add utils to better quantize grayscale values to three bit indices whileGravatar pavel2014-10-23
| | | | | | | | | | compressing coverage masks. Signed-off-by: Pavel Krajcevski <pavel@cs.unc.edu> BUG=skia: Review URL: https://codereview.chromium.org/669243003
* add float equivalent in dumpHex output as commentGravatar caryclark2014-10-23
| | | | | | R=reed@google.com,bsalomon@google.com Review URL: https://codereview.chromium.org/676803002
* Revert of create shaderproc for nofilter-opaque-dx (patchset #7 id:120001 of ↵Gravatar mtklein2014-10-23
| | | | | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/664783004/) Reason for revert: Many GMs fixed. Needs rebaseline, perhaps layout test rebaselines. Original issue's description: > create shaderproc for nofilter-opaque-dx > > > speedup nofilter > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/a40a276bcee2246439dcf816273c1307f5c3c69f TBR=djsollen@google.com,reed@google.com NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/656913005
* use SkMaskCache inside SkBlurMaskFilterGravatar reed2014-10-23
| | | | | | | | depends on https://codereview.chromium.org/670063004/ BUG=skia: Review URL: https://codereview.chromium.org/669993003
* Correct typo in SkRemotableFontMgr_win_dw.cpp.Gravatar bungeman2014-10-23
| | | | | | | New code was introduced behind a flag, but in that code there is a comma before a parameter name instead of after. Review URL: https://codereview.chromium.org/646323003
* create shaderproc for nofilter-opaque-dxGravatar reed2014-10-23
| | | | | | | | speedup nofilter BUG=skia: Review URL: https://codereview.chromium.org/664783004
* Revert of Add test for new FrontBufferedStream behavior. (patchset #1 id:1 ↵Gravatar mtklein2014-10-23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | of https://codereview.chromium.org/641813009/) Reason for revert: Test is SkASSERTing on Macs, e.g: http://chromegw.corp.google.com/i/client.skia/builders/Test-Mac10.6-MacMini4.1-GeForce320M-x86_64-Debug/builds/257/steps/dm/logs/stdio 210 tasks left 1737M peak 1ms test FrontBufferedStream../../src/ports/SkImageDecoder_CG.cpp:43: failed assertion "data" Signal 11: _sigtramp (+0x1a) SkStreamToCGImageSource(SkStream*) (+0x62) SkImageDecoder_CG::onDecode(SkStream*, SkBitmap*, SkImageDecoder::Mode) (+0x2e) SkImageDecoder::decode(SkStream*, SkBitmap*, SkColorType, SkImageDecoder::Mode) (+0x81) SkImageDecoder::DecodeStream(SkStreamRewindable*, SkBitmap*, SkColorType, SkImageDecoder::Mode, SkImageDecoder::Format*) (+0xff) SkImageDecoder::DecodeStream(SkStreamRewindable*, SkBitmap*) (+0x31) test_ShortFrontBufferedStream(skiatest::Reporter*) (+0x97) skiatest::ShortFrontBufferedStreamClass::onRun(skiatest::Reporter*) (+0x19) skiatest::Test::run() (+0x7c) DM::CpuTestTask::draw() (+0x5a) DM::CpuTask::run() (+0x9e) non-virtual thunk to DM::CpuTask::run() (+0x1c) (anonymous namespace)::ThreadPool::Loop(void*) (+0xf4) thread_start(void*) (+0x54) _pthread_start (+0x14b) Original issue's description: > Add test for new FrontBufferedStream behavior. > > Test for https://skia.googlesource.com/skia/+/dd5a1e094c19fa10202c37c50a1f799e5af5dac0 > > Verify that FrontBufferedStream does not attempt to read beyond the > end of its underlying stream. > > Committed: https://skia.googlesource.com/skia/+/da59f05c6738dbb9a92cad21c608cdfae53a76b2 TBR=reed@google.com,scroggo@google.com NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/649553003
* Allocate only the vertices we need for text.Gravatar jvanverth2014-10-23
| | | | | | | | | | This restructures the vertex allocation for text rendering to compute the max number of vertices we would need for a line of text, and then only allocate that much. If this number exceeds the quad index limit, then it will allocate for the max number of quads, and reallocate for the rest later. Review URL: https://codereview.chromium.org/663423003
* updating k1 expectationsGravatar joshualitt2014-10-23
| | | | | | | TBR= BUG=skia: Review URL: https://codereview.chromium.org/642543003
* Removed simple tests steps from CanvasTest.Gravatar piotaixr2014-10-23
| | | | | | | | | | | | | CanvasTest only checks if the state of the different canvas implementations are consistent with the one of the default implementation. Because most of the draw calls are stateless and already tested in GMs the test steps that exercices them are not useful. BUG=skia:3054 Review URL: https://codereview.chromium.org/672273002
* Remove android specific genID from SkPaint.Gravatar djsollen2014-10-23
| | | | Review URL: https://codereview.chromium.org/677453002
* Add test for new FrontBufferedStream behavior.Gravatar scroggo2014-10-23
| | | | | | | | | Test for https://skia.googlesource.com/skia/+/dd5a1e094c19fa10202c37c50a1f799e5af5dac0 Verify that FrontBufferedStream does not attempt to read beyond the end of its underlying stream. Review URL: https://codereview.chromium.org/641813009
* This return_false is now only used on Android. Squelch a warning.Gravatar mtklein2014-10-23
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/659393004
* Rename Nexus9 expectationsGravatar borenet2014-10-23
| | | | | | | BUG=skia:3047 TBR=djsollen Review URL: https://codereview.chromium.org/652003003
* Add SkMaskCacheGravatar qiankun.miao2014-10-23
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/670063004
* Revert of Delete unused method aliases and texture flag. (patchset #1 id:1 ↵Gravatar bsalomon2014-10-23
| | | | | | | | | | | | | | | | | | of https://codereview.chromium.org/672753002/) Reason for revert: Need to remove one more reference to this in blink before relanding. Original issue's description: > Delete unused method aliases and texture flag. > > Committed: https://skia.googlesource.com/skia/+/7fdffe41494cbd78a8ee2a6ef0509e46ec77039f TBR=robertphillips@google.com NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/671203002
* Update fontMgr to take list of bcp47 language tags.Gravatar bungeman2014-10-23
| | | | | | | | | This will enable clients to pass more than one bcp47 tag to ensure that the most appropriate font is selected. BUG=chromium:422180 Review URL: https://codereview.chromium.org/670243002
* Delete unused method aliases and texture flag.Gravatar bsalomon2014-10-23
| | | | Review URL: https://codereview.chromium.org/672753002
* fix memory leakGravatar joshualitt2014-10-22
| | | | | | | TBR= BUG=skia: Review URL: https://codereview.chromium.org/668333005
* some more expectations, update ignoresGravatar joshualitt2014-10-22
| | | | | | | TBR= BUG=skia: Review URL: https://codereview.chromium.org/668393002
* New baselinesGravatar joshualitt2014-10-22
| | | | | | | | TBR= BUG=skia: Review URL: https://codereview.chromium.org/668823006
* one more test to ignoreGravatar joshualitt2014-10-22
| | | | | | | TBR= BUG=skia: Review URL: https://codereview.chromium.org/674713002
* Oval and stroke AA rect now batchGravatar joshualitt2014-10-22
| | | | | | | | | | BUG=skia: Committed: https://skia.googlesource.com/skia/+/bc54fab1a4e5b51628a8c1557c62bc74e078870d Committed: https://skia.googlesource.com/skia/+/18055afb838a278b5a8436cd51dbfbb688e1e0a0 Review URL: https://codereview.chromium.org/664193002
* Revert of Oval and stroke AA rect now batch (patchset #7 id:110001 of ↵Gravatar joshualitt2014-10-22
| | | | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/664193002/) Reason for revert: whoops, accidental commit Original issue's description: > Oval and stroke AA rect now batch > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/bc54fab1a4e5b51628a8c1557c62bc74e078870d > > Committed: https://skia.googlesource.com/skia/+/18055afb838a278b5a8436cd51dbfbb688e1e0a0 TBR=robertphillips@google.com,bsalomon@google.com,joshualitt@chromium.org NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/675623002
* Oval and stroke AA rect now batchGravatar joshualitt2014-10-22
| | | | | | | | BUG=skia: Committed: https://skia.googlesource.com/skia/+/bc54fab1a4e5b51628a8c1557c62bc74e078870d Review URL: https://codereview.chromium.org/664193002
* Revert of Oval and stroke AA rect now batch (patchset #7 id:110001 of ↵Gravatar mtklein2014-10-22
| | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/664193002/) Reason for revert: Many GMs failing, e.g. http://build.chromium.org/p/client.skia.android/builders/Test-Android-Nexus5-Adreno330-Arm7-Release/builds/95/steps/Compare%20GMs/logs/stdio Original issue's description: > Oval and stroke AA rect now batch > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/bc54fab1a4e5b51628a8c1557c62bc74e078870d TBR=robertphillips@google.com,joshualitt@google.com,bsalomon@google.com,joshualitt@chromium.org NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/671993003
* Oval and stroke AA rect now batchGravatar joshualitt2014-10-22
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/664193002
* Add SkTypeface::getBounds()Gravatar reed2014-10-22
| | | | | | | | | mirrored from https://codereview.chromium.org/666303002/ BUG=skia: TBR= Review URL: https://codereview.chromium.org/676523002
* fix some bit-rot in the ios port of sampleappGravatar reed2014-10-22
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/637263004