aboutsummaryrefslogtreecommitdiffhomepage
Commit message (Collapse)AuthorAge
* Update NOPRESUBMIT docs.Gravatar mtklein2015-08-05
| | | | | | | | | | "NOPRESUBMIT=true" does not work anymore for me. BUG=skia: NOTRY=true DOCS_PREVIEW= https://skia.org/?cl=1271423002 Review URL: https://codereview.chromium.org/1271423002
* Scanline decoding for wbmpGravatar msarett2015-08-05
| | | | | | | | | | | | | | | We are also changing the wbmp to use SkSwizzler. This will allow us to take advantage of the sampling routines that are being implemented in SkSwizzler. The image in this upload came from: https://commons.wikimedia.org/wiki/File:Android_robot.png It is licensed under the Creative Commons Attribution-Share Alike 3.0 Unported license. BUG=skia: Review URL: https://codereview.chromium.org/1254483004
* Moved common cross-project functionality out from SkTraceEvent.h into ↵Gravatar oysteine2015-08-05
| | | | | | | | | | | | | | | | SkTraceEventCommon.h Traditionally, SkTraceEvent.h was manually kept in sync with base/trace_event/trace_event.h, with project-specific parts intermixed, and tended to drift out of sync a lot (mainly in Blink, less so in Skia). The SkTraceEventCommon.h now has only the cross-project parts, and can be copy-pasted verbatim between projects (it's an identical copy of base/trace_event/trace_event_common.h), meaning SkTraceEvent.h shouldn't go out of sync as it has before. BUG=skia: [mtklein mucking around below here] NOPRESUBMIT=true CQ_EXCLUDE_TRYBOTS=client.skia.fyi:skia_presubmit-Trybot Review URL: https://codereview.chromium.org/1270783002
* Add skstd::move and skstd::forward.Gravatar bungeman2015-08-05
| | | | | | | | | | Since we can't use the c++11 standard library, re-implement the bits we want to use now. TBR=reed@google.com The next CL is try to move SkTemplates into private. Review URL: https://codereview.chromium.org/1273813002
* Expose coord transforms from GrPipelineGravatar joshualitt2015-08-05
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1275603002
* This CL wires up the backend portion necessary for sending transformed ↵Gravatar joshualitt2015-08-05
| | | | | | | | coords via vertex attributes. BUG=skia: Review URL: https://codereview.chromium.org/1243583002
* Move impl of stroke rect batch from h to cppGravatar bsalomon2015-08-05
| | | | Review URL: https://codereview.chromium.org/1275633003
* Give strokerectbatch a proper homeGravatar bsalomon2015-08-05
| | | | Review URL: https://codereview.chromium.org/1274763002
* fix for GrFragmentProcessor isEqual in GrPipelineGravatar joshualitt2015-08-05
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1273693003
* Remove SHADER_AA_FILL_RECT flagGravatar robertphillips2015-08-05
| | | | Review URL: https://codereview.chromium.org/1271243004
* Update yasm .gyp for tweaked ASAN bots.Gravatar mtklein2015-08-05
| | | | | | | | | | | They're not passing -fsanitize=address anymore, rather -fsanitize=address,bool,etc,etc,etc... NOTREECHECKS=true BUG=skia: Review URL: https://codereview.chromium.org/1269383002
* Revert of Intentionally crash DM to make sure all bots are up-to-date. ↵Gravatar mtklein2015-08-05
| | | | | | | | | | | | | | | | | | | | | | | | (patchset #1 id:1 of https://codereview.chromium.org/1274693002/ ) Reason for revert: win x64 looks fixed Original issue's description: > Intentionally crash DM to make sure all bots are up-to-date. > > NOTREECHECKS=true > NOTRY=true > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/3749b9646103f544f17aa926fab050f1e5178b17 TBR=borenet@google.com NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/1275543003
* Fix make.py: BUILDTYPEGravatar Eric Boren2015-08-05
| | | | | | | | | TBR=mtklein BUG=skia: NOTRY=true NOTREECHECKS=true Review URL: https://codereview.chromium.org/1273793003 .
* Intentionally crash DM to make sure all bots are up-to-date.Gravatar Mike Klein2015-08-05
| | | | | | | | NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/1274693002 .
* Add tools/ubsan.supp.Gravatar Mike Klein2015-08-05
| | | | | | | | Should unbreak TSAN bots. BUG=skia: Review URL: https://codereview.chromium.org/1272213003 .
* init atomic bool in SkImage_GpuGravatar bsalomon2015-08-05
| | | | | | | | | | BUG=chromium:516860 TBR=reed@google.com NOTREECHECKS=true Review URL: https://codereview.chromium.org/1269243004
* Revert of Break LCD and Bitmap text dependency on hardcoded atlas values ↵Gravatar joshualitt2015-08-05
| | | | | | | | | | | | | | | | | | | | | | (patchset #2 id:20001 of https://codereview.chromium.org/1271873002/ ) Reason for revert: Breaks S3 Original issue's description: > Break LCD and Bitmap text dependency on hardcoded atlas values > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/846b022f6b469cfde285372f26e0d5c593d122ac TBR=jvanverth@google.com,bsalomon@google.com,joshualitt@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/1271163003
* Move atlas creation to internal to GrResourceProviderGravatar joshualitt2015-08-05
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1260183006
* - allow for stepping through a picture with 'n' and 'p'Gravatar reed2015-08-04
| | | | | | | | | - save current slide as .skp with 'K' BUG=skia: TBR= Review URL: https://codereview.chromium.org/1272063002
* Revert of IWYU: 'core' target, files starting A-C. (patchset #5 id:80001 of ↵Gravatar reed2015-08-04
| | | | | | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/1265033002/ ) Reason for revert: revert to unblock DEPS roll ../../chrome/browser/chromeos/display/overscan_calibrator.cc:43:10: error: variable has incomplete type 'SkPath' SkPath base_path; Original issue's description: > IWYU: 'core' target, files starting A-C. > > TBR=reed@google.com > Verbal lgtm, does not change API. > > Committed: https://skia.googlesource.com/skia/+/7403d87db8e43d4c2b5b25ac22a0ebc22bd09d69 TBR=reed@google.com,mtklein@google.com,bungeman@google.com NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/1273613002
* IWYU: 'core' target, files starting A-C.Gravatar bungeman2015-08-04
| | | | | | | TBR=reed@google.com Verbal lgtm, does not change API. Review URL: https://codereview.chromium.org/1265033002
* Whitespace change to test Gerrit ACLsGravatar borenet2015-08-04
| | | | | | | BUG=skia: TBR=mtklein Review URL: https://codereview.chromium.org/1255313004
* Port morphology to SkOpts.Gravatar mtklein2015-08-04
| | | | | | | | | | | | Nothing too fancy. Direction enums become enum classes so they don't get all confused. An alternative is to create one single Direction enum that both blur and morphology opts use. BUG=skia:4117 Review URL: https://codereview.chromium.org/1267343004
* Remove experimental/PdfViewerGravatar halcanary2015-08-04
| | | | | | | | Motivation: maintaining this code doesn't seem worth the time, since no one seems to be using it. If someone wants to use it in the future, just revert this CL. Review URL: https://codereview.chromium.org/1266093003
* SkPictureImageGeneratorGravatar fmalita2015-08-04
| | | | | | R=reed@google.com Review URL: https://codereview.chromium.org/1240093004
* Reorganize to keep similar code together.Gravatar Mike Klein2015-08-04
| | | | | | | | | This organizes memset16, memset32, and rsqrt the same way as the other code. No functional change. BUG=skia:4117 R=djsollen@google.com Review URL: https://codereview.chromium.org/1264423002 .
* whitespace change to test whether I can commitGravatar Eric Boren2015-08-04
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1268663005 .
* Remove dead code.Gravatar Mike Klein2015-08-04
| | | | | | | BUG=skia:4117 R=mtklein@google.com Review URL: https://codereview.chromium.org/1262213005 .
* Implement caching of filled paths in the tessellated path renderer.Gravatar senorblanco2015-08-04
| | | | | | | | | | | | | | | | | | | | | | | | | Paths are cached as tessellated triangle meshes in vertex buffers on the GPU. Stroked paths are not (yet) cached. Paths containing no curved segments (linear paths) are reused at all scales. Paths containing curved segments are reused within a scale tolerance threshold. In order to invalidate the cache when an SkPath is changed or deleted, this required implementing genID change notification in SkPath. This is modelled almost exactly on SkPixelRef::GenIDChangeListener. However, It does not currently implement the check for unique genIDs, so notifiers will fire when the first instance of an SkPathRef using a given genID is destroyed. Another caveat is that you cannot successfully add a change notifier to an empty path, since it uses the "canonical" empty path which is never modified or destroyed. For this reason, we prevent adding listeners to it. BUG=skia:4121,skia:4122, 497403 DOCS_PREVIEW= https://skia.org/?cl=1114353004 Committed: https://skia.googlesource.com/skia/+/468dfa72eb6694145487be17876804dfca3b7adb Review URL: https://codereview.chromium.org/1114353004
* Break LCD and Bitmap text dependency on hardcoded atlas valuesGravatar joshualitt2015-08-04
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1271873002
* Create a scanline decoder without creating a codecGravatar scroggo2015-08-04
| | | | | | | | | | | | | | Prior to this CL, if a client wanted to decode scanlines, they had to create an SkCodec in order to get an SkScanlineDecoder. This introduces complications if input data is not easily shared between the two objects. Instead, add methods to SkScanlineDecoder for creating a new one from input data, and remove the creation functions from SkCodec. Update DM and tests. Review URL: https://codereview.chromium.org/1267583002
* Add cheat sheet to wrangler docGravatar joshualitt2015-08-04
| | | | | | | | | BUG=skia: NOTRY=true DOCS_PREVIEW= https://skia.org/?cl=1264313003 TBR=bsalomon@google.com Review URL: https://codereview.chromium.org/1264313003
* Port SkBlurImage opts to SkOpts.Gravatar mtklein2015-08-04
| | | | | | | | | | | | +268 -535 lines I also rearranged the code a little bit to encapsulate itself better, mostly replacing static helper functions with lambdas. This also let me merge the SSE2 and SSE4.1 code paths. BUG=skia:4117 Review URL: https://codereview.chromium.org/1264103004
* Revert "Implement caching of filled paths in the tessellated path renderer."Gravatar mtklein2015-08-04
| | | | | | | | | | | | | | | | | | This reverts commit 468dfa72eb6694145487be17876804dfca3b7adb. This CL caused signficant GM diffs. Revert "Fix resource cache test." This reverts commit b001c41ed37d2018d38ce787b412ed741585d75d. Revert "Fix SkData leaks at GrResourceKey::setCustomData() call sites." This reverts commit c369348aa596d7be05c9ce0ca5d349e5d1903789. These CLs depend on the first one. BUG=skia: TBR=bsalomon@google.com Review URL: https://codereview.chromium.org/1260363007
* update image bench to test gpu and cpu images against raster surfaceGravatar reed2015-08-04
| | | | | | | BUG=513695 TBR= Review URL: https://codereview.chromium.org/1261043003
* cache private readback for gpu-imagesGravatar reed2015-08-04
| | | | | | | | | | | | Does not try to cache calls to readPixels at the moment: - not triggered by drawing - not clear if we want to perform any pixel transformations (that readPixels allows) on the GPU or CPU Can consider that another time. BUG=513695 Review URL: https://codereview.chromium.org/1262923003
* Remove IGNORE_ROT_AA_RECT_OPT flagGravatar robertphillips2015-08-04
| | | | | | It looks like the positive version was added (SHADER_AA_FILL_RECT) but the negative version wasn't removed. Review URL: https://codereview.chromium.org/1271903003
* Added registerChild; transforms, textures, glKey automatically handled.Gravatar wangyix2015-08-04
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1266633003
* Fix SkData leaks at GrResourceKey::setCustomData() call sites.Gravatar Stephen White2015-08-03
| | | | | | | BUG=skia: TBR=bsalomon@google.com Review URL: https://codereview.chromium.org/1270103003 .
* Revert of SkCanvas::onDrawPicture() quick-reject (patchset #3 id:40001 of ↵Gravatar mtklein2015-08-03
| | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/1264133003/ ) Reason for revert: webkit_unit_tests :( Original issue's description: > SkCanvas::onDrawPicture() quick-reject > > R=reed@google.com,mtklein@google.com > > Committed: https://skia.googlesource.com/skia/+/48ed62b29d45e42b971aac8858da06781c93e6d7 TBR=reed@google.com,fmalita@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/1273433002
* SkCanvas::onDrawPicture() quick-rejectGravatar fmalita2015-08-03
| | | | | | R=reed@google.com,mtklein@google.com Review URL: https://codereview.chromium.org/1264133003
* remove flags that control no codeGravatar reed2015-08-03
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1259653004
* Fix elliptical rrect clip shaders for large radii on devices with mediumpGravatar bsalomon2015-08-03
| | | | | | BUG=chromium:426217 Review URL: https://codereview.chromium.org/1256353004
* add bench for gpu-image -> cpu-surfaceGravatar reed2015-08-03
| | | | | | | BUG=513695 TBR= Review URL: https://codereview.chromium.org/1261663004
* Fix resource cache test.Gravatar Stephen White2015-08-03
| | | | | | TBR=bsalomon@chromium.org Review URL: https://codereview.chromium.org/1257253007 .
* Implement caching of filled paths in the tessellated path renderer.Gravatar senorblanco2015-08-03
| | | | | | | | | | | | | | | | | | | | | | | Paths are cached as tessellated triangle meshes in vertex buffers on the GPU. Stroked paths are not (yet) cached. Paths containing no curved segments (linear paths) are reused at all scales. Paths containing curved segments are reused within a scale tolerance threshold. In order to invalidate the cache when an SkPath is changed or deleted, this required implementing genID change notification in SkPath. This is modelled almost exactly on SkPixelRef::GenIDChangeListener. However, It does not currently implement the check for unique genIDs, so notifiers will fire when the first instance of an SkPathRef using a given genID is destroyed. Another caveat is that you cannot successfully add a change notifier to an empty path, since it uses the "canonical" empty path which is never modified or destroyed. For this reason, we prevent adding listeners to it. BUG=skia:4121,skia:4122, 497403 DOCS_PREVIEW= https://skia.org/?cl=1114353004 Review URL: https://codereview.chromium.org/1114353004
* remove getdevice guard for androidGravatar reed2015-08-03
| | | | | | | BUG=skia: NOTRY=True Review URL: https://codereview.chromium.org/1266363002
* Use new API everywhere for GrDefaultGeoProcFactoryGravatar joshualitt2015-08-03
| | | | | | | | BUG=skia: Committed: https://skia.googlesource.com/skia/+/801823b3d825d96baadeb9077c5ed1a8ada05cf2 Review URL: https://codereview.chromium.org/1261083003
* Missing overrides for Sinks in HWUI.Gravatar mtklein2015-08-03
| | | | | | | | See crrev.com/1263113002 BUG=skia:4138 Review URL: https://codereview.chromium.org/1263143004
* Add abliity to set textblob cache budget to GrContextGravatar joshualitt2015-08-03
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1264283002